1620624 - Remove usage of GetURI in nsUrlClassifierUtils.cpp

Status: RESOLVED FIXED

(Core :: DOM: Security, task, P2)

Description

[Sebastian Streich [:sstreich]]

See Metabug

Comment 1

[Sebastian Streich [:sstreich]]

Attachment: Bug 1620624 - Refactor nsUrlClassifier r=ckerschb

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b82effbcaabc
Refactor nsUrlClassifier r=ckerschb

Comment 3

[Cosmin Sabou [:CosminS]]

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

Comment 4

[Alexandru Ionescu (needinfo me) [:alexandrui]]

Sebastian, does the regression below makes sense to you for this bug?
== Change summary for alert #25474 (as of Tue, 24 Mar 2020 04:30:58 GMT) ==

Regressions:

65% build times osx-cross asan asan-fuzzing taskcluster-c5d.4xlarge 821.46 -> 1,351.40

For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=25474

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Backed out on request from sstreich for likely causing performance regression Bug 1625071.

https://hg.mozilla.org/integration/autoland/rev/f70fcfa75fdd81af32133cf9be5575faee2f7685

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Sounds like this can re-land given the recent discussion in bug 1625071.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b0e62a6a0bbd
Refactor nsUrlClassifier r=ckerschb

Comment 8

[Stefan Hindli [:stefan_hindli]]

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

Comment 9

[Sebastian Streich [:sstreich]]

Clearing ni, as since i re-landed the code :)