New issue
Advanced search Search tips

Issue 836783 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

fast/peerconnection/RTCRtpSender-setParameters.html is flaky

Project Member Reported by hbos@chromium.org, Apr 25 2018

Issue description

fast/peerconnection/RTCRtpSender-setParameters.html (not to be confused with external/wpt/webrtc/RTCRtpSender-setParameters) is flakily timing out, for example:

https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win7%20Tests%20%28dbg%29%281%29/68666

https://isolateserver.appspot.com/browse?namespace=default-gzip&hash=6cac11608d49b367648ef58854825f53eaf04ed3
RTCRtpSender-setParameters-stderr.txt does not seem to provide any interesting information,

https://chromium-swarm.appspot.com/task?id=3d15201da23e6d10&refresh=10&show_raw=1
reveals that
03:42:37.105 7236 worker/5 fast/peerconnection/RTCRtpSender-setParameters.html failed:
03:42:37.105 7236 worker/5  test timed out

Please take a look, orphis.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 25 2018

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

commit 032c2ca449090c1475e67f7b46d85d8f7996cf85
Author: Henrik Boström <hbos@chromium.org>
Date: Wed Apr 25 12:24:54 2018

Allow fast/peerconnection/RTCRtpSender-setParameters.html to timeout.

The test is flaky. See https://crbug.com/836783.

TBR=orphis@chromium.org

No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 836783
Change-Id: I854823be77f95bed0d562f5f2968064e7342352e
Reviewed-on: https://chromium-review.googlesource.com/1027871
Reviewed-by: Henrik Boström <hbos@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553518}
[modify] https://crrev.com/032c2ca449090c1475e67f7b46d85d8f7996cf85/third_party/WebKit/LayoutTests/TestExpectations

Comment 2 by hbos@chromium.org, Apr 25 2018

Labels: -Sheriff-Chromium
Now that TestExpectations are [ Pass Timeout ], removing the sheriff label.

Sign in to add a comment