Closed
Bug 1399178
Opened 7 years ago
Closed 7 years ago
Weird effect on arstechnica when scrolling to the top (OMTP)
Categories
(Core :: Graphics, defect, P3)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | unaffected |
People
(Reporter: milan, Assigned: bas.schouten)
References
Details
(Whiteboard: [gfx-noted])
Attachments
(1 file)
https://arstechnica.com/science/2017/08/after-43-years-gentle-touch-of-a-neutrino-is-finally-observed/ scroll (mouse wheel or scroll bar) away from the top, then scroll all the way to the top. With the scrollbar, the text in the first paragraph is black "bounding box" rectangles until I let go of the mouse button. I'll look for a regression range.
Comment 1•7 years ago
|
||
What OS and version did you see this on?
Reporter | ||
Comment 2•7 years ago
|
||
This is Windows 10, and it feels like it's something that the site is doing (e.g., looks like what facebook does when they're loading.) Not successful with mozregression so far, so I'm not sure how actionable this would be.
Reporter | ||
Comment 3•7 years ago
|
||
This page has also gone to "spinning forever" at times, and I also managed to get the boxes to be white/not show. The boxes are per line of text, not per word. I am, however, running OMTP enabled, let me see if that makes a difference.
Reporter | ||
Comment 4•7 years ago
|
||
OMTP only.
Blocks: omtp
status-firefox57:
--- → unaffected
Priority: -- → P3
Summary: Weird effect on arstechnica when scrolling to the top → Weird effect on arstechnica when scrolling to the top (OMTP)
Whiteboard: [gfx-noted]
Reporter | ||
Updated•7 years ago
|
Component: Panning and Zooming → Graphics
Comment hidden (mozreview-request) |
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8907763 [details] Bug 1399178: Record SetPermitSubpixelAA. https://reviewboard.mozilla.org/r/179428/#review184604 ::: gfx/2d/DrawTargetCapture.cpp:85 (Diff revision 1) > void > +DrawTargetCaptureImpl::SetPermitSubpixelAA(bool aPermitSubpixelAA) > +{ > + AppendCommand(SetPermitSubpixelAACommand)(aPermitSubpixelAA); > + > + // Have to update the transform for this DT s/transform/subpixel AA state/
Attachment #8907763 -
Flags: review?(dvander) → review+
Pushed by bschouten@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/8cd211d1292e Record SetPermitSubpixelAA. r=dvander
Comment 8•7 years ago
|
||
Backed out for unexpected passes on quantumrenderer, e.g. in reftest layout/reftests/counters/counter-suffix-ref.html: Push with failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=8cd211d1292e602c0b4c439d39a70aa082ed0c76&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=130770560&repo=mozilla-inbound
Flags: needinfo?(bas)
Backout by archaeopteryx@coole-files.de: https://hg.mozilla.org/integration/mozilla-inbound/rev/79eb1b3a2928 Backed out changeset 8cd211d1292e for unexpected passes on quantumrenderer, e.g. in reftest layout/reftests/counters/counter-suffix-ref.html. r=backout
Assignee | ||
Comment 10•7 years ago
|
||
So I fixed one of your bugs, could you reland this marking the relevant QR tests passing?
Flags: needinfo?(bas) → needinfo?(jmuizelaar)
Comment 11•7 years ago
|
||
I did this, since I had to update the webrender reftest tracking spreadsheet anyway. Try push to verify greenness: https://treeherder.mozilla.org/#/jobs?repo=try&revision=29dbd61db4e93cf3de6376598dabd80c94f685af Will land if that looks good.
Flags: needinfo?(jmuizelaar)
Comment 12•7 years ago
|
||
Pushed by kgupta@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/14913071baca Record SetPermitSubpixelAA. r=dvander
Comment 13•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/14913071baca
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Updated•6 years ago
|
Assignee: nobody → bas
You need to log in
before you can comment on or make changes to this bug.
Description
•