New issue
Advanced search Search tips

Issue 855873 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 23
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

"fast/speech/scripted/start-multiple.html" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Jun 23 2018

Issue description

"fast/speech/scripted/start-multiple.html" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 4 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyMwsSBUZsYWtlIihmYXN0L3NwZWVjaC9zY3JpcHRlZC9zdGFydC1tdWx0aXBsZS5odG1sDA.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 

Comment 1 by shend@chromium.org, Jun 25 2018

Owner: adithyas@chromium.org
Status: Assigned (was: Untriaged)
Test crashes due to check failure:

17:36:00.270 45784   [47830:33283:0622/173600.105545:FATAL:speech_recognition_manager_impl.cc(476)] Check failed: primary_session_id_ == session_id (4 vs. 2)

Only recent change in speech is https://chromium.googlesource.com/chromium/src/+/f5b33428689a900261141871bf5346acf7619640 . adithyas@, can you take a look please? Thank you.
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 25 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/400d32d2e96536797d84ce69be92a0ed33e8ef8a

commit 400d32d2e96536797d84ce69be92a0ed33e8ef8a
Author: Darren Shen <shend@chromium.org>
Date: Mon Jun 25 05:30:39 2018

Disable fast/speech/scripted/start-multiple.html.

Flaky on mac and win7.

TBR=adithyas@chromium.org

Bug:  855873 
Change-Id: I0d55390244a7e96e478a01bd5be254d958a41147
Reviewed-on: https://chromium-review.googlesource.com/1113082
Reviewed-by: Darren Shen <shend@chromium.org>
Commit-Queue: Darren Shen <shend@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569969}
[modify] https://crrev.com/400d32d2e96536797d84ce69be92a0ed33e8ef8a/third_party/WebKit/LayoutTests/TestExpectations

Comment 3 by shend@chromium.org, Jun 25 2018

Labels: -Sheriff-Chromium
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 27 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7e374d6cf106402da7a945f848206f2552a80234

commit 7e374d6cf106402da7a945f848206f2552a80234
Author: Adithya Srinivasan <adithyas@chromium.org>
Date: Wed Jun 27 21:42:47 2018

Reenable fast/speech/script/start-multiple.html

The test was rewritten in https://crrev.com/c/1111910 and now uses a mock
speech recognizer written in js. The previous mock recognizer was removed
and the test was using the actual speech recognition service.

Bug:  855873 
Change-Id: I89a902092ee26c16f3624bd359c0cbf55b74e1d8
Tbr: shend@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/1117620
Reviewed-by: Adithya Srinivasan <adithyas@chromium.org>
Commit-Queue: Adithya Srinivasan <adithyas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570908}
[modify] https://crrev.com/7e374d6cf106402da7a945f848206f2552a80234/third_party/WebKit/LayoutTests/TestExpectations

Status: Fixed (was: Assigned)

Sign in to add a comment