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

Issue 678979 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Android MediaRouter only (left Chro...
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

Disable Media Session API on WebView

Project Member Reported by mlamouri@chromium.org, Jan 6 2017

Issue description

We are aiming to ship the API on Android but we need to add a flag to disable the API on WebView, see:
https://cs.chromium.org/chromium/src/android_webview/lib/main/aw_main_delegate.cc?rcl=0&l=65

Alternatively, we can disable it via a setting (aw_settings.cc) but I think a switch makes the more sense.
 
Status: Started (was: Assigned)
Labels: -OS-Linux OS-Android
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 10 2017

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

commit 30fb31340be3ffd85db42b176f997527fa92538f
Author: zqzhang <zqzhang@chromium.org>
Date: Tue Jan 10 18:52:24 2017

Disable MediaSession API for WebView

Currently there is no UI for media metadata and controls on
WebView. Therefore we should disable the API on WebView. It is
reasonable to let the host app show the UI itself on WebView.

The API intent to ship is still pending approval so it's not
enabled on any platform by default yet. After getting approved,
the next step is to just enable the API on Android and WebView
will be excluded because of this CL.

Intent to ship (pending approval):
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/xIpJ_303phw

BUG= 678979 

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

[modify] https://crrev.com/30fb31340be3ffd85db42b176f997527fa92538f/android_webview/lib/main/aw_main_delegate.cc
[modify] https://crrev.com/30fb31340be3ffd85db42b176f997527fa92538f/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/30fb31340be3ffd85db42b176f997527fa92538f/content/child/runtime_features.cc
[modify] https://crrev.com/30fb31340be3ffd85db42b176f997527fa92538f/content/public/common/content_switches.cc
[modify] https://crrev.com/30fb31340be3ffd85db42b176f997527fa92538f/content/public/common/content_switches.h
[modify] https://crrev.com/30fb31340be3ffd85db42b176f997527fa92538f/third_party/WebKit/Source/web/WebRuntimeFeatures.cpp
[modify] https://crrev.com/30fb31340be3ffd85db42b176f997527fa92538f/third_party/WebKit/public/web/WebRuntimeFeatures.h

Status: Fixed (was: Started)

Sign in to add a comment