Serialize usage of SurfaceView between AVDA instances.

media/gpu/android_video_decode_accelerator.h

Index: media/gpu/android_video_decode_accelerator.h
diff --git a/media/gpu/android_video_decode_accelerator.h b/media/gpu/android_video_decode_accelerator.h
index 60ecff06e0c85f9892aafa2d2531bbb1d4150d3f..ac19026b20fa8c3d638d52ed71ec3159d56c2ee1 100644
--- a/media/gpu/android_video_decode_accelerator.h
+++ b/media/gpu/android_video_decode_accelerator.h
@@ -210,6 +210,18 @@ class MEDIA_GPU_EXPORT AndroidVideoDecodeAccelerator
     DISALLOW_COPY_AND_ASSIGN(CodecConfig);
   };
 
+  // Callback that is called when the SurfaceView becomes available, if it's
+  // not during Initialize.  |success| is true if it is now available, false
+  // if initialization should stop.
+  void OnSurfaceAvailable(bool success);
+
+  // Finish initialization of the strategy.  This is to be called when the
+  // SurfaceView in |surface_id_|, if any, is no longer busy.  It will return
+  // false on failure, and true if initialization was successful.  This includes
+  // synchronous and asynchronous init; the AVDA might not yet have a codec on
+  // success, but async init will at least be in progress.
+  bool InitializeStrategy();

[watk, 2016/05/18 20:38:18]

Header order doesn't match cc order

[liberato (no reviews please), 2016/05/18 21:13:13]

not sure that i follow -- OnSurfaceAvailable preceeds InitializeStrategy.  the
rest of them are out of order, but i kept the new ones close to ::Initialize.

couldn't put them close in the header since ::Initialize is inherited.

[watk, 2016/05/18 21:21:14]

Sorry, I just noticed that InitializeStrategy is followed by DoIOTask so I
thought you inserted them at the wrong place. But you did the right thing and
the order was already weird.

+
   // A part of destruction process that is sometimes postponed after the drain.
   void ActualDestroy();
 
@@ -263,7 +275,7 @@ class MEDIA_GPU_EXPORT AndroidVideoDecodeAccelerator
   void DecodeBuffer(const media::BitstreamBuffer& bitstream_buffer);
 
   // Called during Initialize() for encrypted streams to set up the CDM.
-  void InitializeCdm(int cdm_id);
+  void InitializeCdm();
 
   // Called after the CDM obtains a MediaCrypto object.
   void OnMediaCryptoReady(
@@ -425,6 +437,9 @@ class MEDIA_GPU_EXPORT AndroidVideoDecodeAccelerator
   // called NotifyInitializationComplete.
   bool deferred_initialization_pending_;
 
+  // Cdm id that we will use in InitializeCdm().
+  int cdm_id_;
+
   int surface_id_;
 
   OnDestroyingSurfaceCallback on_destroying_surface_cb_;