Changes to enable HTML5 autoplay for voice search.

content/browser/android/content_view_core_impl.h

Index: content/browser/android/content_view_core_impl.h
diff --git a/content/browser/android/content_view_core_impl.h b/content/browser/android/content_view_core_impl.h
index 0c3a5ec3ea18b8568ec3c00228635e4ed047a1d2..009b03d7936192e6d8a0544852480a46ec3c5f65 100644
--- a/content/browser/android/content_view_core_impl.h
+++ b/content/browser/android/content_view_core_impl.h
@@ -56,6 +56,8 @@ class ContentViewCoreImpl : public ContentViewCore,
   virtual void LoadUrl(NavigationController::LoadURLParams& params) OVERRIDE;
   virtual jint GetCurrentRenderProcessId(JNIEnv* env, jobject obj) OVERRIDE;
   virtual void ShowPastePopup(int x, int y) OVERRIDE;
+  void DisableMediaAutoplay(JNIEnv* env, jobject obj);

[David Trainor- moved to gerrit, 2013/11/26 18:59:32]

Can this be SetMediaAutoplayEnabled()?  It might be cleaner if we pipe the
enable and disable through the same mechanism.

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

Agreed.

And EnableMediaAutoplay isn't exposed to java, which makes this feel unbalanced.

These methods should probably have more documentation to state that this is only
for the current renderer and subsequent navigations may clear this state.  Or if
we don't want that to be the case, we should be listening for render host
changes in this class and updating the prefs accordingly.

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

Done.

+  virtual void EnableMediaAutoplay();
   virtual unsigned int GetScaledContentTexture(
       float scale,
       gfx::Size* out_size) OVERRIDE;
@@ -353,6 +355,9 @@ class ContentViewCoreImpl : public ContentViewCore,
   // Send device_orientation_ to renderer.
   void SendOrientationChangeEventInternal();
 
+  // Update state of media autoplay.
+  void SetMediaAutoplayEnabled(bool isEnabled);

[David Trainor- moved to gerrit, 2013/11/26 18:59:32]

We could get rid of this if we piped through the other method.

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

Done.

+
   // A weak reference to the Java ContentViewCore object.
   JavaObjectWeakGlobalRef java_ref_;
 
@@ -383,6 +388,9 @@ class ContentViewCoreImpl : public ContentViewCore,
   // will be sent to Renderer once it is ready.
   int device_orientation_;
 
+  // Whether media autoplay is enabled for this content view.
+  bool is_media_autoplay_enabled_;
+
   DISALLOW_COPY_AND_ASSIGN(ContentViewCoreImpl);
 };