Restart fullscreen video playback when switching back from background

content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java

Index: content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java b/content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java
index 5e73d4f88c2a44a5dc89f02d9dcc90a873b51fe0..2589711109b84af02da916462d9b83bf9fecc513 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java
@@ -335,11 +335,10 @@ public class ContentVideoView extends FrameLayout implements MediaController.Med
 
     @Override
     public void surfaceDestroyed(SurfaceHolder holder) {
-        if (mNativeContentVideoView != 0) {
-            nativeSetSurface(mNativeContentVideoView, null);
-        }
+        // We expect the native side to call destroyContentVideoView

[qinmin, 2013/12/18 02:09:46]

This is not right. When user hit the home button, SurfaceDestroyed should be
called before the native side call exitFullscreen.

[kbalazs, 2013/12/18 19:45:05]

If the user calls ContentView.onHide in it's activity's onStop than PauseVideo
will be called before the view is destroyed. I think surfaceDestroyed cannot be
called before onStop returns. So this is the case with test_shell and
content_shell and I expect that all clients should do that.

However we can keep it to make sure it's destroyed even if ContentView.onHide is
never called.

+        // before our surface could have been bone.
+        assert mVideoSurfaceView == null;
         mSurfaceHolder = null;
-        post(mExitFullscreenRunnable);
     }
 
     private void setMediaController(MediaController controller) {