Make VEA client of command buffer; move sync. IPC to VDA/VEA::Initialize()

content/common/gpu/media/v4l2_video_decode_accelerator.h

 // Copyright 2014 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.
 //
 // This file contains an implementation of VideoDecodeAccelerator
 // that utilizes hardware video decoders, which expose Video4Linux 2 API
 // (http://linuxtv.org/downloads/v4l-dvb-apis/).
 
 #ifndef CONTENT_COMMON_GPU_MEDIA_V4L2_VIDEO_DECODE_ACCELERATOR_H_
 #define CONTENT_COMMON_GPU_MEDIA_V4L2_VIDEO_DECODE_ACCELERATOR_H_
@@ skipping 408 matching lines @@
 
   // Make our context current before running any EGL entry points.
   base::Callback<bool(void)> make_context_current_;
 
   // EGL state
   EGLDisplay egl_display_;
 
   // The codec we'll be decoding for.
   media::VideoCodecProfile video_profile_;
 
+  // The WeakPtrFactory for |weak_this_|.

[Ami GONE FROM CHROMIUM, 2014/03/17 03:17:54]

No point in having weak_this_ anymore, is there?
(just GetWeakPtr() when you need it)

[sheu, 2014/03/18 22:38:35]

Careful what you wish for.  It's threading all the way down. :-)

weak_this_factory_ is used from the child thread, so it's unsafe to GetWeakPtr()
from the decoder thread when it posts tasks back.

+  base::WeakPtrFactory<V4L2VideoDecodeAccelerator> weak_this_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(V4L2VideoDecodeAccelerator);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_GPU_MEDIA_V4L2_VIDEO_DECODE_ACCELERATOR_H_