Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(27)

Issue 1587803002: [Media Router] Handle route swapping in the WebUI. (Closed)

Created:
4 years, 11 months ago by apacible
Modified:
4 years, 11 months ago
Reviewers:
vadimgo, imcheng
CC:
chromium-reviews, media-router+watch_chromium.org, arv+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Media Router] Handle route swapping in the WebUI. Handle the case when the display route for a device the user is currently viewing controls for is updated to a different route. BUG=577335 Committed: https://crrev.com/f89b9c9425b918c35e342308802f17c77edb2f38 Cr-Commit-Position: refs/heads/master@{#369336}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -3 lines) Patch
M chrome/browser/resources/media_router/elements/media_router_container/media_router_container.js View 1 chunk +11 lines, -3 lines 0 comments Download

Messages

Total messages: 18 (10 generated)
apacible
PTAL, thanks!
4 years, 11 months ago (2016-01-13 21:43:30 UTC) #5
vadimgo
lgtm
4 years, 11 months ago (2016-01-13 21:48:46 UTC) #6
imcheng
lgtm
4 years, 11 months ago (2016-01-13 21:50:32 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1587803002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1587803002/1
4 years, 11 months ago (2016-01-13 21:57:03 UTC) #10
commit-bot: I haz the power
Try jobs failed on following builders: android_arm64_dbg_recipe on tryserver.chromium.android (JOB_TIMED_OUT, no build URL) android_chromium_gn_compile_dbg on ...
4 years, 11 months ago (2016-01-13 23:43:07 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1587803002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1587803002/1
4 years, 11 months ago (2016-01-14 04:45:03 UTC) #14
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 11 months ago (2016-01-14 05:26:56 UTC) #16
commit-bot: I haz the power
4 years, 11 months ago (2016-01-14 05:28:15 UTC) #18
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/f89b9c9425b918c35e342308802f17c77edb2f38
Cr-Commit-Position: refs/heads/master@{#369336}

Powered by Google App Engine
This is Rietveld 408576698