Closed Bug 1315944 Opened 3 years ago Closed 3 years ago

Should use aCurrentBrowser instead of w.gBrowser.selectedBrowser when creating a blank content viewer

Categories

(Firefox :: Tabbed Browser, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 52
Tracking Status
firefox52 --- fixed

People

(Reporter: Gijs, Assigned: Gijs)

References

Details

Attachments

(1 file)

https://dxr.mozilla.org/mozilla-central/rev/86f702229e32c6119d092e86431afee576f033a1/browser/base/content/utilityOverlay.js#376,379

I'm fairly sure that both of these are the same anyway, but really we should always manipulate the about:blank viewer for the tab in which we're loading, not a different one.
Comment on attachment 8808594 [details]
Bug 1315944 - ensure about blank viewer creation happens in the right browser,

https://reviewboard.mozilla.org/r/91404/#review91236
Attachment #8808594 - Flags: review?(florian) → review+
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/d8528906686e
ensure about blank viewer creation happens in the right browser, r=florian
https://hg.mozilla.org/mozilla-central/rev/d8528906686e
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
You need to log in before you can comment on or make changes to this bug.