Open Bug 1410248 Opened 5 years ago Updated 5 years ago

speech-api wpt disabled on import for intermittent crashes

Categories

(Core :: Web Speech, enhancement)

enhancement
Not set
normal

Tracking

()

REOPENED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: jgraham, Unassigned)

Details

No description provided.
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/83dfc2b82faa
Disable speech-api test for intermittent crashes, a=testonly
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/befce45a7a46
Disable speech-api test for intermittent crashes, a=testonly
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/66cafcbfa5ed
Disable speech-api test for intermittent crashes, a=testonly CLOSED TREE
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5e68550b3796
Disable speech-api test for intermittent crashes, a=testonly
https://hg.mozilla.org/mozilla-central/rev/5e68550b3796
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/34cbb525ecce
Disable all speech-api web platform tests until they can run reliably. r=me
James, I had to disable the rest for ongoing failures like the log below.
https://treeherder.mozilla.org/logviewer.html#?job_id=139491747&repo=mozilla-inbound

Rather than play whack-a-mole with a 30+% failure rate, I took the route of fastest to green.
Summary: /speech-api/SpeechSynthesis-speak-twice.html wpt disabled on import for intermittent crashes → speech-api wpt disabled on import for intermittent crashes
SpeechSynthesis-speak-twice.html is added by bug 1372325 and https://github.com/w3c/speech-api/issues/7.  Maybe, wpt should use mock speech synthesis prefs like mochitest.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
https://hg.mozilla.org/mozilla-central/rev/34cbb525ecce
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
RyanVM - Thanks for disabling more tests rather than backing out the whole import. Much appreciated.

m_kato - Probably it should; if you want to do that they can be added to a __dir__.ini file in testing/web-platform/meta/speech-api/ like

prefs: [pref1:value, pref2:value]

However I would expect that the crash needs to be fixed independent of whether those prefs ought to be fixed.
Status: RESOLVED → REOPENED
Flags: needinfo?(james)
Resolution: FIXED → ---
You need to log in before you can comment on or make changes to this bug.