1936834 - SmartBlock should use GetHost instead of DisplayHost

Status: RESOLVED FIXED

(Core :: Privacy: Anti-Tracking, defect, P3)

Description

[edgul]

DisplayHost is ideally used for creating human-readable strings for user display purposes. URLs that are used for logic should probably be using GetHost or similar.

I'm not sure if all of these uses fall under the same component, but these uses of displayHost should be updated to use getHost:

• https://searchfox.org/mozilla-central/rev/7fb746f0be47ce0135af7bca9fffdb5cd1c4b1d5/browser/extensions/webcompat/experiment-apis/trackingProtection.js#191
• https://searchfox.org/mozilla-central/rev/7fb746f0be47ce0135af7bca9fffdb5cd1c4b1d5/browser/extensions/webcompat/experiment-apis/trackingProtection.js#207
• https://searchfox.org/mozilla-central/rev/7fb746f0be47ce0135af7bca9fffdb5cd1c4b1d5/mobile/android/android-components/components/feature/webcompat/src/main/assets/extensions/webcompat/experiment-apis/trackingProtection.js#191
• https://searchfox.org/mozilla-central/rev/7fb746f0be47ce0135af7bca9fffdb5cd1c4b1d5/mobile/android/android-components/components/feature/webcompat/src/main/assets/extensions/webcompat/experiment-apis/trackingProtection.js#207

Thanks!

Comment 1

[William Wen (:wwen)]

Attachment: Bug 1936834 - Use host instead of displayHost for smartblock logic. r=edgul

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/43ec469c601b
Use host instead of displayHost for smartblock logic. r=edgul,twisniewski

Comment 3

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/43ec469c601b