images with fractional opacity are positioned outside container when page is zoomed

RESOLVED FIXED in mozilla10

Status

()

Core
Layout
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: MattN, Assigned: mattwoodrow)

Tracking

(Depends on: 1 bug, {regression})

unspecified
mozilla10
x86
All
regression
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(firefox10-)

Details

(URL)

Attachments

(4 attachments)

Created attachment 570847 [details]
Actual Result: Screenshot of toolbar with images positioned outside container

Since the nightly build on 2011-10-27, the toolbar images on Google Voice are positioned outside of their containing <div> when the page is zoomed in.  See screenshot of the problem.

STR:
1) Load https://www.google.com/voice and login/create an account (sorry, U.S only)
2) Zoom in on the page

Actual result:
some button images are positioned outside the container <div>

Expected result:
button image zooms but stays within the <div>

Last good nightly: 10.0a1 (2011-10-26)
Built from http://hg.mozilla.org/mozilla-central/rev/cc66accc8181

First bad nightly:
Built from http://hg.mozilla.org/mozilla-central/rev/4bb7c983d589

Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=cc66accc8181&tochange=4bb7c983d589
Created attachment 570858 [details]
testcase: <img> with opacity position incorrect after zoom

This seems to be a problem with positioning the img if there is opacity specified on the <img> and the page is zoomed. Not sure if it's a gfx or layout bug.
Summary: Google Voice button images are positioned outside container div when page is zoomed → images with fractional opacity are positioned outside container when page is zoomed

Comment 2

6 years ago
Regression window(m-i)
Works:
http://hg.mozilla.org/integration/mozilla-inbound/rev/98013fe19dcb
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:10.0a1) Gecko/20111025 Firefox/10.0a1 ID:20111025195747
Fails:
http://hg.mozilla.org/integration/mozilla-inbound/rev/d7f3bfc7cd46
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:10.0a1) Gecko/20111025 Firefox/10.0a1 ID:20111025203245
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=98013fe19dcb&tochange=d7f3bfc7cd46
Triggered by:
8b89d7037306	Matt Woodrow — Bug 695275 - Fix conversion of ThebesLayers to ImageLayers. r=roc
Blocks: 695275
OS: Mac OS X → All

Updated

6 years ago
Duplicate of this bug: 698711
devpixels/CSSpixels confusion presumably
Assignee: nobody → matt.woodrow
(Assignee)

Comment 5

6 years ago
Created attachment 571447 [details] [diff] [review]
Make ConfigureLayer use DevPixels instead of AppPixels.
Attachment #571447 - Flags: review?(roc)
Comment on attachment 571447 [details] [diff] [review]
Make ConfigureLayer use DevPixels instead of AppPixels.

Review of attachment 571447 [details] [diff] [review]:
-----------------------------------------------------------------

Reftest?
Attachment #571447 - Flags: review?(roc) → review+
tracking-firefox10: --- → ?
Duplicate of this bug: 700374
(Assignee)

Comment 8

6 years ago
Landed this as https://hg.mozilla.org/integration/mozilla-inbound/rev/95efc21bf5af to try make the next aurora merge.

Will get a reftest added asap, please leave bug open.
Flags: in-testsuite?
https://hg.mozilla.org/mozilla-central/rev/95efc21bf5af

(Leaving open for retest, per comment 8)
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla10

Comment 10

6 years ago
Today's nightly is actually worse for Gvoice/Greader (especially Greader, it's broke even without any zoom...)
Does that build have this fix?

Comment 12

6 years ago
(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #11)
> Does that build have this fix?
I think it will be in next Nightly (09-11-2011), beacause now I can't see any difference.

Comment 13

6 years ago
Google fixed the issue I was having(as long as you don't zoom).  Weird.

Comment 14

6 years ago
I can confirm that it is fixed in my Nightly.

Mozilla/5.0 (Windows NT 6.1; rv:11.0a1) Gecko/20111109 Firefox/11.0a1 ID:20111109031111
Doesn't occur on FF10 for me, even with zooming.
tracking-firefox10: ? → -
(Assignee)

Comment 16

6 years ago
Created attachment 577477 [details] [diff] [review]
Add reftest for zoomed ImageLayer
Attachment #577477 - Flags: review?(roc)
Attachment #577477 - Flags: review?(roc) → review+
(Assignee)

Comment 17

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/6d36479d1250
Reftest landed in mozilla11:
https://hg.mozilla.org/mozilla-central/rev/6d36479d1250
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED

Updated

5 years ago
Depends on: 745549
You need to log in before you can comment on or make changes to this bug.