1639095 - Re-remove allowLinkedWebInFileUriProcess

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, task, P3)

Description

[Paul Bone [:pbone]]

In Bug 1603007 we removed the allowLinkedWebInFileUriProcess pref. But this caused several regressions so we've re-introduced a similar behaviour in Bug 1634779. We still want to remove that code and we can do so after Bug 1550571 lands.

Most likely all we need to remove are the changes to E10SUtils.jsm that Bug 1634779 added.

Comment 1

[Chris Peterson [:cpeterson]]

The original "Remove allowLinkedWebInFileUriProcess" bug 1603007 was a Fission Future bug and thus not a Fission MVP blocker. But since we've had some regressions from bug 1603007, perhaps we should remove allowLinkedWebInFileUriProcess for good now, while it's still paged into Paul's head?

Comment 2

[Paul Bone [:pbone]]

I'm going to put it in Future because in terms of delivering Fission it's not important. But that doesn't mean I can't do it sooner if it's convenient.

Comment 3

[Paul Bone [:pbone]]

Attachment: Bug 1639095 - Re-remove allowLinkedWebInFileUriProcess r=kmag

Depends on D77764

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/31524d1785a2
Re-remove allowLinkedWebInFileUriProcess r=kmag

Comment 5

[Atila Butkovits]

Backed out for mochitest failures on browser_documentChannel.js.

Backout link: https://hg.mozilla.org/integration/autoland/rev/f291dd9e075c239586a5b9e266db47750d19af22

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=31524d1785a21500cb940377501bed4ffd30145f&selectedTaskRun=Y56ZVm7oTDi7KD7PLwz26A.0

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=306733111&repo=autoland&lineNumber=7859

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:pbone, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 7

[Paul Bone [:pbone]]

It's blocked on Bug 1650979.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4084e6a07381
Re-remove allowLinkedWebInFileUriProcess r=kmag

Comment 9

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/4084e6a07381