DOM Inspector bad argument to function in inLayoutUtils::GetSubDocumentFor()

RESOLVED FIXED in mozilla0.9.5

Status

P4
normal
RESOLVED FIXED
17 years ago
14 years ago

People

(Reporter: hjtoi-bugzilla, Assigned: hjtoi-bugzilla)

Tracking

Trunk
mozilla0.9.5
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

nsIPresShell::GetSubShellFor() needs an argument of type nsISupports for the
second parameter. You are using nsIPresShell. This asserts, and can cause crash
conditions etc. I'll attach a fix.

Comment 2

17 years ago
[s]r=hewitt - Thanks heikki!

Comment 3

17 years ago
over to Heikki since he has the fix
Assignee: hewitt → heikki

Comment 4

17 years ago
Comment on attachment 50594 [details] [diff] [review]
Proposed fix

>+      if (!docShell) return nsnull;

Separate lines please

>+      if (!contentViewer) return nsnull;

Separate lines please.
Attachment #50594 - Flags: review+
Checked in with the change harishd requested.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Priority: -- → P4
Resolution: --- → FIXED
Target Milestone: --- → mozilla0.9.5
Obsolete DOM Viewer bugs -> Browser-General; DOM Viewer component is going away.
Component: DOM Viewer → Browser-General
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.