New issue
Advanced search Search tips

Issue 679338 link

Starred by 2 users
Status: Fixed
Owner:
mlamouri@chromium.org
Closed: Jan 2017
Cc:
chrome-media-session-dev@google.com
beccahughes@chromium.org
mlamouri@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

navigator.mediaSession.addEventListener crashes web page

Project Member Reported by fbeaufort@chromium.org, Jan 9 2017 
Chrome Canary (57.0.2974.0)

1. Open DevTools JS console 
2. Enter "navigator.mediaSession.addEventListener('playpause', function(e) { console.log('playpause', e) });"
3. Chrome crashes with "Aw, Snap!" message ;(

For the record, we should now use navigator.mediaSession.setActionHandler("playpause", function() { ... }

Comment 1 by mlamouri@chromium.org, Jan 9 2017

Labels: -Pri-3 M-57 Pri-1
Owner: mlamouri@chromium.org
Status: Started (was: Unconfirmed)
OUPS :)
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 9 2017 

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

commit 06318a315afd262552ee1393cf360f4adb27223a
Author: mlamouri <mlamouri@chromium.org>
Date: Mon Jan 09 16:26:29 2017

Media Session: remove EventTarget from MediaSession interface inheritance.

Was forgotten and had interesting consequences :)

BUG= 679338 
R=zqzhang@chromium.org

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

[modify] https://crrev.com/06318a315afd262552ee1393cf360f4adb27223a/third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/06318a315afd262552ee1393cf360f4adb27223a/third_party/WebKit/Source/modules/mediasession/MediaSession.idl

Comment 3 by mlamouri@chromium.org, Jan 9 2017

Status: Fixed (was: Started)
►

Sign in to add a comment