1391099 - checkURIStrWithPrincipal is too slow

Status: RESOLVED FIXED

(WebExtensions :: General, enhancement)

Description

[Kris Maglione [:kmag]]

It runs URIs through the fixup service and checks them multiple times. We don't need this for our purposes.

Comment 1

[Kris Maglione [:kmag]]

Attachment: Bug 1391099: Avoid using checkLoadURIStrWithPrincipal.

checkLoadURIStrWithPrincipal runs URLs through the URI fixup services and
checks against each of the results. This is both expensive and unnecessary for
our purposes.

Review commit: https://reviewboard.mozilla.org/r/169352/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/169352/

Comment 2

[Kris Maglione [:kmag]]

Comment on attachment 8898043 [details]
Bug 1391099: Avoid using checkLoadURIStrWithPrincipal.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/169352/diff/1-2/

Comment 3

[Shane Caraveo (:mixedpuppy)]

Comment on attachment 8898043 [details]
Bug 1391099: Avoid using checkLoadURIStrWithPrincipal.

https://reviewboard.mozilla.org/r/169352/#review174740

Comment 4

[Kris Maglione [:kmag]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7e26c8dc7d76c31057f434d7766f4dd27b250b81
Bug 1391099: Avoid using checkLoadURIStrWithPrincipal. r=mixedpuppy

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/7e26c8dc7d76