Closed
Bug 1354463
Opened 8 years ago
Closed 8 years ago
Enable 'layers.advanced.canvas-background-color' by default
Categories
(Core :: Graphics: WebRender, enhancement)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: ethlin, Assigned: mtseng)
References
Details
(Keywords: correctness)
Attachments
(4 files)
Reporter | ||
Updated•8 years ago
|
Blocks: 1342343
Keywords: correctness
Assignee | ||
Comment 1•8 years ago
|
||
Reporter | ||
Comment 2•8 years ago
|
||
current try result: https://treeherder.mozilla.org/#/jobs?repo=try&revision=847d2b8969d3edf2dda1449f2b773d35620c668b&selectedJob=100832168
Assignee: ethlin → mtseng
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 6•8 years ago
|
||
Comment 7•8 years ago
|
||
mozreview-review |
Comment on attachment 8874773 [details]
Bug 1354463 - Add fuzzy-if for tests.
https://reviewboard.mozilla.org/r/146146/#review150126
Please change all of these fuzzy annotations to be ranges, e.g. fuzzy-if(webrender,15-15,1000-1000) so that if the fuzziness level drops we will find out and can tighten the bounds.
Attachment #8874773 -
Flags: review?(bugmail) → review+
Comment 8•8 years ago
|
||
mozreview-review |
Comment on attachment 8874772 [details]
Bug 1354463 - Set fails-if for opaque layer check fail.
https://reviewboard.mozilla.org/r/146144/#review150130
Please make sure to update the webrender reftest spreadsheet with these and mark them as WONTFIX since the failures are expected.
Attachment #8874772 -
Flags: review?(bugmail) → review+
Comment 9•8 years ago
|
||
mozreview-review |
Comment on attachment 8874771 [details]
Bug 1354463 - Enable layers.advanced.canvas-background-color if webrender is enabled.
https://reviewboard.mozilla.org/r/146142/#review150132
Attachment #8874771 -
Flags: review?(bugmail) → review+
Comment 10•8 years ago
|
||
mozreview-review |
Comment on attachment 8874771 [details]
Bug 1354463 - Enable layers.advanced.canvas-background-color if webrender is enabled.
https://reviewboard.mozilla.org/r/146142/#review150134
Typo in commit message: first "is" should be "if"
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 14•8 years ago
|
||
Thanks for review. Try looks good.
Here is try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=bf54341ace7793a56239a0500836c7bc073ee343&group_state=expanded
Comment 15•8 years ago
|
||
Pushed by mtseng@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4ac4ad3ba687
Enable layers.advanced.canvas-background-color if webrender is enabled. r=kats
https://hg.mozilla.org/integration/autoland/rev/16d73b417ed5
Set fails-if for opaque layer check fail. r=kats
https://hg.mozilla.org/integration/autoland/rev/b884346aefbe
Add fuzzy-if for tests. r=kats
Comment 16•8 years ago
|
||
Is there a reason you used ranges like 950-1000 instead of a more precise range? Also in some cases you had ranges starting at 0, like "fuzzy-if(webrender,0-1,0-600)" - is this an intermittent? The only reason we should have ranges like this is if the tests are nondeterministic, and if that's the case that worries me.
Assignee | ||
Comment 17•8 years ago
|
||
Yap, I think precise range is better. I'll attach a patch for fixing this. And about fuzzy-if(webrender,0-1,0-600), it seems the different result in different configurations such as e10s v.s. non-e10s, debug v.s. opt...
Assignee | ||
Comment 18•8 years ago
|
||
MozReview-Commit-ID: Cf0d2ZjCMHb
Attachment #8875586 -
Flags: review?(bugmail)
Assignee | ||
Comment 19•8 years ago
|
||
Assignee | ||
Comment 20•8 years ago
|
||
leave the bug open until follow-up patch landed.
Keywords: leave-open
Comment 21•8 years ago
|
||
Comment on attachment 8875586 [details] [diff] [review]
Use precise range for fuzzy-if.
Review of attachment 8875586 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks, this is an improvement. I guess debug vs opt and e10s vs non-e10s is a valid reason to have a wider range. As long as we don't have a given configuration producing different values on each run (i.e. non-deterministic results)
Attachment #8875586 -
Flags: review?(bugmail) → review+
Comment 22•8 years ago
|
||
bugherder |
Comment 23•8 years ago
|
||
Pushed by mtseng@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ad1958745d3f
Use precise range for fuzzy-if. r=kats
Assignee | ||
Updated•8 years ago
|
Keywords: leave-open
Comment 24•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•