Closed
Bug 408095
Opened 17 years ago
Closed 17 years ago
Screen artifacts when opening a hidechrome window
Categories
(Core :: Widget: Gtk, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9beta3
People
(Reporter: skrulx, Assigned: kinetik)
References
Details
Attachments
(3 files)
|
(deleted),
image/jpg
|
Details | |
|
(deleted),
application/vnd.mozilla.xul+xml
|
Details | |
|
(deleted),
patch
|
roc
:
review+
roc
:
superreview+
roc
:
approval1.9+
|
Details | Diff | Splinter Review |
gray box in the wild
When opening a hidechrome window on gtk, there is always a brief appearance of a 200x200 gray box before the actual window opens up. You can catch this gray box in action if you breakpoint nsWindow::HideWindowChrome (http://mxr.mozilla.org/seamonkey/source/widget/src/gtk2/nsWindow.cpp#4041), open a hidechrome windoiw, and step through the method. The gray box will appear after you step past the call to XSync().
Attached is a screen shot of the gray box right after the XSync() line was run. Also attached is the XUL file that opens a hidecrhome window that I used to reproduce the bug.
|
Reporter | |
Comment 1•17 years ago
|
||
|
|
Reporter | |
Comment 2•17 years ago
|
||
|
|
Reporter | |
Updated•17 years ago
|
Attachment #292819 -
Attachment mime type: image/png → image/jpg
|
Assignee | |
Comment 3•17 years ago
|
||
Consider current window visibility when doing hide/show dance in nsWindow::HideWindowChrome.
Assignee: nobody → kinetik
Status: NEW → ASSIGNED
Attachment #296871 -
Flags: superreview?(roc)
Attachment #296871 -
Flags: review?(roc)
Attachment #296871 -
Flags: superreview?(roc)
Attachment #296871 -
Flags: superreview+
Attachment #296871 -
Flags: review?(roc)
Attachment #296871 -
Flags: review+
Comment on attachment 296871 [details] [diff] [review]
patch
very straightforward patch, low risk
Attachment #296871 -
Flags: approval1.9+
|
|
Assignee | |
Updated•17 years ago
|
Whiteboard: checkin-needed
|
|
Assignee | |
Updated•17 years ago
|
Keywords: checkin-needed
Whiteboard: checkin-needed
|
||
Comment 5•17 years ago
|
||
Checking in widget/src/gtk2/nsWindow.cpp;
/cvsroot/mozilla/widget/src/gtk2/nsWindow.cpp,v <-- nsWindow.cpp
new revision: 1.249; previous revision: 1.248
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M11
You need to log in
before you can comment on or make changes to this bug.
Description
•