Fix SetStreamTextureSize crash

content/renderer/media/android/stream_texture_wrapper_impl.h

 // Copyright 2016 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_RENDERER_MEDIA_ANDROID_STREAM_TEXTURE_WRAPPER_IMPL_H_
 #define CONTENT_RENDERER_MEDIA_ANDROID_STREAM_TEXTURE_WRAPPER_IMPL_H_
 
 #include <memory>
 
 #include "content/common/content_export.h"
@@ skipping 37 matching lines @@
       bool enable_texture_copy,
       scoped_refptr<StreamTextureFactory> factory,
       scoped_refptr<base::SingleThreadTaskRunner> main_task_runner);
 
   // Creates the underlying StreamTexture, and binds |stream_texture_proxy_| to
   // |compositor_task_runner|.
   //
   // Additional threading considerations:
   //   - Can be called from any thread.
   //   - Initialization will be posted to |main_task_runner_|.
-  //   - |init_cb| will be run on the calling thread.
+  //   - |init_cb| will be run on the calling thread, and will passed a bool

[watk, 2016/12/07 01:48:11]

"will be"

[tguilbert, 2016/12/07 19:23:55]

Done.

+  //     indicating whether the initialization was successful.
   //   - New frames will be signaled on |compositor_task_runner| via |client|'s
   //     DidReceiveFrame() method.
   void Initialize(
       const base::Closure& received_frame_cb,
       const gfx::Size& natural_size,
       scoped_refptr<base::SingleThreadTaskRunner> compositor_task_runner,
-      const base::Closure& init_cb) override;
+      const StreamTextureWrapperInitCB& init_cb) override;
 
   // Should be called when the Video size changes.
   // Can be called from any thread, but runs on |main_task_runner_|.
   void UpdateTextureSize(const gfx::Size& natural_size) override;
 
   // Returns the latest frame.
   // N.B: We create a single VideoFrame at initialization time (and update it
   // in UpdateTextureSize()), and repeatedly return it here. The underlying
   // texture's changes are signalled via |client|'s DidReceiveFrame() callback.
   scoped_refptr<media::VideoFrame> GetCurrentFrame() override;
 
   // Sends the StreamTexture to the browser process, to fulfill the request
   // identified by |request_token|.
   // Uses the gpu::ScopedSurfaceRequestConduit to forward the underlying
   // SurfaceTexture to the ScopedSurfaceRequestManager.
   void ForwardStreamTextureForSurfaceRequest(
       const base::UnguessableToken& request_token) override;
 
  private:
   StreamTextureWrapperImpl(
       bool enable_texture_copy,
       scoped_refptr<StreamTextureFactory> factory,
       scoped_refptr<base::SingleThreadTaskRunner> main_task_runner);
   ~StreamTextureWrapperImpl() override;
 
   // Destroys |this| safely on |main_task_runner_|.
   void Destroy() override;
 
   void InitializeOnMainThread(const base::Closure& received_frame_cb,
-                              const base::Closure& init_cb);
+                              const StreamTextureWrapperInitCB& init_cb);
 
   void ReallocateVideoFrame(const gfx::Size& natural_size);
 
   void SetCurrentFrameInternal(
       const scoped_refptr<media::VideoFrame>& video_frame);
 
   bool enable_texture_copy_;
 
   // Client GL texture ID allocated to the StreamTexture.
   unsigned texture_id_;
@@ skipping 17 matching lines @@
   scoped_refptr<base::SingleThreadTaskRunner> compositor_task_runner_;
 
   base::WeakPtrFactory<StreamTextureWrapperImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(StreamTextureWrapperImpl);
 };
 
 }  // namespace media
 
 #endif  // CONTENT_RENDERER_MEDIA_ANDROID_STREAM_TEXTURE_WRAPPER_IMPL_H_