New issue
Advanced search Search tips

Issue 821717 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 822231
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug
Flaky-Test: MediaRouterIntegrationOneUABrowserTest.ReconnectSession

Blocking:
issue 698796



Sign in to add a comment

MediaRouterIntegrationOneUABrowserTest.ReconnectSession is Flaky

Project Member Reported by Findit, Mar 14 2018

Issue description

Blocking: 698796
Components: Internals>Cast>Providers
Labels: -Sheriff-Chromium
Owner: mfo...@chromium.org
Status: Assigned (was: Available)
Reverting: https://chromium-review.googlesource.com/c/chromium/src/+/962942

Labels: -Pri-1 Pri-2
 Issue 821716  has been merged into this issue.
 Issue 821714  has been merged into this issue.
 Issue 821712  has been merged into this issue.
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 14 2018

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

commit 8cb5314ca50681891031d7ddd4d6e54b8bed04ae
Author: mark a. foltz <mfoltz@chromium.org>
Date: Wed Mar 14 23:58:19 2018

Reland "[Media Router] Converts integration tests to run on Chromium waterfall."

This is a reland of aa57fe6dc96b40a58acc1ed6c268f47a881b590e

Original change's description:
> [Media Router] Converts integration tests to run on Chromium waterfall.
>
> This updates the Media Router integration tests so they run in the
> Chromium waterfall with the open source Media Router component
> extension and its mock (test) Media Route Provider.
>
> Additional changes:
>
> - Removes extension from top level BUILD.gn as it is only used for
>   browser tests at the moment.
> - Splits the targets for Media Router browser tests into integration tests
>   (part of main browser_tests suite), e2e tests (which require the
>   external component extension), perf tests, and shared support code.
> - Renames the "test" extension to the "telemetry" extension used for perf
>   tests to avoid confusion.
> - Updates README files
> - One test is not compatible with the Chromium component and need
>   to be fixed.  Bugs  818767  filed for follow-up.
>
> Bug:  698796 
> Change-Id: I6cc9302ffd72de95630ce33ca3617b3b73b9e722
> Reviewed-on: https://chromium-review.googlesource.com/917214
> Commit-Queue: mark a. foltz <mfoltz@chromium.org>
> Reviewed-by: Scott Violet <sky@chromium.org>
> Reviewed-by: Derek Cheng <imcheng@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#543005}

TBR=cliffordcheng, imcheng, jrw, sky

Bug:  698796 ,822029,  821717 
Change-Id: Ie5c2fdc9a838dec2906a73f7827fa6ceafdcce1b
Reviewed-on: https://chromium-review.googlesource.com/963064
Commit-Queue: mark a. foltz <mfoltz@chromium.org>
Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543242}
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/BUILD.gn
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/chrome/test/BUILD.gn
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/chrome/test/media_router/BUILD.gn
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/chrome/test/media_router/README.md
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/chrome/test/media_router/media_router_base_browsertest.cc
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/chrome/test/media_router/media_router_integration_browsertest.cc
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/chrome/test/media_router/media_router_integration_ui_browsertest.cc
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/chrome/test/media_router/media_router_one_ua_integration_browsertest.cc
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/chrome/test/media_router/telemetry/README.md
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/chrome/test/media_router/telemetry/benchmarks/media_router_benchmark.py
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/chrome/test/media_router/telemetry/extension/README.md
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/testing/buildbot/filters/mash.browser_tests.filter
[modify] https://crrev.com/8cb5314ca50681891031d7ddd4d6e54b8bed04ae/testing/buildbot/gn_isolate_map.pyl

Comment 7 by mfo...@chromium.org, Mar 15 2018

Mergedinto: 822231
Status: Duplicate (was: Assigned)

Sign in to add a comment