AVDACodecImages keep their backing SurfaceTexture alive

media/gpu/avda_codec_image.h

 // Copyright 2015 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 MEDIA_GPU_AVDA_CODEC_IMAGE_H_
 #define MEDIA_GPU_AVDA_CODEC_IMAGE_H_
 
 #include <stdint.h>
 
 #include <memory>
 
 #include "base/macros.h"
 #include "gpu/command_buffer/service/gl_stream_texture_image.h"
 #include "media/gpu/avda_shared_state.h"
 
 namespace ui {
 class ScopedMakeCurrent;
 }
 
 namespace media {
 
+class VideoCodecBridge;

[liberato (no reviews please), 2016/06/06 21:35:54]

was this because of the include change in shared_state?

[watk, 2016/06/07 02:22:10]

Must have been :\ I just did what I had to in order to please the compiler.

+
 // GLImage that renders MediaCodec buffers to a SurfaceTexture or SurfaceView as
 // needed in order to draw them.
 class AVDACodecImage : public gpu::gles2::GLStreamTextureImage {
  public:
   AVDACodecImage(int picture_buffer_id,
                  const scoped_refptr<AVDASharedState>& shared_state,
                  media::VideoCodecBridge* codec,
                  const base::WeakPtr<gpu::gles2::GLES2Decoder>& decoder,
                  const scoped_refptr<gl::SurfaceTexture>& surface_texture);
 
@@ skipping 121 matching lines @@
 
   // The picture buffer id attached to this image.
   int picture_buffer_id_;
 
   DISALLOW_COPY_AND_ASSIGN(AVDACodecImage);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_GPU_AVDA_CODEC_IMAGE_H_