Closed
Bug 1392723
Opened 7 years ago
Closed 7 years ago
Turn on text layers by default
Categories
(Core :: Graphics: WebRender, enhancement, P1)
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox56 | --- | unaffected |
firefox57 | --- | unaffected |
firefox58 | --- | fixed |
People
(Reporter: kats, Assigned: Gankra)
References
(Depends on 1 open bug)
Details
(Whiteboard: [wr-mvp] [gfx-noted])
Attachments
(9 files)
(deleted),
text/plain
|
Details | |
(deleted),
text/x-review-board-request
|
kats
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
kats
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
kats
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
kats
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
kats
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
kats
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
kats
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
kats
:
review+
|
Details |
Right now text layers are enabled by the "webrendest" flag, but we want them on by default when webrender is enabled. i.e. change the Override_WebRendest at [1] to Override_WebRender.
I'm assuming :Gankro will pick this up when gets back from PTO, assuming nobody gets to it before he does.
[1] http://searchfox.org/mozilla-central/rev/48ea452803907f2575d81021e8678634e8067fc2/gfx/thebes/gfxPrefs.h#543
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → a.beingessner
Assignee | ||
Comment 1•7 years ago
|
||
Depends on: https://github.com/servo/webrender/issues/1624
Updated•7 years ago
|
Status: NEW → ASSIGNED
Priority: P3 → P1
Whiteboard: [gfx-noted] → [wr-mvp] [gfx-noted]
Target Milestone: --- → mozilla57
Updated•7 years ago
|
status-firefox56:
--- → unaffected
status-firefox57:
--- → unaffected
Assignee | ||
Comment 2•7 years ago
|
||
Alright, the journey begins!
~230 reftest failures to triage
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c2acfd1e790cee313a12bc9339f9960a6f768f74&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-classifiedState=unclassified&selectedJob=133490358
(This try build already has ~400 text-writing-mode tests marked as failing)
Assignee | ||
Comment 3•7 years ago
|
||
glenn and lee's work has dropped us down to only about ~20 failures.
Try push with annotations to get a clean build: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2d21e1d151e35dc06b6ec225e007d1df39ee8d5b
Good:
* New passes - yay!
* Fuzz some blurred shadows (reasonable)
* Fuzz some text-selection colors compared to svg (weird, but there was some existing similar fuzzing there, so I guess it's A Thing)
Bad:
* A bunch of failures from bad vertical snapping; probably similar to previous horizontal issue.
* Some quirky decoration behaviours: needs investigation.
* Missing diacritics: needs investigation.
* Some messed up writing mode behaviour: needs investigation.
So all in all, we have ~4 text bugs left on linux!
(Ignoring the ~8 places we fallback to gecko, which represent hundreds of tests we'd otherwise fail)
I think it'd be reasonable to flip the switch with these 4 sets of fails-if annotations as TODO.
Assignee | ||
Comment 4•7 years ago
|
||
Syntax error, better try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=88535944fe40fb1b3fab8f7faa5bd78db2232827&selectedJob=136124376
(still missed one or two, will finish up in the morning)
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 13•7 years ago
|
||
Reporter | ||
Comment 14•7 years ago
|
||
mozreview-review |
Comment on attachment 8917364 [details]
Bug 1392723 - Move text-layers from webrendest to webrender.
https://reviewboard.mozilla.org/r/188376/#review193618
::: commit-message-e897e:1
(Diff revision 1)
> +Bug 1392723 - Move text from webrendest to webrender. r?kats
s/text/text layers/
Attachment #8917364 -
Flags: review?(bugmail) → review+
Reporter | ||
Comment 15•7 years ago
|
||
mozreview-review |
Comment on attachment 8917365 [details]
Bug 1392723 - Mark tests as passing with webrender text-layers.
https://reviewboard.mozilla.org/r/188378/#review193620
::: commit-message-2e2af:1
(Diff revision 1)
> +Bug 1392723 - Mark tests as passing with webrender text. r?kats
s/text/text layers/
Attachment #8917365 -
Flags: review?(bugmail) → review+
Reporter | ||
Comment 16•7 years ago
|
||
mozreview-review |
Comment on attachment 8917366 [details]
Bug 1392723 - Give some fuzzy slack to blurred text shadow comparisons.
https://reviewboard.mozilla.org/r/188380/#review193622
Attachment #8917366 -
Flags: review?(bugmail) → review+
Reporter | ||
Comment 17•7 years ago
|
||
mozreview-review |
Comment on attachment 8917367 [details]
Bug 1392723 - Add some more fuzzyness for selection colors.
https://reviewboard.mozilla.org/r/188382/#review193624
Attachment #8917367 -
Flags: review?(bugmail) → review+
Reporter | ||
Comment 18•7 years ago
|
||
mozreview-review |
Comment on attachment 8917368 [details]
Bug 1392723 - Mark quirky decoration tests as failing.
https://reviewboard.mozilla.org/r/188384/#review193626
Attachment #8917368 -
Flags: review?(bugmail) → review+
Reporter | ||
Comment 19•7 years ago
|
||
mozreview-review |
Comment on attachment 8917369 [details]
Bug 1392723 - Add failures for bad vertical text snapping.
https://reviewboard.mozilla.org/r/188386/#review193628
Attachment #8917369 -
Flags: review?(bugmail) → review+
Reporter | ||
Comment 20•7 years ago
|
||
mozreview-review |
Comment on attachment 8917370 [details]
Bug 1392723 - Add failures for writing mode.
https://reviewboard.mozilla.org/r/188388/#review193630
::: commit-message-e7ddc:1
(Diff revision 1)
> +Bug 1392723 - Add failures for writing mode. r?kats
> +
Update commit message to mention that the != tests pass but the == tests fail so the only annotation that makes sense (since the pattern-matching only applies to the test file) is random-if.
Attachment #8917370 -
Flags: review?(bugmail) → review+
Reporter | ||
Comment 21•7 years ago
|
||
mozreview-review |
Comment on attachment 8917371 [details]
Bug 1392723 - Add failures for diacritic stacking.
https://reviewboard.mozilla.org/r/188390/#review193632
Attachment #8917371 -
Flags: review?(bugmail) → review-
Reporter | ||
Comment 22•7 years ago
|
||
mozreview-review |
Comment on attachment 8917371 [details]
Bug 1392723 - Add failures for diacritic stacking.
https://reviewboard.mozilla.org/r/188390/#review193634
Oops, meant to r+
Attachment #8917371 -
Flags: review- → review+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 31•7 years ago
|
||
Comments addressed. Had to increase one fuzzyness value, due to typo.
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 32•7 years ago
|
||
All the outstanding issues have been migrated to Bug 1407627 as future work.
Reporter | ||
Comment 33•7 years ago
|
||
I retriggered the reftests, let's wait until those are all done.
Keywords: checkin-needed
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 47•7 years ago
|
||
Finally got a clean build: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4a662a2ff7de492e7e9d7f8cb13c874ab57affba
Keywords: checkin-needed
Comment 48•7 years ago
|
||
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a337a84a9160
Move text-layers from webrendest to webrender. r=kats
https://hg.mozilla.org/integration/autoland/rev/42d2a723d446
Mark tests as passing with webrender text-layers. r=kats
https://hg.mozilla.org/integration/autoland/rev/e95a13b4a928
Give some fuzzy slack to blurred text shadow comparisons. r=kats
https://hg.mozilla.org/integration/autoland/rev/93170ca68e3f
Add some more fuzzyness for selection colors. r=kats
https://hg.mozilla.org/integration/autoland/rev/55ab370c38b6
Mark quirky decoration tests as failing. r=kats
https://hg.mozilla.org/integration/autoland/rev/e7a34e1180e2
Add failures for bad vertical text snapping. r=kats
https://hg.mozilla.org/integration/autoland/rev/aa1903983ca7
Add failures for writing mode. r=kats
https://hg.mozilla.org/integration/autoland/rev/4997d8582287
Add failures for diacritic stacking. r=kats
Keywords: checkin-needed
Comment 49•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a337a84a9160
https://hg.mozilla.org/mozilla-central/rev/42d2a723d446
https://hg.mozilla.org/mozilla-central/rev/e95a13b4a928
https://hg.mozilla.org/mozilla-central/rev/93170ca68e3f
https://hg.mozilla.org/mozilla-central/rev/55ab370c38b6
https://hg.mozilla.org/mozilla-central/rev/e7a34e1180e2
https://hg.mozilla.org/mozilla-central/rev/aa1903983ca7
https://hg.mozilla.org/mozilla-central/rev/4997d8582287
You need to log in
before you can comment on or make changes to this bug.
Description
•