Closed
Bug 988520
Opened 11 years ago
Closed 10 years ago
Browser toolbox can't inspect other windows
Categories
(DevTools :: Inspector, defect)
DevTools
Inspector
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 39
People
(Reporter: bbenvie, Assigned: ochameau)
References
(Blocks 1 open bug)
Details
STR:
1.) Open a Scratchpad window
2.) Open Browser Toolbox
3.) Start picking an element
4.) Try to pick something in the Scratchpad window
The picker doesn't like Scratchpad windows.
Even outside the Scratchpad, there is no way to access any other windows from the Browser Toolbox.
Summary: Browser inspector can't inspect Scratchpad windows → Browser toolbox can't inspect other windows
Comment 2•10 years ago
|
||
That's also important for extension developers, who want to inspect and debug their extension's windows.
Sebastian
Comment 3•10 years ago
|
||
So the Browser Toolbox needs some way to select the current window. DOM Inspector already provides that via File > Inspect Chrome Document, though the UX is not the best one, as it's sometimes hard to identify the right window.
So, if possible, the UI for the window/chrome content selection should somehow indicate what will actually be selected, e.g. by bringing the window to the foreground and showing a border around its content.
Sebastian
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Paul, are you sure this is a duplicate? I did not think the iframe selector would give us access to entirely separate top-level browser windows, so this seems like very different work would be needed here.
Flags: needinfo?(paul)
Comment 7•10 years ago
|
||
(In reply to J. Ryan Stinnett [:jryans] from comment #6)
> Paul, are you sure this is a duplicate? I did not think the iframe selector
> would give us access to entirely separate top-level browser windows, so this
> seems like very different work would be needed here.
tabActor lists subdocshells. The rootActor will list all the docshells. Then it should work out of the box, no?
Flags: needinfo?(paul)
(In reply to Paul Rouget [:paul] (slow to respond. Ping me on IRC) from comment #7)
> (In reply to J. Ryan Stinnett [:jryans] from comment #6)
> > Paul, are you sure this is a duplicate? I did not think the iframe selector
> > would give us access to entirely separate top-level browser windows, so this
> > seems like very different work would be needed here.
>
> tabActor lists subdocshells. The rootActor will list all the docshells. Then
> it should work out of the box, no?
Maybe, but the implementation of bug 1059308 is a bit unclear for now, because frame switching is only part of the tab actor, and the root actor is not a tab actor at all.
Let's keep this open and depend on bug 1059308 so it's not lost. When bug 1059308 is done, we can check if this came for free and close then.
Status: REOPENED → NEW
Comment 9•10 years ago
|
||
Does this also cover being able to inspect content windows? I'm trying to figure out why this one button in about:support doesn't work for me ...
Comment 10•10 years ago
|
||
(In reply to Samuel Bronson from comment #9)
> Does this also cover being able to inspect content windows? I'm trying to
> figure out why this one button in about:support doesn't work for me ...
The Browser Toolbox shouldn't have a problem inspecting about:support, but it'd be easier to just use the normal toolbox on that page. Does this not work?
Comment 11•10 years ago
|
||
(In reply to Brian Grinstead [:bgrins] from comment #10)
> (In reply to Samuel Bronson from comment #9)
> > Does this also cover being able to inspect content windows? I'm trying to
> > figure out why this one button in about:support doesn't work for me ...
>
> The Browser Toolbox shouldn't have a problem inspecting about:support, but
> it'd be easier to just use the normal toolbox on that page. Does this not
> work?
No scripts show in the web debugger for about:support. I imagine they're all chrome scripts and that's why they're not visible.
Comment 12•10 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #11)
> (In reply to Brian Grinstead [:bgrins] from comment #10)
> > (In reply to Samuel Bronson from comment #9)
> > > Does this also cover being able to inspect content windows? I'm trying to
> > > figure out why this one button in about:support doesn't work for me ...
> >
> > The Browser Toolbox shouldn't have a problem inspecting about:support, but
> > it'd be easier to just use the normal toolbox on that page. Does this not
> > work?
>
> No scripts show in the web debugger for about:support. I imagine they're all
> chrome scripts and that's why they're not visible.
Nick, any ideas what's going on there? Is there already a bug on file about this?
Flags: needinfo?(nfitzgerald)
Comment 13•10 years ago
|
||
(In reply to Brian Grinstead [:bgrins] from comment #12)
> (In reply to :Gijs Kruitbosch from comment #11)
> > (In reply to Brian Grinstead [:bgrins] from comment #10)
> > > (In reply to Samuel Bronson from comment #9)
> > > > Does this also cover being able to inspect content windows? I'm trying to
> > > > figure out why this one button in about:support doesn't work for me ...
> > >
> > > The Browser Toolbox shouldn't have a problem inspecting about:support, but
> > > it'd be easier to just use the normal toolbox on that page. Does this not
> > > work?
> >
> > No scripts show in the web debugger for about:support. I imagine they're all
> > chrome scripts and that's why they're not visible.
>
> Nick, any ideas what's going on there? Is there already a bug on file about
> this?
I believe this is bug 1085446.
Updated•10 years ago
|
Flags: needinfo?(nfitzgerald)
Comment 14•10 years ago
|
||
(In reply to J. Ryan Stinnett [:jryans] from comment #8)
> (In reply to Paul Rouget [:paul] (slow to respond. Ping me on IRC) from
> comment #7)
> > (In reply to J. Ryan Stinnett [:jryans] from comment #6)
> > > Paul, are you sure this is a duplicate? I did not think the iframe selector
> > > would give us access to entirely separate top-level browser windows, so this
> > > seems like very different work would be needed here.
> >
> > tabActor lists subdocshells. The rootActor will list all the docshells. Then
> > it should work out of the box, no?
>
> Maybe, but the implementation of bug 1059308 is a bit unclear for now,
> because frame switching is only part of the tab actor, and the root actor is
> not a tab actor at all.
>
> Let's keep this open and depend on bug 1059308 so it's not lost. When bug
> 1059308 is done, we can check if this came for free and close then.
Bug 1059308 is done. Is there more to do here?
Flags: needinfo?(jryans)
Nope, we got this after all with bug 1059308.
Assignee: nobody → poirot.alex
Status: NEW → RESOLVED
Closed: 10 years ago → 10 years ago
Flags: needinfo?(jryans)
Resolution: --- → FIXED
Target Milestone: --- → Firefox 39
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•