Closed
Bug 1375956
Opened 7 years ago
Closed 7 years ago
Disable the browser UI in more Talos tests that shouldn't be exercising it.
Categories
(Testing :: Talos, enhancement, P1)
Testing
Talos
Tracking
(firefox56 fixed)
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: mconley, Assigned: mconley)
References
Details
(Whiteboard: [photon-performance][qa-])
Attachments
(1 file)
Similar to bug 1374765, there are more Talos tests where it doesn't make sense to capture behaviour in the browser UI within the context of the tests.
Those tests are:
1. tsvgr_opacity
2. glterrain
3. dromaeo (dom and css)
4. tscrollx
5. basic_compositor_video
6. glvideo
7. a11yr
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → mconley
Assignee | ||
Updated•7 years ago
|
Status: NEW → ASSIGNED
Whiteboard: [photon-performance]
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•7 years ago
|
||
Just a heads up igoldan - this just changes the conditions of the test. Any Talos regressions or improvements from this landing should be ignored.
Flags: needinfo?(ionut.goldan)
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8880910 [details]
Bug 1375956 - Disable browser UI for more Talos tests that should not care about changes in the browser UI.
https://reviewboard.mozilla.org/r/152276/#review157330
this was an easy review, please ensure talos runs fine |mach try -b o -p win32,win64,macosx64,linux64 -u none -t all --rebuild 3| works fine
Attachment #8880910 -
Flags: review?(jmaher) → review+
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/67dca71a25c4
Disable browser UI for more Talos tests that should not care about changes in the browser UI. r=jmaher
Updated•7 years ago
|
Iteration: --- → 56.1 - Jun 26
Priority: -- → P1
Comment 5•7 years ago
|
||
this got landed in the lucky window of a broken build:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=84467f2e7e673ba1ba9995766c4a632f80e10492&tochange=a900b7e7572b52618dd36b645daae1767139cf82
but given the failures and the few patches, it looks like we see a couple regressions:
== Change summary for alert #7520 (as of June 23 2017 22:49 UTC) ==
Regressions:
15% tscrollx summary osx-10-10 opt e10s 3.33 -> 3.81
4% basic_compositor_video summary linux64 pgo e10s3.69 -> 3.84
For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=7520
There has been almost no data since this landed, so I would expect other change to show up and we will associate them with that alert.
Please comment if the numbers seem unexpected.
Flags: needinfo?(ionut.goldan)
Comment 6•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Assignee | ||
Comment 7•7 years ago
|
||
These regressions are certainly due to the fact that we increased the size of the content area, and can be ignored.
Assignee | ||
Updated•7 years ago
|
Whiteboard: [photon-performance] → [photon-performance][qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•