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

RESOLVED FIXED in Firefox 41

Status

()

Core
CSS Parsing and Computation
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: miketaylr, Assigned: miketaylr)

Tracking

unspecified
mozilla41
Points:
---

Firefox Tracking Flags

(p11+, firefox41 fixed)

Details

(Whiteboard: [compat])

Attachments

(1 attachment, 1 obsolete attachment)

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.
Assignee: nobody → miket
tracking-p11: --- → ?
See Also: → bug 1162245
Also:

weathernews.jp
cache.ymail.jp, // for www.yamada-denkiweb.com
img.travel.rakuten.co.jp", // for travel.rakuten.co.jp
Created attachment 8616157 [details] [diff] [review]
1170375-Add-4th-batch-of-top-.jp-sites-to-CSS-un.patch

Daniel, can you take a look?
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.
Attachment #8616157 - Flags: review?(dholbert) → review+
Makes sense, will do. Thanks Daniel.
Created attachment 8617086 [details] [diff] [review]
1170375-Add-4th-batch-of-top-.jp-sites-to-CSS-un.patch

Updated to address nits and carrying forward r+.
Attachment #8616157 - Attachment is obsolete: true
No try run as we're just adding some domains to a whitelist (and building locally succeeds).
Keywords: checkin-needed
tracking-p11: ? → +
Whiteboard: [compat]

Comment 7

3 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/f39cc73b483b
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/f39cc73b483b
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41

Updated

3 years ago
See Also: → bug 1166792

Updated

3 years ago
Blocks: 1162245
You need to log in before you can comment on or make changes to this bug.