Closed
Bug 801301
Opened 12 years ago
Closed 12 years ago
Aurora 18 menu bug
Categories
(Core :: Widget: Win32, defect)
Tracking
()
RESOLVED
FIXED
mozilla19
People
(Reporter: vargadoktor, Assigned: jh.dev0)
References
Details
(Keywords: regression)
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
roc
:
review+
bajaj
:
approval-mozilla-aurora+
MattN
:
checkin+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:18.0) Gecko/18.0 Firefox/18.0
Build ID: 20121012114555
Steps to reproduce:
My Aurora 17 has been updated to version 18 (2012-10-08 win32 en-US, 2012-10-12 win32 en-US, then 2012-10-12 win323 hu).
Actual results:
The Firefox button menu covers opening Web developers submenu. The same happens in bookmarks menu, it covers bookmarks submenus.
Expected results:
Web developers submenu should be drawn over the Firefox button menu.
Reporter | ||
Updated•12 years ago
|
Component: Untriaged → Menus
Comment 1•12 years ago
|
||
Can you reproduce this with your add-ons disabled?
Reporter | ||
Comment 2•12 years ago
|
||
I disabled all my add-ons manually. It didn't help. I created new profile but nothing changed. If I restart the browser in safe mode, it's OK, the bug disappears.
Updated•12 years ago
|
Keywords: qawanted,
regressionwindow-wanted
Comment 3•12 years ago
|
||
Regression range hunting:
m-i
First Bad:http://hg.mozilla.org/integration/mozilla-inbound/rev/7c119b50e7aa
Last OK :http://hg.mozilla.org/integration/mozilla-inbound/rev/65b95ed309b9
So, caused by Bug 793065 or Bug 610713. (Likely Bug 610713)
Comment 4•12 years ago
|
||
And as a note, problem happens only when hardware acceleration in enabled.
Updated•12 years ago
|
Blocks: 610713
tracking-firefox18:
--- → ?
Component: Menus → Widget: Win32
Keywords: regressionwindow-wanted → regression
Product: Firefox → Core
Updated•12 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Are the people affected all running Windows XP? If so, did anyone with this bug on XP experience bug 610713 at all?
Comment 6•12 years ago
|
||
Interestingly, as far as I test, I can't reproduce bug 610713 at all with m-i changeset http://hg.mozilla.org/integration/mozilla-inbound/rev/65b95ed309b9
(Last changeset before the patches of bug 610713 was landed)
I don't have access to an XP computer that supports hardware acceleration so I can't test this properly. Assuming bug 610713 didn't affect XP users adding an OS check could fix this bug without regressing that one.
Comment on attachment 671266 [details] [diff] [review]
Possible patch
Review of attachment 671266 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good to me. I guess we're working around a Windows XP bug here.
Attachment #671266 -
Flags: review+
Comment on attachment 671266 [details] [diff] [review]
Possible patch
Review of attachment 671266 [details] [diff] [review]:
-----------------------------------------------------------------
I think we should take this on trunk ASAP because we need to see if it fixes the problem. The patch itself is quite safe, it's just a partial backout of bug 610713 for Windows XP.
Attachment #671266 -
Flags: checkin?
Updated•12 years ago
|
status-firefox18:
--- → affected
Comment 12•12 years ago
|
||
Pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=752e89fd3a50
Assignee: nobody → jh.dev0
Status: NEW → ASSIGNED
Comment 13•12 years ago
|
||
No problem with try server build on my PC.
Comment 14•12 years ago
|
||
Comment on attachment 671266 [details] [diff] [review]
Possible patch
https://hg.mozilla.org/integration/mozilla-inbound/rev/69e943a96a42
Attachment #671266 -
Flags: checkin? → checkin+
Comment on attachment 671266 [details] [diff] [review]
Possible patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #): 610713
User impact if declined: menus in the wrong z-order make the browser difficult to use
Testing completed (on m-c, etc.): landed on m-c
Risk to taking this patch (and alternatives if risky): low risk. It's effectively a partial backout of bug 610713 for Windows XP only
String or UUID changes made by this patch: none
Attachment #671266 -
Flags: approval-mozilla-aurora?
Thanks for the quick followup work, James.
Comment 17•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Comment 18•12 years ago
|
||
WFM in the Oct 18 build of Seamonkey, WinXP. thank you!
Comment 19•12 years ago
|
||
Comment on attachment 671266 [details] [diff] [review]
Possible patch
Approving for aurora, considering it is low risk and user verification in comment 18 .
Attachment #671266 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 20•12 years ago
|
||
Comment 21•12 years ago
|
||
Verified as fixed on Firefox 18 beta 3 - web developer and bookmarks menus are not covered anymore.
Mozilla/5.0 (Windows NT 5.1; rv:18.0) Gecko/20100101 Firefox/18.0
Build ID: 20121205060959
QA Contact: simona.marcu
You need to log in
before you can comment on or make changes to this bug.
Description
•