Closed
Bug 1100699
Opened 10 years ago
Closed 9 years ago
Deprecate concept of 'inverted' in favor of explicit origin location
Categories
(Core :: Graphics, defect)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: jgilbert, Assigned: jgilbert)
References
Details
Attachments
(1 file, 2 obsolete files)
40.10 KB,
patch
|
jgilbert
:
review+
|
Details | Diff | Splinter Review |
We have a bunch of `isInverted` spaghetti, but it sometimes tricky to figure out what it all means. Y-flip is traditionally tricky because two wrongs can make a right. However, if you skip either of the wrongs, it's now wrong again. Explicitly calling out what the origin for an image is should reduce the likelihood of having these issues.
Attachment #8524205 -
Flags: review?(snorp)
Comment 1•10 years ago
|
||
Comment on attachment 8524205 [details] [diff] [review] 0001-Use-gl-OriginPos-instead-of-isInverted.patch Review of attachment 8524205 [details] [diff] [review]: ----------------------------------------------------------------- This doesn't build for me, but I agree with it in principal. I'll try to fix it up and make sure everything still works with video and Flash.
Attachment #8524205 -
Flags: review?(snorp)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jgilbert
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8524205 -
Attachment is obsolete: true
Attachment #8529440 -
Flags: review?(snorp)
Attachment #8529440 -
Flags: review?(matt.woodrow)
Comment 3•10 years ago
|
||
Comment on attachment 8529440 [details] [diff] [review] 0001-Use-gl-OriginPos-instead-of-isInverted.patch Review of attachment 8529440 [details] [diff] [review]: ----------------------------------------------------------------- Bit of an rs, since I didn't check every change individually, but the concept looks great!
Attachment #8529440 -
Flags: review?(matt.woodrow) → review+
Assignee | ||
Comment 4•10 years ago
|
||
I can't test flash on the device I have, but videos seem fine. Everything on [1] is right-side up. [1]: http://www.quirksmode.org/html5/tests/video.html
Comment 5•10 years ago
|
||
Comment on attachment 8529440 [details] [diff] [review] 0001-Use-gl-OriginPos-instead-of-isInverted.patch Review of attachment 8529440 [details] [diff] [review]: ----------------------------------------------------------------- I'm a fan.
Attachment #8529440 -
Flags: review?(snorp) → review+
Assignee | ||
Comment 6•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8116cca43683
Assignee | ||
Comment 7•9 years ago
|
||
r=mattwoodrow,snorp
Attachment #8529440 -
Attachment is obsolete: true
Attachment #8542349 -
Flags: review+
https://hg.mozilla.org/mozilla-central/rev/e1dd807b8b2a
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
This seems to have caused some MP4 videos to appear upside-down on Android.
Depends on: 1116469
Depends on: 1116469
No longer depends on: 1116469
Comment 10•9 years ago
|
||
I wonder if this caused the webgl map flip problem we got in 37
You need to log in
before you can comment on or make changes to this bug.
Description
•