Merge lp:~osomon/webbrowser-app/fix-uastring-regression into lp:webbrowser-app

Proposed by Olivier Tilloy
Status: Merged
Approved by: Günter Schwann
Approved revision: 180
Merged at revision: 180
Proposed branch: lp:~osomon/webbrowser-app/fix-uastring-regression
Merge into: lp:webbrowser-app
Diff against target: 15 lines (+2/-2)
1 file modified
src/Ubuntu/Browser/UbuntuWebView.qml (+2/-2)
To merge this branch: bzr merge lp:~osomon/webbrowser-app/fix-uastring-regression
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Günter Schwann (community) Approve
Review via email: mp+169133@code.launchpad.net

Commit message

Fix the conditional user-agent string based on form factor.

To post a comment you must log in.
Revision history for this message
Günter Schwann (schwann) wrote :

lgtm

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/Ubuntu/Browser/UbuntuWebView.qml'
--- src/Ubuntu/Browser/UbuntuWebView.qml 2013-06-07 08:37:10 +0000
+++ src/Ubuntu/Browser/UbuntuWebView.qml 2013-06-13 08:46:25 +0000
@@ -59,9 +59,9 @@
59 // this should be changed to a more neutral user-agent in the59 // this should be changed to a more neutral user-agent in the
60 // future as we don’t want websites to recommend installing60 // future as we don’t want websites to recommend installing
61 // their iPhone/iPad apps.61 // their iPhone/iPad apps.
62 if (formFactor === formFactor.phone) {62 if (_webview.formFactor === formFactor.phone) {
63 return "Mozilla/5.0 (iPhone; CPU iPhone OS 5_0 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9A334 Safari/7534.48.3"63 return "Mozilla/5.0 (iPhone; CPU iPhone OS 5_0 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9A334 Safari/7534.48.3"
64 } else if (formFactor === formFactor.tablet) {64 } else if (_webview.formFactor === formFactor.tablet) {
65 return "Mozilla/5.0 (iPad; CPU OS 5_0 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9A334 Safari/7534.48.3"65 return "Mozilla/5.0 (iPad; CPU OS 5_0 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9A334 Safari/7534.48.3"
66 } else {66 } else {
67 return ""67 return ""

Subscribers

People subscribed via source and target branches

to status/vote changes: