Closed
Bug 1418035
Opened 7 years ago
Closed 7 years ago
regression: Sometimes the page stops updating (related to canvas?) until you hover over the chrome
Categories
(Core :: Graphics: WebRender, defect, P1)
Core
Graphics: WebRender
Tracking
()
VERIFIED
FIXED
mozilla59
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox57 | --- | unaffected |
firefox58 | --- | unaffected |
firefox59 | --- | verified |
People
(Reporter: mstange, Assigned: sotaro)
References
Details
(Keywords: regression, Whiteboard: [wr-mvp])
Attachments
(1 file)
(deleted),
patch
|
nical
:
review+
|
Details | Diff | Splinter Review |
Steps to reproduce:
1. Enable WebRender.
2. Load https://perfht.ml/2mCyTQn
3. Wait for everything to stop loading and make sure there aren't any other animations happening in your browser window.
4. Move your mouse over the blue graphs in the top part of the page and move it around in a left-to-right alternating motion, for many seconds.
Expected results:
The vertical line should always follow the mouse and never pause.
Actual results:
After a few seconds, the page freezes.
Usually it becomes unstuck after a few more seconds.
If you move your mouse over the tabs in the UI, the page also becomes unstuck.
I've noticed the same thing on Google Sheets where scrolling would appear to stop working.
Updated•7 years ago
|
QA Whiteboard: [wr-mvp] [triage]
Updated•7 years ago
|
QA Whiteboard: [wr-mvp] [triage]
Whiteboard: [wr-mvp] [triage]
Comment 1•7 years ago
|
||
Wait until everything has loaded.
Move your mouse on the blue area from the left to the right,
Then move it up in into the white area (or on "0.Xs" above it) and to the left. The grey vertical line freezes. You can't expand something from the tree on the bottom.
Verify this by moving the mouse over the tab and trying it again.
Sometimes I had to move my mouse over the tab at first to unfreeze the vertical grey line.
mozregression --good 2017-11-03 --bad 2017-11-16 --pref "layers.acceleration.force-enabled:true" "gfx.webrender.enabled:true" "gfx.webrender.blob-images:true"
> 14:59.23 INFO: Last good revision: e281e797c85d6c24961f504c6f27e364210bb0a7
> 14:59.23 INFO: First bad revision: 4cc53112a8d53d06d6831762aebac41fa2dcf744
> 14:59.23 INFO: Pushlog:
> https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=e281e797c85d6c24961f504c6f27e364210bb0a7&tochange=4cc53112a8d53d06d6831762aebac41fa2dcf744
> 4cc53112a8d5 sotaro — Bug 1412249 - Fix DidComposite timing for EmptyTransaction r=nical
Second test:
mozregression --good 2017-11-14 --bad 2017-11-16 --pref "layers.acceleration.force-enabled:true" "gfx.webrender.enabled:true" "gfx.webrender.blob-images:true"
> 7:56.66 INFO: Last good revision: e281e797c85d6c24961f504c6f27e364210bb0a7
> 7:56.66 INFO: First bad revision: 4cc53112a8d53d06d6831762aebac41fa2dcf744
> 7:56.66 INFO: Pushlog:
> https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=e281e797c85d6c24961f504c6f27e364210bb0a7&tochange=4cc53112a8d53d06d6831762aebac41fa2dcf744
> 4cc53112a8d5 sotaro — Bug 1412249 - Fix DidComposite timing for EmptyTransaction r=nical
Blocks: 1412249
Has Regression Range: --- → yes
Has STR: --- → yes
status-firefox57:
--- → unaffected
status-firefox58:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Keywords: regressionwindow-wanted
Summary: Sometimes the page stops updating (related to canvas?) until you hover over the chrome → regression: Sometimes the page stops updating (related to canvas?) until you hover over the chrome
Reporter | ||
Comment 2•7 years ago
|
||
Thanks!
Sotaro, can you take a look?
Flags: needinfo?(sotaro.ikeda.g)
Assignee | ||
Comment 3•7 years ago
|
||
Thanks! I take a look.
Assignee: nobody → sotaro.ikeda.g
Flags: needinfo?(sotaro.ikeda.g)
Updated•7 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [wr-mvp] [triage] → [wr-mvp]
Assignee | ||
Comment 4•7 years ago
|
||
I could reproduce the problem.
Assignee | ||
Comment 5•7 years ago
|
||
There were 2 problems about deciding sending DidComposite.
WebRenderBridgeParent::RecvEmptyTransaction() sends back DidComposite if WebRenderBridgeParent does not have pending DidComposite requests. Its check was wrong. It check the pending request after adding current DidComposite request.
WebRenderBridgeParent::FlushTransactionIdsForEpoch did not pop multiple pending TransactionIds.
Assignee | ||
Comment 6•7 years ago
|
||
attachment 8929385 [details] [diff] [review] addressed the problem for me.
Assignee | ||
Updated•7 years ago
|
Attachment #8929385 -
Attachment description: patch - Fix timing of sending DidComposite → patch - Fix worng decision of sending DidComposite.
Assignee | ||
Comment 7•7 years ago
|
||
Assignee | ||
Updated•7 years ago
|
Attachment #8929385 -
Flags: review?(nical.bugzilla)
Assignee | ||
Comment 8•7 years ago
|
||
(In reply to Sotaro Ikeda [:sotaro] from comment #5)
> Created attachment 8929385 [details] [diff] [review]
> patch - Fix worng decision of sending DidComposite.
> WebRenderBridgeParent::FlushTransactionIdsForEpoch did not pop multiple
> pending TransactionIds.
WebRenderBridgeParent::FlushTransactionIdsForEpoch did not pop multiple pending TransactionIds that have same epoch.
Updated•7 years ago
|
Attachment #8929385 -
Flags: review?(nical.bugzilla) → review+
Updated•7 years ago
|
Blocks: stage-wr-nightly
Pushed by sikeda@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c8cf6aa1633a
Fix worng decision of sending DidComposite r=nical
Comment 10•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Updated•7 years ago
|
Flags: qe-verify+
Comment 11•7 years ago
|
||
I reproduced this bug following the steps from comment 0 and comment 1 on 59.0a1 (2017-11-16) under macOS 10.13.
I retested it on the latest Nightly (2018-01-30) and 59.0b5 under macOs 10.13, Windows 10 x64 and Ubuntu 16.04 x64 using the prefs described in comment 2. The "gfx.webrender.blob-images:true" is no longer a boolean value, but an integer, so I tested this also, on an earlier beta build that contain the fix (58.0b5), when the pref in cause was still a boolean, just to make sure fix was successfully implemented. My concern is if the recently changed pref ("gfx.webrender.blob-images:true" is now integer) will affect this fix in any way.
Any thoughts about this?
Flags: needinfo?(sotaro.ikeda.g)
Assignee | ||
Comment 12•7 years ago
|
||
Since Bug 1425260 fix, pref "gfx.webrender.all" to true is a recommended way to enable WebRender. Could you check with it on latest version? Thanks.
Flags: needinfo?(sotaro.ikeda.g)
Comment 13•7 years ago
|
||
I tested this issue based on the above comment, by setting the gfx.webrender.all on true and then retested by setting also the layers.acceleration.force-enabled on true . I obtained the following results:
* macOS 10.13
- 59.0b6 - Not reproducible
- 60.0a1 - I reproduced one time the crash filled in bug 1434630 (only after I set the gfx.webrender.all on true and also the layers.acceleration.force-enabled on true), but all the other attempts of verifying the issue on https://perfht.ml/2mCyTQn was successfully done, this is no longer reproducible.
* Windows 10 x64
- 59.0b6 - Not reproducible
- 60.0a1 - Not reproducible
- Note: On older Nightly versions (e.g.2017-02-02) I encounter the crashes mentioned in bug 1435200 and bug 1433646 (only on one station - out of three I tested on). This is not the case anymore on the latest Nightly (2017-02-05).
On the same station, everytime I close the Firefox Nightly window (while gfx.webrender.all is set on true), a popup is displayed with the error message ""Firefox Nightly has stopped working. A problem caused the program to stop working correctly"". Is this related and/or concerning?
* Ubuntu 16.04 x32
- 59.0b6 - Not reproducible
- 60.0a1 - Blocked by bug 1434961
* Ubuntu 16.04 x64
- 59.0b6 - The vertical line follows smoothly the mouse cursor, no delays or page freezing, but sometimes glitches are triggered, not only on this particular site (all the opened windows/tabs are affected) (only on one station - out of two I tested on) - see the screencast https://goo.gl/MEMcxA
- 60.0a1 - The same issue as described on beta is applicable here, but the glitches are more noticeable.
Should these above mentioned potential issues be assessed separately?
Flags: needinfo?(sotaro.ikeda.g)
Assignee | ||
Comment 14•7 years ago
|
||
(In reply to Iulia Cristescu, QA [:JuliaC] from comment #13)
> I tested this issue based on the above comment, by setting the
> gfx.webrender.all on true and then retested by setting also the
> layers.acceleration.force-enabled on true . I obtained the following results:
> * macOS 10.13
> - 59.0b6 - Not reproducible
> - 60.0a1 - I reproduced one time the crash filled in bug 1434630 (only
> after I set the gfx.webrender.all on true and also the
> layers.acceleration.force-enabled on true), but all the other attempts of
> verifying the issue on https://perfht.ml/2mCyTQn was successfully done, this
> is no longer reproducible.
> * Windows 10 x64
> - 59.0b6 - Not reproducible
> - 60.0a1 - Not reproducible
> - Note: On older Nightly versions (e.g.2017-02-02) I encounter the crashes
> mentioned in bug 1435200 and bug 1433646 (only on one station - out of three
> I tested on). This is not the case anymore on the latest Nightly
> (2017-02-05).
Thanks for testing! It seems that the problem of this bug is addressed.
> On the same station, everytime I close the Firefox Nightly window (while
> gfx.webrender.all is set on true), a popup is displayed with the error
> message ""Firefox Nightly has stopped working. A problem caused the program
> to stop working correctly"". Is this related and/or concerning?
Does the crash still happens with latest nightly? It seems like a different problem. Did you get a crash report of the crash?
> * Ubuntu 16.04 x32
> - 59.0b6 - Not reproducible
> - 60.0a1 - Blocked by bug 1434961
> * Ubuntu 16.04 x64
> - 59.0b6 - The vertical line follows smoothly the mouse cursor, no delays
> or page freezing, but sometimes glitches are triggered, not only on this
> particular site (all the opened windows/tabs are affected) (only on one
> station - out of two I tested on) - see the screencast https://goo.gl/MEMcxA
> - 60.0a1 - The same issue as described on beta is applicable here, but the
> glitches are more noticeable.
> Should these above mentioned potential issues be assessed separately?
Yea, they needs to be assessed separately. They seems like different problems.
Flags: needinfo?(sotaro.ikeda.g)
Comment 15•7 years ago
|
||
(In reply to Sotaro Ikeda [:sotaro] from comment #14)
> > On the same station, everytime I close the Firefox Nightly window (while
> > gfx.webrender.all is set on true), a popup is displayed with the error
> > message ""Firefox Nightly has stopped working. A problem caused the program
> > to stop working correctly"". Is this related and/or concerning?
>
> Does the crash still happens with latest nightly? It seems like a different
> problem. Did you get a crash report of the crash?
>
There is no crash encountered here, but a error message instead (see the screenshot https://drive.google.com/file/d/1d0LZf49pQPOqQQaDH4bVmfVZV5Sq85OC/view?usp=sharing). When clicking on the "Close the program" button, nothing happens.
Flags: needinfo?(sotaro.ikeda.g)
Assignee | ||
Comment 16•7 years ago
|
||
(In reply to Iulia Cristescu, QA [:JuliaC] from comment #15)
> (In reply to Sotaro Ikeda [:sotaro] from comment #14)
> There is no crash encountered here, but a error message instead (see the
> screenshot
> https://drive.google.com/file/d/1d0LZf49pQPOqQQaDH4bVmfVZV5Sq85OC/
> view?usp=sharing). When clicking on the "Close the program" button, nothing
> happens.
Thanks. I did not see the problem on my Win10 PCs. Can you crate a bug for it and paste about:support to there?
Flags: needinfo?(sotaro.ikeda.g)
Comment 17•7 years ago
|
||
(In reply to Sotaro Ikeda [:sotaro] from comment #16)
> (In reply to Iulia Cristescu, QA [:JuliaC] from comment #15)
> > (In reply to Sotaro Ikeda [:sotaro] from comment #14)
> > There is no crash encountered here, but a error message instead (see the
> > screenshot
> > https://drive.google.com/file/d/1d0LZf49pQPOqQQaDH4bVmfVZV5Sq85OC/
> > view?usp=sharing). When clicking on the "Close the program" button, nothing
> > happens.
>
> Thanks. I did not see the problem on my Win10 PCs. Can you crate a bug for
> it and paste about:support to there?
Filed bug 1437442.
Also, based om the previous comments and the fact that the above mentioned encountered issues are covered by other bugs, I will mark this as verified fixed.
You need to log in
before you can comment on or make changes to this bug.
Description
•