Remove WebMediaPlayerDelegate null checks

content/renderer/media/android/webmediaplayer_android.h

Index: content/renderer/media/android/webmediaplayer_android.h
diff --git a/content/renderer/media/android/webmediaplayer_android.h b/content/renderer/media/android/webmediaplayer_android.h
index 10c095c47913125fb12919c0aff309be2d3a6b98..9ffaee92ccde1eb978fb9080f9f05c28d67d67e7 100644
--- a/content/renderer/media/android/webmediaplayer_android.h
+++ b/content/renderer/media/android/webmediaplayer_android.h
@@ -82,7 +82,7 @@ class WebMediaPlayerAndroid
       blink::WebFrame* frame,
       blink::WebMediaPlayerClient* client,
       blink::WebMediaPlayerEncryptedMediaClient* encrypted_client,
-      base::WeakPtr<media::WebMediaPlayerDelegate> delegate,
+      media::WebMediaPlayerDelegate* delegate,
       RendererMediaPlayerManager* player_manager,
       scoped_refptr<StreamTextureFactory> factory,
       int frame_id,
@@ -277,7 +277,7 @@ class WebMediaPlayerAndroid
   // |delegate_id_|; an id provided after registering with the delegate.  The
   // WebMediaPlayer may also receive directives (play, pause) from the delegate
   // via the WebMediaPlayerDelegate::Observer interface after registration.
-  base::WeakPtr<media::WebMediaPlayerDelegate> delegate_;
+  media::WebMediaPlayerDelegate* delegate_;

[clamy, 2017/01/18 13:56:16]

Could you add a comment describing why this is safe to store as a naked pointer?
This is far from obvious and it would help readers. Thanks!

Same in other header files.

[tguilbert, 2017/01/19 02:39:12]

Done.

   int delegate_id_;
 
   // Callback responsible for determining if loading of media should be deferred