Issue 657566 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	█ avayvod@chromium.org Last visit > 30 days ago
Closed:	Nov 2016
Components:	Blink>Media
EstimatedDays:	----
NextAction:	----
OS:	All
Pri:	1
Type:	Bug
M-56 RemotePlayback

Blocked on:
issue 659669

Blocking:
issue 578833

Implement disableRemotePlayback behavior

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

Issue description

The spec says:

When the disableRemotePlayback attribute is added to the media element, the user agent must run these steps:

Reject any pending promises returned by the RemotePlayback methods with InvalidStateError.
Clear the set of availability callbacks for the media element.
If its state is not disconnected, run the disconnect from remote playback device algorithm for the remote playback device the media element is connected or connecting to.

[ https://w3c.github.io/remote-playback/#disabling-remote-playback ]

Project Member

Comment 1 by bugdroid1@chromium.org, Oct 20 2016

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

commit abdffaa22fbf5d88087a1c5f6dfbfdb41e67d301
Author: avayvod <avayvod@chromium.org>
Date: Thu Oct 20 17:18:20 2016

[Blink, RemotePlayback] Handle disabling remote playback

Notifies RemotePlayback about the attribute being set that
rejects the pending prompt() promise and cancels all availability watch callbacks.

BUG= 657566 
TEST=RemotePlaybackTest

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

[modify] https://crrev.com/abdffaa22fbf5d88087a1c5f6dfbfdb41e67d301/third_party/WebKit/Source/modules/remoteplayback/HTMLMediaElementRemotePlayback.cpp
[modify] https://crrev.com/abdffaa22fbf5d88087a1c5f6dfbfdb41e67d301/third_party/WebKit/Source/modules/remoteplayback/HTMLMediaElementRemotePlayback.h
[modify] https://crrev.com/abdffaa22fbf5d88087a1c5f6dfbfdb41e67d301/third_party/WebKit/Source/modules/remoteplayback/RemotePlayback.cpp
[modify] https://crrev.com/abdffaa22fbf5d88087a1c5f6dfbfdb41e67d301/third_party/WebKit/Source/modules/remoteplayback/RemotePlayback.h
[modify] https://crrev.com/abdffaa22fbf5d88087a1c5f6dfbfdb41e67d301/third_party/WebKit/Source/modules/remoteplayback/RemotePlaybackTest.cpp

Comment 2 by avayvod@chromium.org, Oct 26 2016

Blockedon: 659669

Comment 3 by avayvod@chromium.org, Nov 4 2016

Status: Fixed (was: Started)

►

Issue 657566 link

Issue metadata

Implement disableRemotePlayback behavior

Issue description

Comment 1 by bugdroid1@chromium.org, Oct 20 2016

Comment 1 Deleted

Comment 2 by avayvod@chromium.org, Oct 26 2016

Comment 2 Deleted

Comment 3 by avayvod@chromium.org, Nov 4 2016

Comment 3 Deleted

Sign in to add a comment