Closed Bug 1605454 Opened 5 years ago Closed 5 years ago

[Experiment]: Generic Experiment: Fenix Webrender performance experiment

Categories

(Core :: Graphics: WebRender, task, P3)

73 Branch
task

Tracking

()

RESOLVED FIXED
mozilla74
Tracking Status
firefox73 --- fixed
firefox74 --- fixed

People

(Reporter: experimenter, Assigned: ktaeleman)

References

()

Details

Attachments

(1 file, 1 obsolete file)

Control webrender rollout for Fenix

The graphics team is ready to start rolling out Webrender on specific device models. This experiment aims to reduce the risk and give the ability to monitor the impact of the change.

Experimenter is the source of truth for details and delivery. Changes to Bugzilla are not reflected in Experimenter and will not change delivery configuration.

More information: https://experimenter.services.mozilla.com/experiments/control-webrender-rollout-for-fenix/

Attachment #9118212 - Attachment is obsolete: true
Pushed by ktaeleman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/debd90b3f579
Use GeckoRuntimeSettings intent extras to avoid API change for webrender fenix experiment. r=snorp
Backout by aiakab@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bd68783bf033
Backed out changeset debd90b3f579 for causing lint failure on GeckoRuntime.java
Pushed by ktaeleman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/45ed19600a37
Use GeckoRuntimeSettings intent extras to avoid API change for webrender fenix experiment. r=snorp
Blocks: wr-73-android
No longer blocks: wr-android-mvp
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED

Re-opening until Fenix github changes have landed:
https://github.com/mozilla-mobile/fenix/pull/7487

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Component: Shield Study → Graphics: WebRender
Product: Shield → Core
Target Milestone: --- → mozilla73
Version: unspecified → 73 Branch

Comment on attachment 9118430 [details]
Bug 1605454 - Use GeckoRuntimeSettings intent extras to avoid API change for webrender fenix experiment.

Beta/Release Uplift Approval Request

  • User impact if declined: We won't be able to do a performance measurement on Fenix with/without webrender.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This code only affects geckoview and is disabled unless the product (Fenix/Fennec/...) sets the flag. There is a seperate pullrequest for enabling this branch on Fenix that will be reviewed by QA. The codepath that this enables has been extensively tested before as the code just flips a preference to disable the new rendering engine.
  • String changes made/needed:
Attachment #9118430 - Flags: approval-mozilla-beta?

Comment on attachment 9118430 [details]
Bug 1605454 - Use GeckoRuntimeSettings intent extras to avoid API change for webrender fenix experiment.

Needed for Fenix WebRender experiments. Approved for GV73.

Attachment #9118430 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Normal practice is to use tracking flags to track uplifts (with the Target Milestone tracking when a patch hit m-c). Feel free to hold off on resolving the bug until the Fenix patch has landed, though.

Target Milestone: mozilla73 → mozilla74

Control WebRender rollout for Fenix on Pixel 2/3

Fenix Beta 73

We have finished testing the Control webrender rollout for Fenix on Pixel 2/3 experiment.

QA’s recommendation: YELLOW - SHIP IT, CONDITIONALLY

Reasoning:

  • We found 1 issue that wasn’t fixed and it seems to be generated by the specific web pages visited that are not respecting the about:config option and are keeping the WebRender ON.

  • We are concerned by more of an edge case where trying to re-enable WebRender through ADB command, after it was previously disabled also through ADB, it is not possible.

What needs to be done?

  • The aforementioned issues need to be addressed or clarified in order to change the sign-off status to GREEN.

Testing Summary:

Tested Platforms:

  • Android 8.0.0 (Google Pixel 2 and 2XL)

  • Android 10.0.0 (Google Pixel 3)

Tested Firefox versions:

  • Firefox Preview Beta 73

After discussions held on #slack and emails clarifications offered for the reported issues, we have retested the scenarios in cause, and we provided a GREEN sign-off now.

Summary: [Experiment]: Generic Experiment: Control webrender rollout for Fenix → [Experiment]: Generic Experiment: Fenix Webrender performance experiment
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: