Closed
Bug 409223
Opened 17 years ago
Closed 17 years ago
Content can make browser chrome disappear
Categories
(Core :: Layout, defect, P2)
Tracking
()
VERIFIED
FIXED
People
(Reporter: ecfbugzilla, Assigned: roc)
References
Details
(Keywords: regression)
Attachments
(2 files)
I hit this behavior while investigating a different issue. A button inside a XUL frame changes the src attribute of this frame to "about:blank" - and the browser chrome simply disappears, you have no choice but to press Alt-F4. According to DOM Inspector, the inner size of the browser window is 1280x744 and outer size is 1280x639 (yes, outer size is actually smaller than inner size). Only outer size changes when clicking the button, the original value is 1288x778. I am filing it under layout because that's my best guess but I have trouble imagining what causes this behavior.
Testcase is attached, you simply have to click the button to reproduce. I see this behavior in Gecko/2007122005 Minefield/3.0b3pre started in safe mode on Windows XP, Firefox 2.0.0.11 however has no trouble whatsoever with the testcase.
Comment 1•17 years ago
|
||
Maybe related to bug 405472?
Reporter | ||
Comment 2•17 years ago
|
||
Yes, probably caused by the same bug (document going away while click event is being handled?) - but the problem in the testcase here is 100% reproducible.
Comment 3•17 years ago
|
||
I can't reproduce on Windows XP. Tried first with the 12/19 build I had and then the same 12/20 build you cite, with and without -safe-mode.
When do you think this regressed for you?
Reporter | ||
Comment 4•17 years ago
|
||
I only mentioned -safe-mode because I have some extensions installed. Now I tried starting with a clean profile - the problem is still reproducible.
Regression range is 2007092005 to 2007092105.
Reporter | ||
Comment 5•17 years ago
|
||
Looking at bonsai, bug 396587 is my prime suspect to have caused this behavior. Other possibilities are bug 396099, bug 389636 and nsGenericElement.cpp revision 3.596 (checked in with wrong bug number).
Reporter | ||
Comment 6•17 years ago
|
||
Baking out bug 396587 (manually, because the patch no longer applies) did not fix this. Baking out bug 396099 fixed it however. Just to be sure I patched bug 396099 back in and the same behavior came back.
Blocks: 396099
Comment 7•17 years ago
|
||
This really feels like a Windows widgetry bug (given that I don't see this on Mac and the other two ways to reproduce this in the bugs this blocks).
If needed we can try backing out bug 396099, but it seems like a much better idea to fix the real issue, since there are likely other ways to get this to happen...
roc, ere, any idea what might be going on here?
Blocks: 405472
Flags: blocking1.9?
Comment 8•17 years ago
|
||
I don't know layout well enough to say how it happens, but SyncFrameViewGeometryDependentProperties in nsContainerFrame.cpp calls nsWindow::SetHasTransparentBackground(true) setting the chrome translucent, and the chrome in question happens to be the browser window.. To me it seems that widget layer just does what's been told to it, but the content is getting a bit too much power here.
Assignee | ||
Comment 9•17 years ago
|
||
Yikes, it looks like we're doing a reflow at a pretty bad time there! Perhaps nsDocumentViewer::SetTextZoom/SetFullZoom should pass NS_VMREFRESH_DEFERRED instead of NO_SYNC? But we should figure out the root cause here anyway. I guess I can take it...
Assignee: nobody → roc
Comment 10•17 years ago
|
||
Sounds like basically bug 322074, then? See analysis there.
Depends on: 322074
Assignee | ||
Updated•17 years ago
|
Flags: blocking1.9? → blocking1.9+
Priority: -- → P2
Assignee | ||
Updated•17 years ago
|
Whiteboard: [depends on 322074]
Assignee | ||
Comment 11•17 years ago
|
||
fixed by bug 322074
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•17 years ago
|
Whiteboard: [depends on 322074]
Reporter | ||
Comment 12•17 years ago
|
||
Confirmed, this is fixed in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b3pre) Gecko/2008011005 Minefield/3.0b3pre.
Updated•17 years ago
|
Flags: in-testsuite?
Comment 13•17 years ago
|
||
verified fixed using Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b3) Gecko/2008020514 Firefox/3.0b3 and also per comment #12
no crash on testcase -> Verified fixed
Status: RESOLVED → VERIFIED
Updated•15 years ago
|
Group: core-security
Flags: wanted1.8.1.x-
You need to log in
before you can comment on or make changes to this bug.
Description
•