Generate UA override lists at build time to allow variable substitutions

Bug #1591220 reported by Olivier Tilloy
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Medium
Unassigned
webbrowser-app (Ubuntu)
Fix Released
Medium
Olivier Tilloy

Bug Description

The UA override lists are plain javascript files shipped by qml-module-ubuntu-web. The overrides are hardcoded, and often contain version numbers such as the webkit build version, the chrome/chromium build version, the ubuntu release, e.g.:

  "Mozilla/5.0 (Linux; Ubuntu 14.04 like Android 4.4) AppleWebKit/537.36 Chromium/35.0.1870.2 Mobile Safari"

It would be useful if we could generate those files at build time, and allow for variables to be replaced. For example, the above UA string would become:

  "Mozilla/5.0 (Linux; Ubuntu @UBUNTU_VERSION@ like Android 4.4) AppleWebKit/@WEBKIT_VERSION@ Chromium/@CHROMIUM_VERSION@ Mobile Safari/@WEBKIT_VERSION@"

Not all overrides would be required to use those variables, if necessary some version numbers could remain hardcoded.

Related branches

Olivier Tilloy (osomon)
Changed in webbrowser-app (Ubuntu):
importance: Undecided → Medium
Olivier Tilloy (osomon)
Changed in webbrowser-app (Ubuntu):
status: New → In Progress
assignee: nobody → Olivier Tilloy (osomon)
Changed in canonical-devices-system-image:
importance: Undecided → Medium
milestone: none → 13
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package webbrowser-app - 0.23+16.10.20160701.1-0ubuntu1

---------------
webbrowser-app (0.23+16.10.20160701.1-0ubuntu1) yakkety; urgency=medium

  * Generate UA override list files at build time to un-hardcode ubuntu
    and chromium version numbers. (LP: #1591220)
  * Fix one flaky autopilot test. (LP: #1591120)
  * Temporarily skip tests on arm64 to unblock package build without
    oxide. (LP: #1585942)
  * Update the target that lists non-compiled files.

 -- Olivier Tilloy <email address hidden> Fri, 01 Jul 2016 13:10:14 +0000

Changed in webbrowser-app (Ubuntu):
status: In Progress → Fix Released
Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.