Closed
Bug 1396136
Opened 7 years ago
Closed 7 years ago
stylo: Fix skipped and random-if tests in layout/reftests/forms/
Categories
(Core :: CSS Parsing and Computation, enhancement, P2)
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox55 | --- | unaffected |
firefox56 | --- | wontfix |
firefox57 | --- | fixed |
People
(Reporter: cpeterson, Assigned: xidorn)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
+++ This bug was initially created as a clone of Bug #1396135 +++ layout/reftests/forms/input/number/reftest.list 39 fuzzy-if(skiaContent,2,5) needs-focus skip-if(styloVsGecko||stylo) == focus-handling.html focus-handling-ref.html Why do we skip this test in both stylo and styloVsGecko mode?
Reporter | ||
Comment 1•7 years ago
|
||
Also, why is this test random-if styloVsGecko? layout/reftests/forms/progress/reftest.list 5 random-if(styloVsGecko) == bar-pseudo-element.html bar-pseudo-element-ref.html
Summary: stylo: Fix focus-handling.html test in layout/reftests/forms/input/number/reftest.list → stylo: Fix skipped and random-if tests in layout/reftests/forms/
Updated•7 years ago
|
Priority: -- → P2
Assignee | ||
Comment 2•7 years ago
|
||
So it seems that bar-pseudo-element.html is fine, while focus-handling.html has high frequent intermittent: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7fe1bf6ccef121c287ba3099105625db1038d25b Locally, I can reproduce the issue of focus-handling.html. It seems when the failure happens, the first input element get focus a second time, in which, the focus() call doesn't trigger refocusing to the second input element. I have no idea why this would be stylo-specific. It seems to me that it happens to non-stylo as well, in a very low frequency (see bug 1284330 and bug 1126084), I suspect it is a timing issue which stylo happens to make more frequent. We can probably just workaround the test somehow.
Assignee | ||
Comment 3•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=0ef48977bd34ac4b6d3accfed02ae3578eb5e769
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → xidorn+moz
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8904435 [details] Bug 1396136 part 1 - Make forms/input/number/focus-handling.html more robust. https://reviewboard.mozilla.org/r/176284/#review181502
Attachment #8904435 -
Flags: review?(jwatt) → review+
Assignee | ||
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8904436 [details] Bug 1396136 part 2 - Re-enable reftests in forms. https://reviewboard.mozilla.org/r/176286/#review181526
Attachment #8904436 -
Flags: review+
Pushed by xquan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f6a9c476a2db part 1 - Make forms/input/number/focus-handling.html more robust. r=jwatt https://hg.mozilla.org/integration/autoland/rev/ffb160236c24 part 2 - Re-enable reftests in forms. r=xidorn
Comment 9•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f6a9c476a2db https://hg.mozilla.org/mozilla-central/rev/ffb160236c24
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in
before you can comment on or make changes to this bug.
Description
•