Fix an issue that MediaController is disaled when entering fullscreen

content/browser/android/content_video_view.h

 // 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.
 
 #ifndef CONTENT_BROWSER_ANDROID_CONTENT_VIDEO_VIEW_H_
 #define CONTENT_BROWSER_ANDROID_CONTENT_VIDEO_VIEW_H_
 
 #include <jni.h>
 
 #include "base/android/jni_helper.h"
 #include "base/android/scoped_java_ref.h"
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
 #include "base/timer/timer.h"
 
 namespace content {
 
 class BrowserMediaPlayerManager;
 
 // Native mirror of ContentVideoView.java. This class is responsible for
 // creating the Java video view and pass all the player status change to
 // it. It accepts media control from Java class, and forwards it to
 // MediaPlayerManagerImpl.
@@ skipping 18 matching lines @@
 
   // Return the singleton object or NULL.
   static ContentVideoView* GetInstance();
 
   // Getter method called by the Java class to get the media information.
   int GetVideoWidth(JNIEnv*, jobject obj) const;
   int GetVideoHeight(JNIEnv*, jobject obj) const;
   int GetDurationInMilliSeconds(JNIEnv*, jobject obj) const;
   int GetCurrentPosition(JNIEnv*, jobject obj) const;
   bool IsPlaying(JNIEnv*, jobject obj);
-  void UpdateMediaMetadata(JNIEnv*, jobject obj);
+  void RequestMediaMetadata(JNIEnv*, jobject obj);
 
   // Called when the Java fullscreen view is destroyed. If
   // |release_media_player| is true, |manager_| needs to release the player
   // as we are quitting the app.
   void ExitFullscreen(JNIEnv*, jobject, jboolean release_media_player);
 
   // Supposed to be called when the application paused or stopped.
   // Destroys the fullscreen view in a way that it can be recreated
   // via ResumeFullscreenIfSuspended.
   void SuspendFullscreen();
@@ skipping 36 matching lines @@
 
   // Weak reference of corresponding Java object.
   JavaObjectWeakGlobalRef j_content_video_view_;
 
   enum FullscreenState {
     ENTERED,
     SUSPENDED,
     RESUME
   } fullscreen_state_;
 
+  // Weak pointer for posting tasks.
+  base::WeakPtrFactory<ContentVideoView> weak_this_;

[Ted C, 2014/01/16 21:45:08]

weak_factory_ is much more common (~452 occurrences vs ~10)

+
   DISALLOW_COPY_AND_ASSIGN(ContentVideoView);
 };
 
 } // namespace content
 
 #endif  // CONTENT_BROWSER_ANDROID_CONTENT_VIDEO_VIEW_H_