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

Issue 612452 link

Starred by 1 user
Status: Fixed
Owner:
█ zqzh...@chromium.org
Android MediaRouter only (left Chro...
Closed: May 2016
Cc:
█ zqzh...@chromium.org
█ avayvod@chromium.org
mlamouri@chromium.org
mfo...@chromium.org
btolsch@chromium.org
taku...@chromium.org
powerb@chromium.org
tguilbert@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Conflicting media notification between MediaFling and PresentationAPI

Project Member Reported by zqzh...@chromium.org, May 17 2016 
1. Open https://avayvod.github.io/mediaflingtest.html
2. Fling some video
3. Navigate to https://storate.googleapis.com/presentation-api/index.html
4. Start PresentationAPI

The MediaFling notification and Cast (PresentationAPI) notification shows at the same time, and the Cast notification shows the MediaFling title.

Not sure what happened, could be possible that PresentationAPI is joining the MediaFling session?

Comment 1 by zqzh...@chromium.org, May 17 2016 

OK. Looks like we use different AppId for MediaFling. For Chromium build, the AppId happens to be the same with the PresentationAPI demo page.

Did a couple of tests (for same AppIds):
1. Start PresentationAPI, start PresentationAPI: The second request cause the existing connection to disconnect and then start the second presentation.
2. Start MediaFling, start PresentationAPI: The second request joins the existing connection.

Anton, do we need to fix this case?

Comment 2 by avayvod@chromium.org, May 17 2016

Owner: zqzh...@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, May 17 2016 

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

commit 01fae9e8879524e52d4c04b08446098de69ef9a8
Author: zqzhang <zqzhang@chromium.org>
Date: Tue May 17 16:22:09 2016

Relaunch application when starting new PresentationAPI connection

BUG= 612452 
TEST=manual tests

Review-Url: https://codereview.chromium.org/1981193003
Cr-Commit-Position: refs/heads/master@{#394141}

[modify] https://crrev.com/01fae9e8879524e52d4c04b08446098de69ef9a8/chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CreateRouteRequest.java

Comment 4 by zqzh...@chromium.org, May 17 2016

Status: Fixed (was: Available)
►

Sign in to add a comment