Closed Bug 1392723 Opened 7 years ago Closed 7 years ago

Turn on text layers by default

Categories

(Core :: Graphics: WebRender, enhancement, P1)

Other Branch
enhancement

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)

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: nobody → a.beingessner
Depends on: 1394568
Depends on: 1394570
Depends on: 1395098
Depends on: 1395729
Depends on: 1395748
Status: NEW → ASSIGNED
Priority: P3 → P1
Whiteboard: [gfx-noted] → [wr-mvp] [gfx-noted]
Target Milestone: --- → mozilla57
Depends on: 1399274
Depends on: 1399278
Depends on: 1399564
Depends on: 1400382
Depends on: 1400384
Depends on: 1400400
Depends on: 1400917
Depends on: 1402413
No longer depends on: 1402413
Depends on: 1403261
Depends on: 1403262
Depends on: 1404370
Depends on: 1405765
Depends on: 1405927
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.
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 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+
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+
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+
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+
Attachment #8917368 - Flags: review?(bugmail) → review+
Attachment #8917369 - Flags: review?(bugmail) → 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+
Attachment #8917371 - Flags: review?(bugmail) → 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+
Comments addressed. Had to increase one fuzzyness value, due to typo.
Keywords: checkin-needed
All the outstanding issues have been migrated to Bug 1407627 as future work.
I retriggered the reftests, let's wait until those are all done.
Keywords: checkin-needed
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
Depends on: 1407983
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: