Inspect Contained Document followed by view Stylesheets screw ups

RESOLVED FIXED in Future

Status

Other Applications
DOM Inspector
RESOLVED FIXED
16 years ago
7 years ago

People

(Reporter: Joe Hewitt (gone), Assigned: crussell)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

16 years ago
When I inspect mozilla, then go to the <browser> and inspect its contained
document, then switch the document pane viewer to the Stylesheets view, it
reverts to the old navigator document object instead of the new sub-document for
the browser.
(Reporter)

Updated

16 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → Future
Mass re-assigning bugs to dom.inspector@extensions.bugs
Assignee: hewitt → dom.inspector
Status: ASSIGNED → NEW
Product: Core → Other Applications
Assignee: dom-inspector → nobody
QA Contact: timeless → dom-inspector
>   cmdInspectBrowser: function DVr_CmdInspectBrowser()
>   {
>     var node = this.selectedNode;
>     var n = node && node.localName.toLowerCase();
>     if (n == "iframe" || n == "frame" ||
>         (node.namespaceURI == kXULNSURI && (n == "browser" ||
>                                             n == "tabbrowser" ||
>                                             n == "editor"))) {
>       this.subject = node.contentDocument;
>     }
>   },

This should update the panel's subject, instead of directly setting this instance's subject.

This is a one line fix, but no patch because I don't want my patch for bug 310370 to rot.
Status: NEW → ASSIGNED
This is a mass change. Every comment has "assigned-to-new" in it.

I didn't look through the bugs, so I'm sorry if I change a bug which shouldn't be changed. But I guess these bugs are just bugs that were once assigned and people forgot to change the Status back when unassigning.
Status: ASSIGNED → NEW
Created attachment 493518 [details] [diff] [review]
make sure the subject persists by setting the pane's subject instead of the DOM Nodes viewer's own subject
Assignee: nobody → Sevenspade
Status: NEW → ASSIGNED
Attachment #493518 - Flags: review?(neil)
Blocks: 592530

Updated

7 years ago
Attachment #493518 - Flags: review?(neil) → review+
Pushed: http://hg.mozilla.org/dom-inspector/rev/db9d7e55ee3e
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.