Update the chromium version in the user agent
Bug #1565063 reported by
Pat McGowan
This bug affects 21 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical System Image |
Fix Released
|
Medium
|
Bill Filler | ||
Oxide |
Fix Released
|
Medium
|
Olivier Tilloy | ||
webbrowser-app (Ubuntu) |
Fix Released
|
Medium
|
Olivier Tilloy | ||
Xenial |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
We have never updated the version, which is 35, and now we are based on 49
While this may not have any impact it does reflect the core support we have, and its possible some sites may check it.
Ideally we would query oxide for this dynamically.
[Manual test case]
Open the browser and browse to http://
Verify that the chromium version number in the UA string is a recent one (> 50), and not the previously hardcoded value of "35.0.1870.2".
Related branches
lp:~osomon/oxide/versions
- Chris Coulson: Approve
-
Diff: 204 lines (+85/-5)8 files modifiedqt/core/api/oxideqglobal.cc (+13/-1)
qt/core/api/oxideqglobal.h (+5/-2)
qt/core/common/oxide_version.h.in (+23/-0)
qt/core/core.gyp (+18/-0)
qt/qmlplugin/oxide.qmltypes (+2/-0)
qt/quick/api/oxideqquickglobal.cc (+9/-1)
qt/quick/api/oxideqquickglobal_p.h (+7/-1)
qt/tests/qmltests/api/tst_OxideGlobal.qml (+8/-0)
lp:~osomon/webbrowser-app/ua-chromium-version-from-oxide
- PS Jenkins bot: Needs Fixing (continuous-integration)
- system-apps-ci-bot: Needs Fixing (continuous-integration)
- Ubuntu Phablet Team: Pending requested
-
Diff: 43 lines (+4/-7)2 files modifieddebian/control (+1/-1)
src/Ubuntu/Web/UserAgent02.qml (+3/-6)
lp:~osomon/webbrowser-app/use-oxide-1.15-apis
- system-apps-ci-bot: Needs Fixing (continuous-integration)
- Ubuntu Phablet Team: Pending requested
-
Diff: 295 lines (+126/-21)6 files modifieddebian/control (+2/-2)
src/Ubuntu/Web/UbuntuWebView02.qml (+22/-4)
src/Ubuntu/Web/UserAgent02.qml (+2/-5)
src/app/webbrowser/Browser.qml (+36/-1)
tests/autopilot/webbrowser_app/tests/test_keyboard.py (+49/-1)
tests/autopilot/webbrowser_app/tests/test_touchselection.py (+15/-8)
lp:~osomon/webbrowser-app/xenial-sru-1
- Ubuntu Phablet Team: Pending requested
-
Diff: 905 lines (+274/-86)27 files modifieddebian/changelog (+15/-0)
debian/control (+2/-1)
src/Ubuntu/CMakeLists.txt (+4/-0)
src/Ubuntu/Web/UserAgent02.qml (+4/-10)
src/Ubuntu/Web/plugin.cpp (+7/-0)
src/Ubuntu/Web/ua-overrides-desktop.js (+5/-0)
src/Ubuntu/Web/ua-overrides-mobile.js (+6/-1)
src/app/browserapplication.cpp (+32/-24)
src/app/browserapplication.h (+1/-2)
src/app/config.h.in (+0/-1)
src/app/single-instance-manager.cpp (+39/-3)
src/app/single-instance-manager.h (+1/-1)
src/app/webbrowser/Browser.qml (+21/-15)
src/app/webbrowser/BrowserTab.qml (+3/-2)
src/app/webbrowser/NavigationBar.qml (+1/-2)
src/app/webbrowser/webbrowser-app.cpp (+1/-1)
src/app/webcontainer/WebApp.qml (+1/-0)
src/app/webcontainer/WebViewImplOxide.qml (+28/-14)
src/app/webcontainer/WebappContainerWebview.qml (+3/-1)
src/app/webcontainer/webapp-container.cpp (+32/-1)
src/app/webcontainer/webapp-container.h (+1/-0)
src/app/webcontainer/webapp-container.qml (+2/-0)
tests/autopilot/webapp_container/tests/__init__.py (+4/-0)
tests/autopilot/webapp_container/tests/fake_servers.py (+29/-0)
tests/autopilot/webapp_container/tests/test_popup_webview_overlay.py (+19/-0)
tests/autopilot/webapp_container/tests/test_saml_url_patterns.py (+9/-3)
tests/unittests/single-instance-manager/tst_SingleInstanceManagerTests.cpp (+4/-4)
Changed in webbrowser-app (Ubuntu): | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in oxide: | |
status: | New → Confirmed |
assignee: | nobody → Olivier Tilloy (osomon) |
Changed in webbrowser-app (Ubuntu): | |
assignee: | nobody → Olivier Tilloy (osomon) |
Changed in oxide: | |
importance: | Undecided → Medium |
Changed in oxide: | |
milestone: | none → branch-1.15 |
status: | Confirmed → In Progress |
Changed in oxide: | |
status: | In Progress → Fix Released |
Changed in webbrowser-app (Ubuntu): | |
status: | Confirmed → In Progress |
Changed in canonical-devices-system-image: | |
status: | New → In Progress |
Changed in canonical-devices-system-image: | |
importance: | Undecided → Medium |
assignee: | nobody → Bill Filler (bfiller) |
milestone: | none → 12 |
Changed in canonical-devices-system-image: | |
status: | In Progress → Fix Committed |
Changed in webbrowser-app (Ubuntu): | |
status: | In Progress → Fix Released |
milestone: | none → ubuntu-16.10 |
Changed in canonical-devices-system-image: | |
status: | Fix Committed → Fix Released |
description: | updated |
To post a comment you must log in.
In oxide, the CHROME_ VERSION_ STRING token contains the chromium version that it’s built against, so we could possibly expose that to the QML API as a readonly property of OxideQQuickGlobal.