1646582 - Remove RemoteWebProgressManager

Status: RESOLVED FIXED

(Firefox :: General, task)

Description

[Matt Woodrow (:mattwoodrow)]

Bug 1644943 added a new nsIWebProgress that stays available across process switches, and works for both remote and local <browser>s.

We should be able to remove all of RemoteWebProgressManager now, since it isn't being used.

Comment 1

[Matt Woodrow (:mattwoodrow)]

Attachment: Bug 1646582 - Remove nsIRemoteWebProgressRequest since it's not being used anywhere. r?barret

Comment 2

[Matt Woodrow (:mattwoodrow)]

Attachment: Bug 1646582 - Remove DOM(Inner)WindowID from nsIWebProgress. r?nika

Depends on D80127

Comment 3

[Matt Woodrow (:mattwoodrow)]

Attachment: Bug 1646582 - Remove RemoteWebProgressManager. r?nika,Gijs

Depends on D80128

Comment 4

[Matt Woodrow (:mattwoodrow)]

Attachment: Bug 1646582 - Pull the inner window id from the WindowGlobalParent, rather than passing it across PBrowser for updateForLocationChange. r?nika

Depends on D80129

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7de0ff1af560
Remove nsIRemoteWebProgressRequest since it's not being used anywhere. r=barret
https://hg.mozilla.org/integration/autoland/rev/74d89cb24414
Remove DOM(Inner)WindowID from nsIWebProgress. r=nika
https://hg.mozilla.org/integration/autoland/rev/2ac69a2b6708
Remove RemoteWebProgressManager. r=nika,Gijs
https://hg.mozilla.org/integration/autoland/rev/ce5cdfc5d7cf
Pull the inner window id from the WindowGlobalParent, rather than passing it across PBrowser for updateForLocationChange. r=nika,Gijs

Comment 6

[Narcis Beleuzu [:NarcisB]]

Backed out for mochitest failures on test_ext_tabs_create.html

Backout link: https://hg.mozilla.org/integration/autoland/rev/f2a31b56a7e507e4c59a923662f444b5cc2a1cbf
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=307182631&repo=autoland&lineNumber=6650

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9d42ec89240c
Remove nsIRemoteWebProgressRequest since it's not being used anywhere. r=barret
https://hg.mozilla.org/integration/autoland/rev/49b4ae8c8be0
Remove DOM(Inner)WindowID from nsIWebProgress. r=nika
https://hg.mozilla.org/integration/autoland/rev/e6c496c5a4b1
Remove RemoteWebProgressManager. r=nika,Gijs
https://hg.mozilla.org/integration/autoland/rev/2111dd0d6dc0
Pull the inner window id from the WindowGlobalParent, rather than passing it across PBrowser for updateForLocationChange. r=nika,Gijs

Comment 8

[Dorel Luca [:dluca]]

Backed out 4 changesets (bug 1646582) for Browser-chrome failures on nsIWebProgress.addProgressListener. CLOSED TREE

Log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=307332531&repo=autoland&lineNumber=11774

Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&revision=2111dd0d6dc0487b1120ce38d919cb4c9fe98894

Backout:
https://hg.mozilla.org/integration/autoland/rev/697bb0cad425724b3dff1be4d8ac365bbf7a895c

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9fc2e30af285
Remove nsIRemoteWebProgressRequest since it's not being used anywhere. r=barret
https://hg.mozilla.org/integration/autoland/rev/1a749d5aaa71
Remove DOM(Inner)WindowID from nsIWebProgress. r=nika
https://hg.mozilla.org/integration/autoland/rev/14b9b4515aab
Remove RemoteWebProgressManager. r=nika,Gijs
https://hg.mozilla.org/integration/autoland/rev/47d0de3f55cd
Pull the inner window id from the WindowGlobalParent, rather than passing it across PBrowser for updateForLocationChange. r=nika,Gijs

Comment 10

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/9fc2e30af285
https://hg.mozilla.org/mozilla-central/rev/1a749d5aaa71
https://hg.mozilla.org/mozilla-central/rev/14b9b4515aab
https://hg.mozilla.org/mozilla-central/rev/47d0de3f55cd