DevTools openWebLink helper is not working if DevTools are in a separate window

RESOLVED FIXED in Firefox 61

Status

defect
P2
normal
RESOLVED FIXED
Last year
Last year

People

(Reporter: jdescottes, Assigned: jdescottes)

Tracking

61 Branch
Firefox 61
Dependency tree / graph

Firefox Tracking Flags

(firefox61 fixed)

Details

Attachments

(2 attachments)

STRs:
- open devtools 
- undock
- meatball menu > click "Documentation..." or  "Community..."

ER: a new tab should open
AR: nothing happens
Comment on attachment 8972629 [details]
Bug 1458591 - Use most recent chrome window for DevTools link helper;

In other spots in the code base, we use Services.wm.getMostRecentWindow in order to get a valid window for opening links, so proposing this here.

Before I get into cleanup and test writing, do you think this approach is valid here or are there limitations to consider?
Attachment #8972629 - Flags: feedback?(jryans)
Comment on attachment 8972629 [details]
Bug 1458591 - Use most recent chrome window for DevTools link helper;

It seems fine to me.  I suggested[1] doing basically this to :birtles when he added the link helper, but for some reason it failed for him.  Maybe it was because things were renamed to `openWebLink` around the same time as his work?

Anyway, it _should_ be doable this way.

[1]: https://bugzilla.mozilla.org/show_bug.cgi?id=1444301#c74
Attachment #8972629 - Flags: feedback?(jryans) → feedback+
Ah interesting, thanks for the feedback. Not sure how accurate the snippet from the initial comment is but:

  if (!win || !win.openUILinkIn !== "function") { return; }

is always going to be true and return. Hopefully this was the issue and not a tricky edge case.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Try is at https://treeherder.mozilla.org/#/jobs?repo=try&revision=1282628cb46bde1f809b0c23c3945e309f691bc9

(added a small changeset to remove an unused proptype, can move to separate bug if needed)
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED

Comment 8

Last year
mozreview-review
Comment on attachment 8972629 [details]
Bug 1458591 - Use most recent chrome window for DevTools link helper;

https://reviewboard.mozilla.org/r/241180/#review247110

Looks reasonable, thanks for working on it! :)
Attachment #8972629 - Flags: review?(jryans) → review+

Comment 9

Last year
mozreview-review
Comment on attachment 8972672 [details]
Bug 1458591 - remove unused toolbar propType;

https://reviewboard.mozilla.org/r/241210/#review247112
Attachment #8972672 - Flags: review?(jryans) → review+

Comment 10

Last year
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5da4a0c6ba28
Use most recent chrome window for DevTools link helper;r=jryans
https://hg.mozilla.org/integration/autoland/rev/5be595f4f1d9
remove unused toolbar propType;r=jryans

Comment 11

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/5da4a0c6ba28
https://hg.mozilla.org/mozilla-central/rev/5be595f4f1d9
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61

Updated

Last year
Product: Firefox → DevTools
Depends on: 1467431
You need to log in before you can comment on or make changes to this bug.