webbrowser-app

Merge lp:~osomon/webbrowser-app/angrybirds-chrome-ua-override into lp:webbrowser-app

  1. angrybirds-chrome-ua-override
  2. Merge into trunk
Proposed by Olivier Tilloy
Status: Merged
Approved by: Olivier Tilloy
Approved revision: 598
Merged at revision: 599
Proposed branch: lp:~osomon/webbrowser-app/angrybirds-chrome-ua-override
Merge into: lp:webbrowser-app
Diff against target: 19 lines (+2/-0)
2 files modified
src/Ubuntu/Web/ua-overrides-desktop.js (+1/-0)
src/Ubuntu/Web/ua-overrides-mobile.js (+1/-0)
To merge this branch: bzr merge lp:~osomon/webbrowser-app/angrybirds-chrome-ua-override
Reviewer Review Type Date Requested Status
Adnane Belmadiaf (community) Approve
David Barth (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+225451@code.launchpad.net

Commit message

Override the UA string for chrome.angrybirds.com (on both desktop and mobile).

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:598
http://jenkins.qa.ubuntu.com/job/webbrowser-app-ci/913/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1521
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic/1319
    SUCCESS: http://jenkins.qa.ubuntu.com/job/webbrowser-app-utopic-amd64-ci/112
    SUCCESS: http://jenkins.qa.ubuntu.com/job/webbrowser-app-utopic-armhf-ci/112
        deb: http://jenkins.qa.ubuntu.com/job/webbrowser-app-utopic-armhf-ci/112/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/webbrowser-app-utopic-i386-ci/112
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/1803
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2502
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2502/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/9240
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/1090
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/1483
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/1483/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/webbrowser-app-ci/913/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
David Barth (dbarth) wrote :

Works fine here

review: Approve
Revision history for this message
Adnane Belmadiaf (daker) wrote :

Works fine here too.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/Ubuntu/Web/ua-overrides-desktop.js'
2--- src/Ubuntu/Web/ua-overrides-desktop.js 2014-05-29 15:30:11 +0000
3+++ src/Ubuntu/Web/ua-overrides-desktop.js 2014-07-03 10:35:25 +0000
4@@ -20,4 +20,5 @@
5
6 var overrides = [
7 [/^https?:\/\/(www\.)?google\.com\/calendar/, "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 Chromium/35.0.1870.2 Chrome/35.0.1870.2 Safari/537.36"],
8+ [/^http:\/\/chrome\.angrybirds\.com\//, "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 Chrome/35.0.1870.2 Safari/537.36"], // http://pad.lv/1284158
9 ];
10
11=== modified file 'src/Ubuntu/Web/ua-overrides-mobile.js'
12--- src/Ubuntu/Web/ua-overrides-mobile.js 2014-06-09 10:44:22 +0000
13+++ src/Ubuntu/Web/ua-overrides-mobile.js 2014-07-03 10:35:25 +0000
14@@ -21,4 +21,5 @@
15 var overrides = [
16 [/^https?:\/\/mail\.google\.com\//, "Mozilla/5.0 (Linux; Ubuntu 14.04 like Android 4.4) AppleWebKit/537.36 Chromium/35.0.1870.2 Mobile Safari"],
17 [/^https?:\/\/m\.youtube\.com\//, "Mozilla/5.0 (Linux; Ubuntu 14.04 like Android 4.4;) AppleWebKit/537.36 Chromium/35.0.1870.2 Mobile Safari/537.36"], // http://pad.lv/1228415
18+ [/^http:\/\/chrome\.angrybirds\.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/1284158
19 ];

Subscribers

People subscribed via source and target branches

to status/vote changes: