Closed Bug 571541 Opened 11 years ago Closed 11 years ago

Make sure to shut down the document in ShutdownDocAccessiblesInTree even if it has no docshell

Categories

(Core :: Disability Access APIs, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.3a5

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Details

Attachments

(1 file)

That can happen for resource documents.
Attached patch Proposed fixSplinter Review
Attachment #450687 - Flags: review?(surkov.alexander)
Comment on attachment 450687 [details] [diff] [review]
Proposed fix

r=me, though I would happy to see 'continue' statement aligned properly :)

Technically when we'll have notification from presshell is destroyed then we don't need to run through subdocuments, right?
Attachment #450687 - Flags: review?(surkov.alexander) → review+
> though I would happy to see 'continue' statement aligned properly :)

Fixed; tabs had snuck in.  I also added a modeline to prevent that happening again.

> then we don't need to run through subdocuments, right?

That'll depend on what we're trying to do.  See comments in bug 571459.
Pushed http://hg.mozilla.org/mozilla-central/rev/e16ee5d09003
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
You need to log in before you can comment on or make changes to this bug.