Open Bug 509023 Opened 15 years ago Updated 2 years ago

eWindowType_invisible is no longer unique

Categories

(Core :: Widget: Win32, defect)

x86_64
Windows 7
defect

Tracking

()

People

(Reporter: emk, Unassigned)

Details

Spin off from bug 507926. WM_POWERBROADCAST handler assumes that only hidden window have eWindowType_invisible. That assumption is broken after bug 433616.
Can you summarize why they're not? I thought we were to only have one hidden window. This affects at least one other bug I'm working on (power related oddly enough).
There is still only one hidden window. We need to change how to determine whether the window is hidden or not because resource documents will also have eWindowType_invisible. http://mxr.mozilla.org/mozilla-central/source/widget/src/windows/nsWindow.cpp#3650 http://mxr.mozilla.org/mozilla-central/source/layout/base/nsDocumentViewer.cpp#2269 I'm afraid I don't know much about resource documents. Roc will know.
The first eWindowType_invisible window created will be the hidden window. The power notification code might be able to move somewhere else when bug 454660 is fixed (the newer Vista API will requiring hooking the hidden window - see bug 507736 which figures out the hidden window).
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.