Closed Bug 1508171 Opened 7 years ago Closed 5 years ago

Remove gBrowser.initialBrowser

Categories

(Firefox :: Tabbed Browser, task, P3)

task

Tracking

()

RESOLVED FIXED
Firefox 75
Tracking Status
firefox75 --- fixed

People

(Reporter: Gijs, Assigned: dao)

References

Details

Attachments

(1 file)

After bug 1506608 we don't actually need this property in the browser, so we should just remove it.
Joel, I'm seeing this in searchfox, can I just change this in m-c or is there another repo for these tests? ( https://searchfox.org/mozilla-central/search?q=.initialBrowser )
Component: General → Talos
Flags: needinfo?(jmaher)
Product: Firefox → Testing
(In reply to :Gijs (he/him) from comment #1) > Joel, I'm seeing this in searchfox, can I just change this in m-c or is > there another repo for these tests? ( > https://searchfox.org/mozilla-central/search?q=.initialBrowser ) Better link: https://searchfox.org/mozilla-central/search?q=.initialBrowser&case=false&regexp=false&path=talos
that is the only place for talos code, so if you update and validate the results on try, that would be excellent.
Flags: needinfo?(jmaher)
Priority: -- → P3

XULBrowserWindow.forceInitialBrowserNonRemote also uses gBrowser.initialBrowser. Afaik we can just use gBrowser.selectedBrowser or gBrowser.browsers[0] there.

Type: enhancement → task
Component: Talos → Tabbed Browser
Product: Testing → Firefox
Summary: Make pageloader tests not access gBrowser.initialBrowser so we can get rid of it. → Remove gBrowser.initialBrowser
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Blocks: 1616603
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 75
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: