New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 756037 link

Starred by 4 users

Issue metadata

Status: WontFix
Owner:
OOO Dec 22 - Jan 8
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug-Regression



Sign in to add a comment

WebRtcAudioBrowserTest.* flaky on Android Tests (dbg) (L Nexus9) on chromium.webrtc.fyi

Project Member Reported by olka@chromium.org, Aug 16 2017

Issue description

Comment 1 by olka@chromium.org, Aug 16 2017

a lot of [INFO:CONSOLE(23)] "Still waiting for the fake audio signal.", source: http://127.0.0.1:38697/media/webrtc_test_audio.js (23)

Comment 2 by olka@chromium.org, Aug 16 2017

Issue 754714 has been merged into this issue.

Comment 3 by olka@chromium.org, Aug 16 2017

nothing suspicious on WebRTC blame list.
chrome revision range is 8c8ab3db65a753303b542df2e1db2abe5b4c1ef9 (build 24154) .. eb35bbd49aacbaaa015c9cc63d9c6ae16a26d64c (build 24156)

Comment 4 by olka@chromium.org, Aug 17 2017

The only media-related changes I see in Chromium range:

[1]https://chromium-review.googlesource.com/c/601057
Plumb output color space to video decoder

[2]https://chromium-review.googlesource.com/c/602596
Defer AOC::OnError() to avoid spurious device change errors.

[3]https://chromium-review.googlesource.com/c/611002
[media] Enable ChunkDemuxer to signal OnProgress events.


[3] is suspicious, but Issue 754043 it's associate with has some complicated history, and it's already been merged to M61.

Comment 5 by olka@chromium.org, Aug 17 2017

Cc: dalecur...@chromium.org
Owner: sande...@chromium.org
Status: Assigned (was: Untriaged)
sandersd@ could you PTAL?

Comment 6 by olka@chromium.org, Aug 17 2017

Some WebRtcBrowserTest.* also got flaky, for example:

WebRtcBrowserTest.NegotiateOfferWithBLine
https://uberchromegw.corp.google.com/i/chromium.webrtc.fyi/builders/Android%20Tests%20%28dbg%29%20%28L%20Nexus9%29/builds/24318

WebRtcBrowserTest.NoCrashWhenConnectChromiumSinkToRemoteTrack
https://uberchromegw.corp.google.com/i/chromium.webrtc.fyi/builders/Android%20Tests%20%28dbg%29%20%28L%20Nexus9%29/builds/24317


with messages like "Still waiting for video to satisfy function isVideoPlaying"
(https://uberchromegw.corp.google.com/i/chromium.webrtc.fyi/builders/Android%20Tests%20%28dbg%29%20%28L%20Nexus9%29/builds/24318/steps/content_browsertests/logs/stdio)


So I'm not sure if conclusion on Issue 754714 is correct.

Cc: sande...@chromium.org
Owner: olka@chromium.org
Shouldn't be (3) since the WebRtc tests don't generally use WMPI, but WMP_MS and additionally don't use Media Source Extensions which that patch affects.

Also given that there are at least 4 good builds among the bad  I think the starting revision should be pulled back to 1ac8f7d910cbf05080a406d51be987affd8b4870 24150 to be sure.


Comment 8 by olka@chromium.org, Aug 18 2017

I'm looking at chromium range https://chromium.googlesource.com/chromium/src/+log/9a7e43c26d697a994c50c82fa5dfb81c24f44351..eb35bbd49aacbaaa015c9cc63d9c6ae16a26d64c?pretty=fuller
(this is starting 24149)

And have no good candidates to blame.
Is it possible to run the bot on an earlier green revision? If the tests pass reliably on earlier revisions, it would rule out hardware issues.

Comment 10 by olka@chromium.org, Aug 18 2017

Cc: ehmaldonado@chromium.org

Comment 11 by olka@chromium.org, Aug 21 2017

Cc: kjellander@chromium.org sakal@chromium.org

Comment 12 by olka@chromium.org, Aug 23 2017

Cc: olka@chromium.org
Owner: sakal@chromium.org
sakal@ could you PTAL?

kjellander@ can we do an experiment like #9? Can we also run the tests on another bot? This Nexus 9 seems to be the only device running them.
These bots are running with Chromium src always at HEAD and src/third_party/webrtc at the revision that the poller polled at https://chromium.googlesource.com/external/webrtc/trunk/webrtc

You could probably force a build passing a revision from https://chromium.googlesource.com/external/webrtc/trunk/webrtc in the text box by the Force button at https://uberchromegw.corp.google.com/i/chromium.webrtc.fyi/builders/Android%20Builder%20ARM64%20%28dbg%29 and it should build Chromium src at current HEAD with src/third_party/webrtc at the revision you pass.
I'm not sure what that will provide though.

Regarding the test - the Nexus 9 is running the same test as the other Android testers in this waterfall: out/Debug/bin/run_content_browsertests --gtest_filter=WebRtc*

Comment 14 by sakal@chromium.org, Aug 28 2017

Cc: maxmorin@chromium.org
Owner: ----
Status: Available (was: Assigned)
Owner: olka@chromium.org
Status: Assigned (was: Available)
[Re-triage] Since this is a P1 regression, it should be assigned to someone. Back to olka@ who was the owner before sakal@. Maybe it could be a P2 though since it's Android?

Comment 16 by sakal@chromium.org, Sep 22 2017

To me it looks like this fixed now?

Comment 17 by olka@chromium.org, Oct 5 2017

Status: WontFix (was: Assigned)
The bot is all green now.

Sign in to add a comment