1743702 - Make browser_ProcessPriorityManager.js load pages with https

Status: RESOLVED FIXED

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

Description

[Andrew McCreight [:mccr8]]

This takes Christoph's patch in bug 1742785 for browser_ProcessPriorityManager.js and alters them so the tests still pass. My only change was making it so that "http://mochi.test:8888" URLs are instead all mapped to "https://example.net", which ensures that they continue to not be same-site with example.com and example.org. The not-same-site-ness is needed for some of the fiddly Fission tests.

I tried testing with dom.security.https_first set to true, and they passed, but they also passed without changing mochi.test:8888 to an HTTPS URI so I might have been doing something wrong.

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Bug 1743702 - Make browser_ProcessPriorityManager.js use https.

The trick here is to ensure that URLs continue to not be same-site after
changing to https URLs.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c101c00e8dce
Make browser_ProcessPriorityManager.js use https. r=ckerschb

Comment 3

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/c101c00e8dce