Changes to enable HTML5 autoplay for voice search.

content/browser/android/content_view_core_impl.cc

 // Copyright 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 "content/browser/android/content_view_core_impl.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/android/scoped_java_ref.h"
@@ skipping 311 matching lines @@
 }
 
 void ContentViewCoreImpl::OnShow(JNIEnv* env, jobject obj) {
   Show();
 }
 
 void ContentViewCoreImpl::Show() {
   GetWebContents()->WasShown();
 }
 
+void ContentViewCoreImpl::DisableMediaAutoplay(JNIEnv* env, jobject obj) {
+  SetMediaAutoplayEnabled(false);
+}
+
+void ContentViewCoreImpl::EnableMediaAutoplay() {
+  SetMediaAutoplayEnabled(true);
+}
+
+void ContentViewCoreImpl::SetMediaAutoplayEnabled(bool isEnabled) {

[Ted C, 2013/11/26 19:24:29]

c++ naming for variables doesn't use camel casing.  So this should be
'is_enabled' or just 'enable'

[apiccion, 2013/12/03 02:29:19]

Done.

+  if (isEnabled == is_media_autoplay_enabled_) return;

[Ted C, 2013/11/26 19:24:29]

return needs to be on the next line.

Also, why do we need to keep track of is_media_autoplay_enabled_ internally?

Can't this just be:

RenderViewHost* host = web_contents_->GetRenderViewHost();
if (!host)
  return;
WebPreferences prefs = host->GetWebkitPreferences();
if (prefs.user_gesture_required_for_media_playback != !enable) {
  prefs.user_gesture_required_for_media_playback = !enable;
  host->UpdateWebkitPreferences(prefs);
}

[apiccion, 2013/12/03 02:29:19]

GetWebkitPreferences() would be better understood as
'GetDefaultWebkitPreferences'. Its state doesn't update to reflect our changes.
 
See
RenderViewHostImpl::GetWebkitPreferences-->'WebContentsImpl::GetWebkitPrefs()-->RenderViewHostImpl::GetWebkitPrefs(GURL)'

On 2013/11/26 19:24:29, Ted C wrote:

+  is_media_autoplay_enabled_ = isEnabled;
+  RenderViewHost* host = web_contents_->GetRenderViewHost();
+  WebPreferences prefs = host->GetWebkitPreferences();
+  prefs.user_gesture_required_for_media_playback = !isEnabled;
+  host->UpdateWebkitPreferences(prefs);
+}
+
+

[Ted C, 2013/11/26 19:24:29]

remove extra blank line

[apiccion, 2013/12/03 02:29:19]

Done.

 void ContentViewCoreImpl::Hide() {
   GetWebContents()->WasHidden();
   PauseVideo();
 }
 
 void ContentViewCoreImpl::PauseVideo() {
   RenderViewHost* host = web_contents_->GetRenderViewHost();
   if (host)
     host->Send(new ViewMsg_PauseVideo(host->GetRoutingID()));
 }
@@ skipping 1294 matching lines @@
       reinterpret_cast<ui::ViewAndroid*>(view_android),
       reinterpret_cast<ui::WindowAndroid*>(window_android));
   return reinterpret_cast<intptr_t>(view);
 }
 
 bool RegisterContentViewCore(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace content