Closed Bug 1279833 Opened 4 years ago Closed 4 years ago

Make less use of gBrowser and content tabs in devtools

Categories

(DevTools :: General, enhancement)

Product:
DevTools
Component:
General
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(firefox50 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 50
Tracking Flags:
Tracking Status
firefox50 --- fixed

People

(Reporter: Fallen, Assigned: Fallen)

References

Details

Attachments

(1 file, 1 obsolete file)

Fix - v2
10.14 KB, patch
Fallen
: review+
Details | Diff | Splinter Review 
In order to reduce the number of shims needed for Thunderbird, I'd like to reduce the use of gBrowser in devtools, mostly by replacing gBrowser.addTab with openUILinkIn.

I also went ahead and made some changes to the eyedropper tool so it works even if there is no content tab to screenshot. This kind of fits into the same area of making devtools less Firefox-specific, if you'd prefer to split these patches let me know.
Attached patch Fix - v1 (obsolete) — — Splinter Review 
This patch depends on bug 1279784, so here is a combined try push.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=f61e19d9ed3e
Attachment #8762426 - Flags: review?(jwalker)
Attachment #8762426 - Flags: review?(jwalker) → review+
Depends on: 1279784
Keywords: checkin-needed
Attached patch Fix - v2 — — Splinter Review 
Debitrotted patch for checkin
Attachment #8762426 - Attachment is obsolete: true
Attachment #8765800 - Flags: review+
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/fx-team/rev/3f58f958a982
Make less use of gBrowser and content tabs in devtools. r=jwalker
Keywords: checkin-needed
Severity: normal → enhancement
https://hg.mozilla.org/mozilla-central/rev/3f58f958a982
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.