New issue
Advanced search Search tips

Issue 657964 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

MediaRouterElementsBrowserTest.MediaRouterContainerFilter is flaky

Project Member Reported by ma...@chromium.org, Oct 20 2016

Issue description

Components: Internals>Media
Labels: Build-Tests
Note that this test is consistently very slow, near the timeout limit, on Windows and Linux debug, and relatively slow on Mac debug as well.

Here's a dashboard link without the unrelated test:

https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=browser_tests&tests=MediaRouterElementsBrowserTest.MediaRouterContainerFilter
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 21 2016

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

commit 2ed5864143ca2b00f68606ab7a3efb781ce379e5
Author: pkasting <pkasting@chromium.org>
Date: Fri Oct 21 03:49:21 2016

Disable test that is flakily timing out on debug.

BUG= 657964 
TEST=none
TBR=takumif, dbeam

Review-Url: https://chromiumcodereview.appspot.com/2436223003
Cr-Commit-Position: refs/heads/master@{#426720}

[modify] https://crrev.com/2ed5864143ca2b00f68606ab7a3efb781ce379e5/chrome/test/data/webui/media_router/media_router_elements_browsertest.js

Cc: taku...@chromium.org
Owner: btolsch@chromium.org
Brandon, would you mind taking a look at this? It seems like we have too many JS tests grouped together as one test case. Maybe we should split them up.
Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 10 2017

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

commit 40f304cc99d6d6241fcbdba2f294db2b7d46be77
Author: btolsch <btolsch@chromium.org>
Date: Fri Nov 10 21:39:47 2017

Split search and filter MediaRouterElementsBrowserTests into 2 parts

Search and filter tests have been flaky on Windows bots because they
time out.  In the past, the fix for this was to split the tests up into
more cases so this change further splits filter and search tests.

Bug:  657964 ,  760288 
Change-Id: I01bd179aed3b7181a50ef0627320ccc643f8437a
Reviewed-on: https://chromium-review.googlesource.com/761260
Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Brandon Tolsch <btolsch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515691}
[modify] https://crrev.com/40f304cc99d6d6241fcbdba2f294db2b7d46be77/chrome/test/data/webui/media_router/media_router_container_filter_tests.js
[modify] https://crrev.com/40f304cc99d6d6241fcbdba2f294db2b7d46be77/chrome/test/data/webui/media_router/media_router_container_search_tests.js
[modify] https://crrev.com/40f304cc99d6d6241fcbdba2f294db2b7d46be77/chrome/test/data/webui/media_router/media_router_elements_browsertest.js

Status: Fixed (was: Started)
Bots are looking good after the test split.

Sign in to add a comment