Closed
Bug 1369097
Opened 7 years ago
Closed 7 years ago
After terminating a GPU process youtube videos remain in an infinite loading state
Categories
(Core :: Graphics: Layers, defect)
Tracking
()
RESOLVED
FIXED
mozilla56
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox54 | --- | unaffected |
firefox55 | --- | disabled |
firefox56 | + | fixed |
People
(Reporter: ccomorasu, Assigned: domfarolino)
References
Details
(Keywords: regression, Whiteboard: [gfx-noted])
Attachments
(1 file)
660.60 KB,
image/png
|
Details |
[Note]: - please check that "layers.gpu-process.max_restarts" integer preference is set to "3" (from about:config) - please check that the "Use recommended performance settings" checkbox below the "Performance" is checked (from about:preferences -> General) [Affected versions]: Fx 55.0a1 [Affected platforms]: Windows 8.1 x32 Windows 7 x64 [Steps to reproduce]: 1. Launch Firefox. 2. Go to a youtube video. 3. Terminate the CPU process from about:support . [Expected result]: A short glitch, afterwords the video plays smoothly. [Actual result]: The video remains in an infinite loading state. [Regression range]: Will return with the regression range as soon as possible. [Additional notes]: A gif with the issue: http://imgur.com/a/CQR6A .
Updated•7 years ago
|
Summary: After terminating a CPU process youtube videos remain in an infinite loading state → After terminating a GPU process youtube videos remain in an infinite loading state
It'd be also good to know if pause/play or reloading the page fixes the problem.
Keywords: regression
Also, can you (restart not required) set layers.gpu-process.max_restarts to 0 and repeat the workflow and see if the problems are gone or different?
Flags: needinfo?(cristian.comorasu)
Updated•7 years ago
|
Whiteboard: [gfx-noted]
Reporter | ||
Comment 3•7 years ago
|
||
I tested with the pref "layers.gpu-process.max_restarts" to "0" using Fx 55.0a1, build ID: 20170605030204, on Windows 10 x64, Windows 8.1 x32 and Windows 7 x64. This issue is not reproducible under these circumstances. Cheers!
Flags: needinfo?(cristian.comorasu)
David, let's reset this preference back to the default (I never know if that's 0 or 1), so that we never restart to accelerated beyond nightly. It'll give us a chance to sort out the issues, but it'll stay on nightly until we're sure we're good and sort out the telemetry/etc. we need for making the right call. [Tracking Requested - why for this release]: This should block 55 - we don't want a preference where we have some negative anecdotal, and no telemetry/crash evidence to tell us otherwise.
Sounds good. I'll file a new bug so we can track this issue separately.
Flags: needinfo?(dvander)
Depends on: 1370621
This functionality will be off in 55, so no need to track it for 55.
Pushed by danderson@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b6314a490ce8 Only restart the GPU process multiple times on Nightly. (bug 1369097, r=milan)
Comment 9•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b6314a490ce8
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Updated•7 years ago
|
Updated•7 years ago
|
status-firefox54:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Updated•7 years ago
|
Flags: qe-verify+
Comment 11•7 years ago
|
||
Hi, This issue doesn't seem fixed to me. The actual result is different now but the steps for getting there are the same. If the GPU process is terminated while a YouTube video is playing an error message appears and not even a refresh can make it work. A browser restart is needed for watching the clip again. This only happens on 32 bit builds of Firefox. Tested on: - Windows 7 x86 & x64 - Windows 8.1 x64 - Windows 10 x64 Please see the attachment for the error.
Comment 12•7 years ago
|
||
Dominic, did you get the chance to look at Comment 11?
Flags: needinfo?(domfarolino)
Dominic's internship ended in August, taking needinfo
Flags: needinfo?(domfarolino) → needinfo?(dvander)
Flags: needinfo?(dvander)
You need to log in
before you can comment on or make changes to this bug.
Description
•