Closed Bug 1579984 Opened 5 years ago Closed 5 years ago

1-second pauses during map dragging on Google Maps due to GPU switching heuristics

Categories

(Core :: Graphics: CanvasWebGL, defect, P1)

All
macOS
defect

Tracking

()

RESOLVED FIXED
mozilla74
Tracking Status
firefox-esr68 --- unaffected
firefox72 --- wontfix
firefox73 --- wontfix
firefox74 --- fixed

People

(Reporter: mstange, Assigned: jgilbert)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

This happens on dual-GPU Macs.

Steps to reproduce:

  1. Go to https://www.google.com/maps/ and wait for it to load completely.
  2. Drag the map around a bit.
  3. At some point, dragging freezes for a second.
  4. Interact some more, and then stop dragging.
  5. Look at the map for 6 seconds.
  6. Drag it some more.
  7. Encounter another 1 second of jank.

Profile: https://perfht.ml/2ZTAAuW

Expected results:
Ideally, no jank at all. But if there needs to be a GPU switch, I think I'd prefer to have it at the very beginning and then not anymore until I close the tab.

Actual results:
Frequent back-and-forth switching between GPUs.

This is a tricky one. I think maybe there should be a permaban state. GMaps should probably be asking for low-power, honestly.

Priority: -- → P1

Google recently announced they would treat "default" power preference as "low-power", so we should just do that, and possibly ditch the (broken!) live-switching implementation for WebGL.

Assignee: nobody → jgilbert

Chrome started shipping this recently, so it's the easiest way to resolve
our dynamic-gpu switching issues for WebGL.

Pushed by jgilbert@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e12e7c32c8ca Default to low-power for WebGL powerPreference. r=lsalzman
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74

This sounds like a change we might want to let bake for a bit, but LMK if you think this is something we should uplift to Beta for Fx73.

Blocks: 1617091
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: