Fix SetStreamTextureSize crash

content/renderer/media/android/media_player_renderer_client.cc

 // 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.
 
 #include "content/renderer/media/android/media_player_renderer_client.h"
 
 #include "base/callback_helpers.h"
 
 namespace content {
 
@@ skipping 27 matching lines @@
   // any size information from the MediaPlayer yet.
   // The size will be automatically updated in OnVideoNaturalSizeChange() once
   // we parse the media's metadata.
   // Unretained is safe here because |stream_texture_wrapper_| resets the
   // Closure it has before destroying itself on |compositor_task_runner_|,
   // and |this| is garanteed to live until the Closure has been reset.
   stream_texture_wrapper_->Initialize(
       base::Bind(&MediaPlayerRendererClient::OnFrameAvailable,
                  base::Unretained(this)),
       gfx::Size(1, 1), compositor_task_runner_,
-      base::Bind(&MediaPlayerRendererClient::InitializeRemoteRenderer,
+      base::Bind(&MediaPlayerRendererClient::OnStreamTextureWrapperInitialized,
                  weak_factory_.GetWeakPtr(), demuxer_stream_provider));
 }
 
-void MediaPlayerRendererClient::InitializeRemoteRenderer(
-    media::DemuxerStreamProvider* demuxer_stream_provider) {
+void MediaPlayerRendererClient::OnStreamTextureWrapperInitialized(
+    media::DemuxerStreamProvider* demuxer_stream_provider,
+    bool success) {
   DCHECK(media_task_runner_->BelongsToCurrentThread());
+  if (!success) {
+    base::ResetAndReturn(&init_cb_).Run(
+        media::PipelineStatus::PIPELINE_ERROR_INITIALIZATION_FAILED);
+    return;
+  }
+
   mojo_renderer_->Initialize(
       demuxer_stream_provider, this,
-      base::Bind(&MediaPlayerRendererClient::CompleteInitialization,
+      base::Bind(&MediaPlayerRendererClient::OnRemoteRendererInitialized,
                  weak_factory_.GetWeakPtr()));
 }
 
 void MediaPlayerRendererClient::OnScopedSurfaceRequested(
     const base::UnguessableToken& request_token) {
   DCHECK(request_token);
   stream_texture_wrapper_->ForwardStreamTextureForSurfaceRequest(request_token);
 }
 
-void MediaPlayerRendererClient::CompleteInitialization(
+void MediaPlayerRendererClient::OnRemoteRendererInitialized(
     media::PipelineStatus status) {
   DCHECK(media_task_runner_->BelongsToCurrentThread());
   DCHECK(!init_cb_.is_null());
 
   // TODO(tguilbert): Measure and smooth out the initialization's ordering to
   // have the lowest total initialization time.
   mojo_renderer_->InitiateScopedSurfaceRequest(
       base::Bind(&MediaPlayerRendererClient::OnScopedSurfaceRequested,
                  weak_factory_.GetWeakPtr()));
 
@@ skipping 61 matching lines @@
 
 void MediaPlayerRendererClient::OnVideoOpacityChange(bool opaque) {
   client_->OnVideoOpacityChange(opaque);
 }
 
 void MediaPlayerRendererClient::OnDurationChange(base::TimeDelta duration) {
   client_->OnDurationChange(duration);
 }
 
 }  // namespace content