Bug 1810141 - fix loadURI callers that already have an nsIURI reference or where creating one is clearly safe, r=mossop,geckoview-reviewers,extension-reviewers,settings-reviewers,mconley,m_kato

There are 3 types of changes in this commit:

- from `loadURI(foo.spec)` to `loadURI(foo)`, as there's already a URI
- from `loadURI("string")` to `loadURI(Services.io.newURI("string"))` as the URL is hardcoded
- one or two where there is perhaps an intermediate variable but the patch
  context should still make it trivial to ascertain the change is correct.

Depends on D168393

Differential Revision: https://phabricator.services.mozilla.com/D168394
This commit is contained in:
Gijs Kruitbosch
2023-02-13 12:55:24 +00:00
parent 214358c5c7
commit 6122444136
19 changed files with 42 additions and 24 deletions

View File

@@ -58,7 +58,7 @@ function loadContentWindow(browser, url) {
oa
),
};
browser.loadURI(uri.spec, loadURIOptions);
browser.loadURI(uri, loadURIOptions);
let { webProgress } = browser;
let progressListener = {