Crashes on WinXP as well.
OS: Mac OS X 10.4 → All
Hardware: Macintosh → All
The needed fix in this case is just (doc->GetRootContent() != this), but I think it makes sense to add also that NS_ENSURE_STATE. HideWindowChrome() is used only with <window> and IMO we can assume that HideWindowChrome works only when <window> is used as a root element.
Comment on attachment 246381 [details] [diff] [review] proposed patch [I wonder whether it's worth adding the tag check here or moving the doc check into the caller...]
Attachment #246381 - Flags: superreview?(neil) → superreview+
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Verified FIXED using build 2006-11-24-10 of SeaMonkey trunk under Windows XP using https://bugzilla.mozilla.org/attachment.cgi?id=240399&action=view as the testcase; no crash.
Status: RESOLVED → VERIFIED
Crashtest checked in.
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: xptoolkit.xul → xptoolkit.widgets
Crash Signature: [@ nsXULElement::HideWindowChrome]
You need to log in before you can comment on or make changes to this bug.