ReleaseOutputBuffer to surface back and front buffers where possible.

content/common/gpu/media/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 CONTENT_COMMON_GPU_MEDIA_AVDA_CODEC_IMAGE_H_
 #define CONTENT_COMMON_GPU_MEDIA_AVDA_CODEC_IMAGE_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 35 matching lines @@
                             const gfx::RectF& crop_rect) override;
   void OnMemoryDump(base::trace_event::ProcessMemoryDump* pmd,
                     uint64_t process_tracing_id,
                     const std::string& dump_name) override;
   // gpu::gles2::GLStreamTextureMatrix implementation
   void GetTextureMatrix(float xform[16]) override;
 
   // Decoded buffer index that has the image for us to display.
   void SetMediaCodecBufferIndex(int buffer_index);
 
-  // Return the codec buffer that we will return to the codec, or
-  // <0 if there is no such buffer.
-  int GetMediaCodecBufferIndex() const;
-
   // Set the size of the current image.
   void SetSize(const gfx::Size& size);
 
+  enum class UpdateMode {
+    DISCARD_CODEC_BUFFER,   // Discards the codec buffer, no UpdateTexImage().
+    RENDER_TO_BACK_BUFFER,  // Renders to back buffer, doesn't UpdateTexImage().
+    RENDER_TO_FRONT_BUFFER  // Renders to back buffer, calls UpdateTexImage(),
+                            // which turns the back buffer into the front.

[liberato (no reviews please), 2016/04/22 20:25:08]

this comment isn't right for SV.  we use FRONT_BUFFER but we don't call
UpdateTexImage.

in the SV case, we're technically rendering to a back buffer, but surfaceflinger
will promote it automatically for us.  i still like the name TO_FRONT_BUFFER for
this case, since that's where it ends up after this call returns, just
asynchronously.

[DaleCurtis, 2016/04/22 20:58:02]

Comment clarified.

+  };
+
+  // Releases the attached codec buffer (if not already released) indicated by
+  // |codec_buffer_index_| and updates the surface if specified by the given
+  // |update_mode|.  See UpdateMode documentation for details.
+  void UpdateSurface(UpdateMode update_mode);
+
   // Updates the MediaCodec for this image; clears |codec_buffer_index_|.
   void CodecChanged(media::MediaCodecBridge* codec);
 
   void SetTexture(gpu::gles2::Texture* texture);
 
+  // Indicates if the codec buffer has been released to the back buffer.
+  bool is_rendered_to_back_buffer() const {
+    return codec_buffer_index_ == kUpdateOnly;
+  }
+
+  // Indicates if the codec buffer has been released to the front or back
+  // buffer.
+  bool is_rendered() const {
+    return codec_buffer_index_ <= kInvalidCodecBufferIndex;
+  }
+
  protected:
   ~AVDACodecImage() override;
 
  private:
-  enum { kInvalidCodecBufferIndex = -1 };
+  enum { kInvalidCodecBufferIndex = -1, kUpdateOnly = -2 };
 
   // Make sure that the surface texture's front buffer is current.  This will
   // save / restore the current context.  It will optionally restore the texture
   // bindings in the surface texture's context, based on |mode|.  This is
   // intended as a hint if we don't need to change contexts.  If we do need to
   // change contexts, then we'll always preserve the texture bindings in the
   // both contexts.  In other words, the caller is telling us whether it's
   // okay to change the binding in the current context.
   enum RestoreBindingsMode { kDontRestoreBindings, kDoRestoreBindings };
   void UpdateSurfaceTexture(RestoreBindingsMode mode);
 
+  // Internal helper for UpdateSurface() that allows callers to specify the
+  // RestoreBindingsMode when a SurfaceTexture is already attached prior to
+  // calling this method.
+  void UpdateSurfaceInternal(UpdateMode update_mode,
+                             RestoreBindingsMode attached_bindings_mode);
+
+  // Releases the attached codec buffer (if not already released) indicated by
+  // |codec_buffer_index_|. Never updates the actual surface. See UpdateMode
+  // documentation for details. For the purposes of this function the values
+  // RENDER_TO_FRONT_BUFFER and RENDER_TO_BACK_BUFFER do the same thing.
+  void ReleaseOutputBuffer(UpdateMode update_mode);
+
   // Attach the surface texture to our GL context to whatever texture is bound
   // on the active unit.
   void AttachSurfaceTextureToContext();
 
   // Make shared_state_->context() current if it isn't already.
   std::unique_ptr<ui::ScopedMakeCurrent> MakeCurrentIfNeeded();
 
-  // Return whether or not the current context is in the same share group as
-  // |surface_texture_|'s client texture.
-  // TODO(liberato): is this needed?
-  bool IsCorrectShareGroup() const;
-
   // Return whether there is a codec buffer that we haven't rendered yet.  Will
   // return false also if there's no codec or we otherwise can't update.
   bool IsCodecBufferOutstanding() const;
 
   // Shared state between the AVDA and all AVDACodecImages.
   scoped_refptr<AVDASharedState> shared_state_;
 
   // The MediaCodec buffer index that we should render. Only valid if not equal
   // to |kInvalidCodecBufferIndex|.
   int codec_buffer_index_;
@@ skipping 22 matching lines @@
 
   // The picture buffer id attached to this image.
   int picture_buffer_id_;
 
   DISALLOW_COPY_AND_ASSIGN(AVDACodecImage);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_GPU_MEDIA_AVDA_CODEC_IMAGE_H_