webbrowser-app

Merge lp:~santoshbit2007/webbrowser-app/webbrowser-app into lp:webbrowser-app

  1. webbrowser-app
  2. Merge into trunk
Proposed by Santosh
Status: Merged
Approved by: Olivier Tilloy
Approved revision: 1474
Merged at revision: 1484
Proposed branch: lp:~santoshbit2007/webbrowser-app/webbrowser-app
Merge into: lp:webbrowser-app
Diff against target: 25 lines (+7/-0)
2 files modified
debian/changelog (+6/-0)
src/Ubuntu/Web/ua-overrides-mobile.js (+1/-0)
To merge this branch: bzr merge lp:~santoshbit2007/webbrowser-app/webbrowser-app
Reviewer Review Type Date Requested Status
Olivier Tilloy Approve
David Barth Pending
Review via email: mp+296521@code.launchpad.net

Commit message

Add a UA override for mobile nytimes.com.

Description of the change

overriding UA for nytimes for bug #1573620

To post a comment you must log in.
Revision history for this message
Olivier Tilloy (osomon) wrote :

Looks good to me, and fixes the bug indeed.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2016-05-26 17:01:10 +0000
3+++ debian/changelog 2016-06-07 04:41:41 +0000
4@@ -1,3 +1,9 @@
5+webbrowser-app (0.23+16.10.20160526-0ubuntu3) yakkety; urgency=medium
6+
7+ * Overriding UA for nytimes (LP: #1573620)
8+
9+ -- Santosh Mahto <santosh.mahto@canonical.com> Tue, 07 Jun 2016 09:49:12 +0530
10+
11 webbrowser-app (0.23+16.10.20160526-0ubuntu1) yakkety; urgency=medium
12
13 [ Alexandre Abreu ]
14
15=== modified file 'src/Ubuntu/Web/ua-overrides-mobile.js'
16--- src/Ubuntu/Web/ua-overrides-mobile.js 2016-04-25 19:28:03 +0000
17+++ src/Ubuntu/Web/ua-overrides-mobile.js 2016-06-07 04:41:41 +0000
18@@ -25,6 +25,7 @@
19 ["^https?:\/\/(\w+\.)*hsbc\.com\.br\/", "Mozilla/5.0 (Linux; Ubuntu 14.04 like Android 4.4;) AppleWebKit/537.36 Chrome/35.0.1870.2 Mobile Safari/537.36"], // http://pad.lv/1380657
20 ["^http:\/\/(\w+\.)*espn\.(go\.)?com\/", "Mozilla/5.0 (Linux; Ubuntu 14.04 like Android 4.4;) AppleWebKit/537.36 Chrome/35.0.1870.2 Mobile Safari/537.36"], // http://pad.lv/1316259
21 ["^https?:\/\/(www|m)\.facebook\.com\/", "Mozilla/5.0 (Linux; Ubuntu 14.04; Android 5.0; Nexus 5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/38.0.2125.102 Mobile Safari/537.36"], // http://pad.lv/1538056, http://pad.lv/1457661
22+ ["^https?:\/\/(mobile\.)?nytimes\.com\/", "Mozilla/5.0 (Linux; Ubuntu 14.04; Android 5.0; Nexus 5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/38.0.2125.102 Mobile Safari/537.36"], // http://pad.lv/1573620
23 // Google hangouts (https://launchpad.net/bugs/1565055)
24 ["^https?:\/\/hangouts\.google\.com\/", "Mozilla/5.0 (Linux; Ubuntu 14.04 like Android 4.4;) AppleWebKit/537.36 Chrome/49.0.2623.87 Mobile Safari/537.36"],
25 ["^https?:\/\/talkgadget\.google\.com\/hangouts\/", "Mozilla/5.0 (Linux; Ubuntu 14.04 like Android 4.4;) AppleWebKit/537.36 Chrome/49.0.2623.87 Mobile Safari/537.36"],

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: