New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 862181 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 852373



Sign in to add a comment

Modify Telemetry to allow specify user agent at page level & change user agent for each page in the test

Project Member Reported by nednguyen@chromium.org, Jul 10

Issue description

Currently we can't merge tough_pinch_zoom_cases to rendering.mobile (issue 852373) because the stories in that story set use the 'desktop' user agent (to test rendering performance of desktop content on mobile), whereas the rest of rendering.mobile uses 'mobile' user agent.

Part of the reason why "user agent" is set at page_set/shared_state level is because in the past, we don't restart browser in between runs. So once you start the browser to run the test, you are stuck with a single user agent (user agent is specified as browser startup flag in https://cs.chromium.org/chromium/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome/chrome_startup_args.py?rcl=985295aafdbfaecd1f27cb6357871f616ca92c99&l=60).

Today, we almost always restart browser between tests (https://cs.chromium.org/chromium/src/third_party/catapult/telemetry/telemetry/page/shared_page_state.py?rcl=985295aafdbfaecd1f27cb6357871f616ca92c99&l=167), so it should be able to possible to move user_agent to be property of page (https://cs.chromium.org/chromium/src/third_party/catapult/telemetry/telemetry/page/__init__.py?rcl=985295aafdbfaecd1f27cb6357871f616ca92c99&l=18)



Another way to approach this is also look into see if there is devtool proptocol that allows us to update user agent without restarting browser.
 

Comment 1 by benhenry@google.com, Jan 16 (6 days ago)

Components: Test>Telemetry

Comment 2 by benhenry@google.com, Jan 16 (6 days ago)

Components: -Speed>Telemetry

Sign in to add a comment