306826 - display of content within DOM Inspector doesn't update after opting browser

Status: RESOLVED WONTFIX

(Other Applications :: DOM Inspector, defect)

Description

[Hari Kumar G]

Browser display within DOM Inspector is not updated when the option is chosen
from the View > Browser of the DOM insp.  

Steps to reproduce: 

Open firefox > some web page.  
Open DOM Inspector.  
Click the View > Browser to see the content. The split window is blank.  Select
any of the nodes. Nothing happens.  
Now click 'inspect' along side the url in the address box again.  The page is
reloaded in the main window and the split pane within DOM Inspector.  Now
content is visible. 

I suppose the content should be immediately shown as the user picks the option.

Comment 1

[Martijn Wargers (dead)]

See:
http://lxr.mozilla.org/seamonkey/source/extensions/inspector/resources/content/inspector.js#196

The "Inspect" button follows this code path:
http://lxr.mozilla.org/seamonkey/source/extensions/inspector/resources/content/inspector.js#329

So I gues it would make sense if the View > Browser menu-item also does a
gotoTypedURL when that hasn't been done yet.

Comment 2

[Martijn Wargers (dead)]

Attachment: patch

Something like this, I guess.
If the domi url bar value isn't the same as the domi browser location, then
update like the 'inspect' button does.
The situation is a bit difficult, because the domi url bar shows the url from
'outside' windows and from the domi browser window.
This patch ensures that when the the domi browser window is open, the content
of that browser window is inspected and the url bar has the same value as the
the browser window.
I think this is a more natural behavior.

Comment 3

[Martijn Wargers (dead)]

Comment on attachment 195071 [details] [diff] [review]
patch

Never mind, this check isn't good enough.
You can have identical url's but different content (browser windows all have
chrome://browser/content/browser.xul )

Comment 4

[Martijn Wargers (dead)]

Hmm, then the only option I can see is the View > Browser menu-item to behave
like 'Inspect', when it gets opened.

Comment 5

[Martijn Wargers (dead)]

Attachment: patch2

Like this.

Comment 6

[[e-mail]]

Comment on attachment 195078 [details] [diff] [review]
patch2

I don't like this, because it reloads every time you toggle the browser. Is it
possible to check that the browser's document is currently inspected?

Comment 7

[timeless]

This isn't what we want. Wearing module owner hat: WONTFIX. Sorry

What we would accept (at least, I would today) is a patch (in a new bug) to make the browser area default to a hint saying "click inspect to see content here".