Closed
Bug 1377303
Opened 7 years ago
Closed 7 years ago
Advanced Layers breaks rendering on Google Maps
Categories
(Core :: Graphics: Layers, defect)
Core
Graphics: Layers
Tracking
()
RESOLVED
FIXED
People
(Reporter: dvander, Assigned: dvander)
References
Details
Attachments
(2 files)
1016 bytes,
patch
|
mattwoodrow
:
review+
|
Details | Diff | Splinter Review |
6.79 KB,
patch
|
mattwoodrow
:
review+
|
Details | Diff | Splinter Review |
Screenshot is in bug 1375743. Pretty easy to reproduce by panning around.
Assignee | ||
Comment 1•7 years ago
|
||
This is a bug in texture coordinate flipping required for OpenGL textures, so it only manifests with WebGL.
Assignee | ||
Comment 2•7 years ago
|
||
I had copied texture coordinate flipping code from ImageHost, but ImageHost assumes the texture rect will always be (0, 0, 1, 1). Since AL uses a more precise region, we have to fully invert the y axis.
Attachment #8882619 -
Flags: review?(matt.woodrow)
Updated•7 years ago
|
Attachment #8882619 -
Flags: review?(matt.woodrow) → review+
Pushed by danderson@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/d4146ec1d551 Fix OpenGL texture coordinate flipping in Advanced Layers. (bug 1377303, r=mattwoodrow)
Assignee | ||
Comment 4•7 years ago
|
||
This takes an existing webgl reftest and just plops a div over it to reproduce the texture coordinate bug.
Attachment #8882647 -
Flags: review?(matt.woodrow)
Assignee | ||
Updated•7 years ago
|
Whiteboard: keep-open
Updated•7 years ago
|
Attachment #8882647 -
Flags: review?(matt.woodrow) → review+
Pushed by danderson@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b201ff7ac015 Add a reftest for bug 1377303. r=mattwoodrow
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d4146ec1d551 https://hg.mozilla.org/mozilla-central/rev/b201ff7ac015
Assignee | ||
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Flags: in-testsuite+
Whiteboard: keep-open
You need to log in
before you can comment on or make changes to this bug.
Description
•