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

Issue 660408 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug

Blocking:
issue 578833



Sign in to add a comment

Implement RemotePlayback.onconnecting event

Project Member Reported by avayvod@chromium.org, Oct 28 2016

Issue description

As of today, it doesn't fire when prompt() is resolved.

 
Status: Started (was: Assigned)
https://codereview.chromium.org/2464163004
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 8 2016

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

commit fee74d8fdab4a7bae779fac126a910827dfce5bf
Author: avayvod <avayvod@chromium.org>
Date: Tue Nov 08 00:00:08 2016

[RemotePlayback] Implement 'connecting' state

BUG= 660408 
TEST=manual

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

[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/chrome/android/java/src/org/chromium/chrome/browser/media/remote/AbstractMediaRouteController.java
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/chrome/android/java/src/org/chromium/chrome/browser/media/remote/MediaRouteController.java
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/chrome/android/java/src/org/chromium/chrome/browser/media/remote/RemoteMediaPlayerBridge.java
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/chrome/browser/media/android/remote/remote_media_player_bridge.cc
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/chrome/browser/media/android/remote/remote_media_player_bridge.h
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/chrome/browser/media/android/remote/remote_media_player_manager.cc
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/chrome/browser/media/android/remote/remote_media_player_manager.h
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/content/common/media/media_player_messages_android.h
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/content/renderer/media/android/renderer_media_player_manager.cc
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/content/renderer/media/android/renderer_media_player_manager.h
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/content/renderer/media/android/webmediaplayer_android.cc
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/content/renderer/media/android/webmediaplayer_android.h
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/content/renderer/media/webmediaplayer_ms_unittest.cc
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/media/blink/renderer_media_player_interface.h
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/media/blink/webmediaplayer_cast_android.cc
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/media/blink/webmediaplayer_cast_android.h
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/media/blink/webmediaplayer_impl_unittest.cc
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/third_party/WebKit/Source/core/html/HTMLMediaElement.h
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/third_party/WebKit/Source/modules/remoteplayback/RemotePlayback.cpp
[modify] https://crrev.com/fee74d8fdab4a7bae779fac126a910827dfce5bf/third_party/WebKit/public/platform/WebMediaPlayerClient.h

Status: Fixed (was: Started)
Labels: Needs-Feedback
avayvod@ : Could you please let us know if it can be verified from TE end, if so please help with the steps to verify.
Status: Available (was: Fixed)
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 16 2016

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

commit 3458baf0849127220e308e4aed86b9228cad8c2c
Author: avayvod <avayvod@chromium.org>
Date: Wed Nov 16 18:49:07 2016

[RemotePlayback] Added 'connecting' state to the IDL file

BUG= 660408 
TEST=none
TBR=mlamouri@chromium.org

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

[modify] https://crrev.com/3458baf0849127220e308e4aed86b9228cad8c2c/third_party/WebKit/Source/modules/remoteplayback/RemotePlayback.idl

Status: Fixed (was: Available)
Re #4 I don't think any verification is needed, it's a Web API feature tracking bug.

Sign in to add a comment