[Android] A prototype of the interactive media notification.

chrome/browser/android/chrome_web_contents_delegate_android.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/chrome_web_contents_delegate_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/command_line.h"
 #include "chrome/browser/android/feature_utilities.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/file_select_helper.h"
 #include "chrome/browser/media/media_capture_devices_dispatcher.h"
 #include "chrome/browser/media/media_stream_capture_indicator.h"
 #include "chrome/browser/media/protected_media_identifier_permission_context.h"
 #include "chrome/browser/media/protected_media_identifier_permission_context_factory.h"
 #include "chrome/browser/prerender/prerender_manager.h"
 #include "chrome/browser/prerender/prerender_manager_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/blocked_content/popup_blocker_tab_helper.h"
 #include "chrome/browser/ui/browser_navigator.h"
 #include "chrome/browser/ui/find_bar/find_notification_details.h"
 #include "chrome/browser/ui/find_bar/find_tab_helper.h"
 #include "chrome/browser/ui/tab_helpers.h"
 #include "chrome/common/chrome_switches.h"
 #include "components/app_modal/javascript_dialog_manager.h"
+#include "content/public/browser/android/media_controls_delegate.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/file_chooser_params.h"
 #include "jni/ChromeWebContentsDelegateAndroid_jni.h"
 #include "third_party/WebKit/public/web/WebWindowFeatures.h"
 #include "ui/gfx/geometry/rect.h"
@@ skipping 308 matching lines @@
         NULL,
         user_gesture);
   }
 
   if (was_blocked)
     *was_blocked = !handled;
   if (!handled)
     delete new_contents;
 }
 
+
+void ChromeWebContentsDelegateAndroid::ShowMediaControls(
+    WebContents* web_contents) {
+  DCHECK(web_contents);
+
+  JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env);
+  if (obj.is_null())
+    return;
+
+  content::MediaControlsDelegate* delegate =
+      web_contents->GetMediaControlsDelegate();
+  jboolean jis_paused = delegate->IsPaused();

[mlamouri (slow - plz ping), 2015/06/18 16:43:22]

If I understand correctly, the only reason why you have this IsPaused() call is
because you will call ShowMediaControls() to signal that there is a play/pause
state change, right? It seems that we could generalize that behaviour by having
simply a UpdateMediaControls() and have the method check with the
MediaControlsDelegate what should be done. Alternatively, we could have an
object hanging from WebContentsDelegate that would represent the Media Controls
so the MediaSession could directly tell it to Show(), Hide(), Update(), etc.

[whywhat, 2015/06/19 16:00:34]

So why would you want to split Show and Update?
This method currently does exactly what you describe as the first option. For
the alternative, WebContentsDelegate works like that object now with
MediaSession calling it directly when it needs to show/hide the notification.

I'd like to minimize cross components and Java/C++ interface so would like to
avoid Show/UpdateTitle/UpdateState/UpdateIcon/Hide style interface.

[mlamouri (slow - plz ping), 2015/06/22 14:34:17]

What would you think of having UpdateMediaControls() then have the
WebContentsDelegate use the MediaControlsDelegate to check whether the controls
should be paused, shown, etc. It might be a better signal if we want later to
update the title, etc. Instead of Show/Hide and Show being used for updates too,
we have one Update signal. WDYT?

[whywhat, 2015/06/23 19:39:10]

Per offline discussion we agreed that there're two distinct use cases:
- when media session changes its active state (so that the embedder could
start/stop listening to media keys, setup/tear down the lock screen and software
keys via the notification, etc)
- when something relevant to the visible ui changes, e.g. title, icon, playback
state, etc

I think we could have a single method still to notify of any state change and
the delegate would then determine what changed and do the appropriate action.

+  Java_ChromeWebContentsDelegateAndroid_showMediaControls(
+      env, obj.obj(), jis_paused);
+}
+
+void ChromeWebContentsDelegateAndroid::HideMediaControls(
+    WebContents* web_contents) {
+  DCHECK(web_contents);
+
+  JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env);
+  if (obj.is_null())
+    return;
+
+  Java_ChromeWebContentsDelegateAndroid_hideMediaControls(env, obj.obj());
+}
+
 }  // namespace android
 }  // namespace chrome
 
 jboolean IsCapturingAudio(JNIEnv* env,
                           jclass clazz,
                           jobject java_web_contents) {
   content::WebContents* web_contents =
       content::WebContents::FromJavaWebContents(java_web_contents);
   scoped_refptr<MediaStreamCaptureIndicator> indicator =
       MediaCaptureDevicesDispatcher::GetInstance()->
@@ skipping 12 matching lines @@
   return indicator->IsCapturingVideo(web_contents);
 }
 
 jboolean HasAudibleAudio(JNIEnv* env,
                          jclass clazz,
                          jobject java_web_contents) {
   content::WebContents* web_contents =
       content::WebContents::FromJavaWebContents(java_web_contents);
   return web_contents->WasRecentlyAudible();
 }
+
+void OnMediaControlsPause(
+    JNIEnv* env, jclass clazz, jobject java_web_contents) {
+  content::WebContents* web_contents =
+      content::WebContents::FromJavaWebContents(java_web_contents);
+  content::MediaControlsDelegate* delegate =
+      web_contents->GetMediaControlsDelegate();
+  if (delegate)
+    delegate->OnControlsPause();
+}
+
+void OnMediaControlsResume(
+    JNIEnv* env, jclass clazz, jobject java_web_contents) {
+  content::WebContents* web_contents =
+      content::WebContents::FromJavaWebContents(java_web_contents);
+  content::MediaControlsDelegate* delegate =
+      web_contents->GetMediaControlsDelegate();
+  if (delegate)
+    delegate->OnControlsResume();
+}