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).