Closed
Bug 913407
Opened 11 years ago
Closed 11 years ago
rendering issue in Addon manager
Categories
(Core :: Graphics: Layers, defect)
Tracking
()
VERIFIED
FIXED
mozilla27
People
(Reporter: fb+mozdev, Assigned: jrmuizel)
References
Details
(Keywords: regression)
Attachments
(5 files)
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:26.0) Gecko/20100101 Firefox/26.0 (Beta/Release)
Build ID: 20130905030206
Steps to reproduce:
0. Make sure you have one or more plugins installed.
1. Open Addon manager.
2. Switch to "plugins".
3. Switch to another tab.
4.a) Switch back to the Addon manager tab.
4.b) Mouse-hover over a "more" link of a plugin.
4.c) Switch to "Appearance"
Actual results:
Initially, the "content" background area of the addon manager has a light overlay.
(a) After switching back and forth between tabs, the background is darker (overlay lost).
(b) Hovering over "More" shows the background for the link area.
(c) Switching to "Appearance" shows a part of the background.
This happens troughout the Addon manager. Aurora 25 is also affected but not Beta 24.
Expected results:
Background is consistent.
Reporter | ||
Comment 1•11 years ago
|
||
Oh, turning hardware acceleration off via the preferences (NOT about:config) does not change anything.
HW is a MBA 13" 2012 with Intel HD 4000.
Most likely a GFX issue (Compositing?).
Reporter | ||
Comment 2•11 years ago
|
||
Reporter | ||
Comment 3•11 years ago
|
||
Reporter | ||
Comment 4•11 years ago
|
||
Reporter | ||
Comment 5•11 years ago
|
||
Reporter | ||
Comment 6•11 years ago
|
||
Unfortunately I cannot set the correct params for bug flags. As said in Comment 0, Aurora & Nightly are affected, but not Beta (24).
Reporter | ||
Comment 7•11 years ago
|
||
(sorry for bugspam, I'm done now)
Keywords: regression,
regressionwindow-wanted
Comment 8•11 years ago
|
||
Confirming, also on a 2012 Mac (MacBook Pro). All you have to do is open about:addons, switch to another tab, switch back, and then notice that the background of the selected about:addons section has changed. Not sure whether this belongs in some Core component like Graphics or Layout, or Toolkit, so moving to the latter for further investigation.
Status: UNCONFIRMED → NEW
Component: Untriaged → Add-ons Manager
Ever confirmed: true
Product: Firefox → Toolkit
Reporter | ||
Comment 9•11 years ago
|
||
OK I tried this again in Aurora 25, 2013-09-16: Disabling HW accel or OMTC fixes this issue.
Can someone check if this happens on other platforms than Mac, too?
Blocks: 756601
Reporter | ||
Comment 10•11 years ago
|
||
I just tried UX Nightly (2013-09-17), too, disabling OMTC fixes this issue there, too.
You can also trigger this by simply reloading (Cmd+R) when in the addons manager.
Updated•11 years ago
|
Component: Add-ons Manager → Graphics: Layers
Product: Toolkit → Core
Reporter | ||
Comment 11•11 years ago
|
||
Firefox 25 Beta 1 through Firefox 27 Nightly 2013-09-24 (all OMTC) are affected now. Nom'ing for tracking.
Comment 12•11 years ago
|
||
Matt, can you please see if you can reproduce this on your Mac and if so please find a regression window?
Comment 13•11 years ago
|
||
It appears that the first bad build of nightly FF25 is 2013-07-23. The rendering issue presents itself with the steps above, or sometimes just upon first viewing of the add-ons tab.
This is with a Retina MacBook, FWIW.
Let me know if you need more information.
Keywords: qawanted,
regressionwindow-wanted
Comment 14•11 years ago
|
||
That gives us the following pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2268ff80683a&tochange=5ceea82a79c7
It might be interesting to note that this is the exact same regression range as bug 920044.
Version: 26 Branch → 25 Branch
Comment 15•11 years ago
|
||
Matt, if you have the time would you be able to regress this manually by bisecting that range further on your Mac?
Comment 17•11 years ago
|
||
Seems a regression from bug 896489. If I change gfx.content.azure.enabled to false (and restart), the problem goes away.
Blocks: 896489
Flags: needinfo?(jmuizelaar)
Comment 18•11 years ago
|
||
Jeff will take this.
Assignee: nobody → jmuizelaar
Flags: needinfo?(jmuizelaar)
Comment 19•11 years ago
|
||
We would likely take a low risk uplift, but this is not a critical regression that requires tracking.
Assignee | ||
Comment 20•11 years ago
|
||
The original code wasn't handling the case when the gradient line was already completely outside of the extents. This new code handles that and should be more systematic than the previous code.
This is still a little rough but see if you can figure it out and if it makes sense.
Attachment #813476 -
Flags: review?(matt.woodrow)
Comment 21•11 years ago
|
||
Comment on attachment 813476 [details] [diff] [review]
patch
Review of attachment 813476 [details] [diff] [review]:
-----------------------------------------------------------------
r- for now since I don't understand how it works :)
The patch in bug 902591 seems a lot simpler to understand, and safer since it has testing already.
If you think this approach is superior, then lets work on getting it commented to a level that I can follow when you're back from PTO.
::: gfx/2d/DrawTargetCG.cpp
@@ +401,5 @@
> + float sin = (end - start).y/length;
> + Point diff = end - start;
> +
> + // change the coordinates of the extents box to an axis with start at the origin
> + // and end in the positive x direction
I really don't understand what this means.
A high level comment explaining what problem this is solving and how it does that would probably help.
Attachment #813476 -
Flags: review?(matt.woodrow) → review-
Assignee | ||
Comment 22•11 years ago
|
||
I believe this may have been fixed by bug 902591
Comment 23•11 years ago
|
||
(In reply to Jeff Muizelaar [:jrmuizel] from comment #22)
> I believe this may have been fixed by bug 902591
Please confirm this is fixed in the following builds:
* http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-trunk/firefox-27.0a1.en-US.mac.dmg
* http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-aurora/firefox-26.0a2.en-US.mac.dmg
* http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/25.0-candidates/build3/mac/en-US/Firefox%2025.0.dmg
Thank you
Flags: needinfo?(fb+mozdev)
Reporter | ||
Comment 24•11 years ago
|
||
Can no longer reproduce on 25b12, 26.0a2 (2013-10-28), 27.0a1 (2013-10-27). Thanks a lot!
(Please resolve as FIXED. I should apply for elevated privileges some time.)
Flags: needinfo?(fb+mozdev)
Comment 25•11 years ago
|
||
Thanks for the help, Florian.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•11 years ago
|
Reporter | ||
Updated•11 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•