Closed Bug 1409243 Opened 5 years ago Closed 5 years ago
Enable dotted and dashed border support for wr in gecko side
59 bytes, text/x-review-board-request
According to Bug 1396491 comment 8, there will be some failure in the crashtest while dotted and dashed border are enabled. We should find the root cause and fix it.  https://treeherder.mozilla.org/#/jobs?repo=try&revision=9cb4ecee842b5b9d0948f0d89cd5b72be52573fd
The main reason of the timeout failure in the crashtest is the missing DidComposite messages. The content process didn't received DidComposite messages after the animaton was started so the system always made the early return in nsRefreshDriver::Tick which caused the timeout.  http://searchfox.org/mozilla-central/rev/d0448c970093f94bd986a21d3a9e8c366b316eb4/layout/base/nsRefreshDriver.cpp#1827
So the real problem happened in large-border-radius-dashed.html. When we try to render a border with really large border-radius and styled-border, it takes lots of time to render it, Webrender draws nothing since that test case and the test framework finally gets failure in 1319072-1.html due to the timeout since there is a reftest-wait in it. In gecko, there is a mechanism to handle this situation which is draw the solid border instead. But that solution is a little weird to me. The following is my suggestion about this bug: (1) Implement the same mechanism(like ) in Webrender. (2) Mark fails-if to these crashtest and enable the styled-border in this bug since that's a rare case and open a following bug to discuss if there are other solutions to this.  http://searchfox.org/mozilla-central/rev/8a6a6bef7c54425970aa4fb039cc6463a19c0b7f/layout/generic/crashtests/large-border-radius-dashed.html  http://searchfox.org/mozilla-central/rev/8a6a6bef7c54425970aa4fb039cc6463a19c0b7f/layout/style/crashtests/1319072-1.html  https://bugzilla.mozilla.org/show_bug.cgi?id=382721
5 years ago
I choose option 2.
Comment on attachment 8921757 [details] Bug 1409243 - Enable dotted and dashed border support for webrender; https://reviewboard.mozilla.org/r/192772/#review200856
Attachment #8921757 - Flags: review?(ethlin) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/4ecf1d9c1975 Enable dotted and dashed border support for webrender; r=ethlin
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.