Chromium Code Reviews| Index: content/browser/android/content_video_view.h |
| diff --git a/content/browser/android/content_video_view.h b/content/browser/android/content_video_view.h |
| index ad8945a9683f494ac36f2ba25aefa1e430109721..e75482faca3b2160628fb1afa09a9b797c551604 100644 |
| --- a/content/browser/android/content_video_view.h |
| +++ b/content/browser/android/content_video_view.h |
| @@ -12,6 +12,7 @@ |
| #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 { |
| @@ -50,7 +51,7 @@ class ContentVideoView { |
| 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 |
| @@ -107,6 +108,9 @@ class ContentVideoView { |
| 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 occurrence
|
| + |
| DISALLOW_COPY_AND_ASSIGN(ContentVideoView); |
| }; |