1170375 - Add top Japanese sites to unprefixing service whitelist (part 4)

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Mike Taylor [:miketaylr]]

We have a few more sites to diagnose and need to make some tweaks to some existing sites.

To be added:

m.finance.yahoo.co.jp
s.yimg.jp

To be moved to IsOnBaseDomainWhitelist:

tsite.jp (needs s.tsite, img.tsite, tmall.tsite, etc).

I plan on finishing diagnosing the latest sites and writing a patch for this in the next day or so.

Comment 1

[Mike Taylor [:miketaylr]]

Also:

weathernews.jp
cache.ymail.jp, // for www.yamada-denkiweb.com
img.travel.rakuten.co.jp", // for travel.rakuten.co.jp

Comment 2

[Mike Taylor [:miketaylr]]

Attachment: 1170375-Add-4th-batch-of-top-.jp-sites-to-CSS-un.patch

Daniel, can you take a look?

Comment 3

[Daniel Holbert [:dholbert]]

Comment on attachment 8616157 [details] [diff] [review]
1170375-Add-4th-batch-of-top-.jp-sites-to-CSS-un.patch

r=me, just one nit:

>+++ b/caps/nsPrincipal.cpp
>@@ -457,25 +457,28 @@ IsOnFullDomainWhitelist(nsIURI* aURI)
>     NS_LITERAL_CSTRING("m.haosou.com"),
>     NS_LITERAL_CSTRING("m.mi.com"),
>     NS_LITERAL_CSTRING("wappass.baidu.com"),
>     NS_LITERAL_CSTRING("m.video.baidu.com"),
>     NS_LITERAL_CSTRING("m.video.baidu.com"),
>     NS_LITERAL_CSTRING("imgcache.gtimg.cn"), // for m.v.qq.com
>     NS_LITERAL_CSTRING("i.yimg.jp"), // for *.yahoo.co.jp
>     NS_LITERAL_CSTRING("ai.yimg.jp"), // for *.yahoo.co.jp
>+    NS_LITERAL_CSTRING("m.finance.yahoo.co.jp"),
[...]
>     NS_LITERAL_CSTRING("s.tabelog.jp"),
>+    NS_LITERAL_CSTRING("s.yimg.jp"), // for s.tabelog.jp

Let's move these two tabelog-related lines up higher, above "i.yimg.jp", so that we've got our $FOO.yimg.jp lines adjacent to each other.

Comment 4

[Mike Taylor [:miketaylr]]

Makes sense, will do. Thanks Daniel.

Comment 5

[Mike Taylor [:miketaylr]]

Attachment: 1170375-Add-4th-batch-of-top-.jp-sites-to-CSS-un.patch

Updated to address nits and carrying forward r+.

Comment 6

[Mike Taylor [:miketaylr]]

No try run as we're just adding some domains to a whitelist (and building locally succeeds).

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f39cc73b483b

Comment 8

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

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