653531 - Shared knowledge of selected node in highlighter and web console

Status: RESOLVED FIXED

Description

[Rob Campbell [:rc] (:robcee)]

We need to implement the $0 helper for the Web Console to gain access to the currently-selected node in the highlighter. This should also be available via the InspectorUI.selection property.

Comment 1

[Panos Astithas [:past] (please needinfo?)]

Created attachment 549057 [details] [diff] [review]
Not working idea

I thought that something like this patch would suffice, but for the life of me I can't find a way to get a reference to the InspectorUI object. The console logging calls are some of my attempts at this. How can I get a reference to something #included in browser.js from a jsm?

Comment 2

[Rob Campbell [:rc] (:robcee)]

I had a similar "not working" idea earlier. If you do a search through HUDService.jsm, you should find a reference to gBrowser. InspectorUI is a property attached to gBrowser so if you can get to that, you'll get to InspectorUI.

Comment 3

[Panos Astithas [:past] (please needinfo?)]

Created attachment 549158 [details] [diff] [review]
Not working idea v2

(In reply to comment #2)
> I had a similar "not working" idea earlier. If you do a search through
> HUDService.jsm, you should find a reference to gBrowser. InspectorUI is a
> property attached to gBrowser so if you can get to that, you'll get to
> InspectorUI.

This doesn't seem to work either :-/

Comment 4

[Panos Astithas [:past] (please needinfo?)]

Created attachment 549400 [details] [diff] [review]
Working version

It was so simple after all, only 7 LOC to get a bloody reference...

Comment 5

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 549400 [details] [diff] [review]
Working version

+      let window = HUDService.currentContext().window;
+      let mw = window.QueryInterface(Components.interfaces.nsIInterfaceRequestor)
+                     .getInterface(Components.interfaces.nsIWebNavigation)
+                     .QueryInterface(Components.interfaces.nsIDocShellTreeItem)
+                     .rootTreeItem
+                   .QueryInterface(Components.interfaces.nsIInterfaceRequestor)
+                     .getInterface(Components.interfaces.nsIDOMWindow);

wah, that' a lotta QI!

Note, you can use the Ci constant for Components.interfaces.

I'm not sure we need to access through the contentWindow. The hudReference has a chromeWindow property which is the XUL window. I think you can access the InspectorUI property directly.

Or HUDService.currentContext() returns the current browser window.

return HUDService.currentContext().InspectorUI.selection *might* work?

Comment 6

[Rob Campbell [:rc] (:robcee)]

yup, replacing all that with:

Object.defineProperty(aJSTerm.sandbox, "$0", {
    get: function() {
      let mw = HUDService.currentContext();
      try {
        return mw.InspectorUI.selection;
      }
      catch (ex) {
        aJSTerm.console.error(ex.message);
      }
    },
    enumerable: true,
    configurable: false
  });

works.

Comment 7

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 549400 [details] [diff] [review]
Working version

r+ with the above change

Comment 8

[Panos Astithas [:past] (please needinfo?)]

Created attachment 549557 [details] [diff] [review]
[in-fx-team] Simpler working patch

Doh, thanks for that, it should help restore my faith in humanity, and Mozillians in particular...

Comment 9

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 549557 [details] [diff] [review]
[in-fx-team] Simpler working patch

http://hg.mozilla.org/integration/fx-team/rev/0d0395cb99ff

Comment 10

[Tim Taubert [:ttaubert]]

http://hg.mozilla.org/mozilla-central/rev/0d0395cb99ff

Comment 11

[Panos Astithas [:past] (please needinfo?)]

Does this bug need a dev-doc-needed keyword, or should we wait for the Highlighter to be enabled first?

Comment 12

[Rob Campbell [:rc] (:robcee)]

(In reply to Panos Astithas [:past] from comment #11)
> Does this bug need a dev-doc-needed keyword, or should we wait for the
> Highlighter to be enabled first?

Yes, I think we should wait until this is turned on.