touch.dailymotion.com is mobile optimized for Fennec UA today. lets do the same for the b2g UA: Mozilla/5.0 (Mobile; rv:12.0) Gecko/12.0 Firefox/12.0 Currently on 10-19-2012 build, it shows the desktop site: www.dailymotion.com
Summary: Add UA override for yelp.com for B2G → Add UA override for dailymotion.com for B2G
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 798694
Oops, too quick fingers.
Assignee: nobody → lmandel
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
resetting blocker flag
blocking-basecamp: + → ?
Oh yeah, my justification for blocking+ is that touch.dailymotion.com falls back video content to h.264 on Fennec. We need the same for b2g.
(In reply to Tony Chung [:tchung] from comment #4) > Oh yeah, my justification for blocking+ is that touch.dailymotion.com falls > back video content to h.264 on Fennec. We need the same for b2g. If we don't fix this, how well does the site operate? Is it a top site in any locales we are tracking?
This is the same bug as 803742... We want one bug for "contact DailyMotion", and one bug for "add a UA override". We seem to have two of the latter, and none of the former. https://wiki.mozilla.org/Evangelism/UA_Override_List_Policy is how this should work. :-) Gerv
Bug 803742 is a tech evangelism bug.
High enough alexa ranking to justify fixing.
blocking-basecamp: ? → +
Milestoning for C2 (deadline of 12/10), as this meets the criteria of "known P2 bugs found before or during C1".
Target Milestone: --- → B2G C2 (20nov-10dec)
I have a couple question. My understanding is I should do roughly the same as https://github.com/mozilla-b2g/gaia/pull/6435/files but for dailymotion.com. This patch does: +pref("general.useragent.override.yelp.com", "\(Mobile#(Android; Mobile"); I need to know what the second string here, "\(Mobile#(Android; Mobile", really does (what syntax is that) and how I adapt it to do the UA string overriding asked for in comment 0 --- is the patch as simple as +pref("general.useragent.override.dailymotion.com", "\(Mobile#(Android; Mobile"); ?
Ah, got it: http://mxr.mozilla.org/mozilla-central/source/netwerk/protocol/http/UserAgentOverrides.jsm#74 So this was just about replacing "Mobile" by "Android; Mobile" in the UA string. Is this what you want done here as well for dailymotion?
Can you also add a comment to that block of prefs explaining the format? :-) Gerv
Created attachment 686558 [details] [diff] [review] Add UA override for dailymotion
Attachment #686558 - Flags: review?(21)
pull request: https://github.com/mozilla-b2g/gaia/pull/6725
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago → 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.