Pass Client pointer in Initialize() for VDA/VEA

content/common/gpu/media/exynos_video_encode_accelerator.h

 // Copyright 2013 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_EXYNOS_VIDEO_ENCODE_ACCELERATOR_H_
 #define CONTENT_COMMON_GPU_MEDIA_EXYNOS_VIDEO_ENCODE_ACCELERATOR_H_
 
 #include <list>
 #include <vector>
 
@@ skipping 18 matching lines @@
 
 namespace content {
 
 // This class handles Exynos video encode acceleration by interfacing with the
 // V4L2 devices exported by the Multi Format Codec and GScaler hardware blocks
 // on the Exynos platform.  The threading model of this class is the same as the
 // ExynosVideoDecodeAccelerator (from which class this was designed).
 class CONTENT_EXPORT ExynosVideoEncodeAccelerator
     : public media::VideoEncodeAccelerator {
  public:
-  explicit ExynosVideoEncodeAccelerator(
-      media::VideoEncodeAccelerator::Client* client);
+  ExynosVideoEncodeAccelerator();
   virtual ~ExynosVideoEncodeAccelerator();
 
   // media::VideoEncodeAccelerator implementation.
-  virtual void Initialize(media::VideoFrame::Format format,
+  virtual void Initialize(Client* client,
+                          media::VideoFrame::Format format,
                           const gfx::Size& input_visible_size,
                           media::VideoCodecProfile output_profile,
                           uint32 initial_bitrate) OVERRIDE;
   virtual void Encode(const scoped_refptr<media::VideoFrame>& frame,
                       bool force_keyframe) OVERRIDE;
   virtual void UseOutputBitstreamBuffer(
       const media::BitstreamBuffer& buffer) OVERRIDE;
   virtual void RequestEncodingParametersChange(uint32 bitrate,
                                                uint32 framerate) OVERRIDE;
   virtual void Destroy() OVERRIDE;
@@ skipping 141 matching lines @@
   // are members of this class, any task running on those threads is guaranteed
   // that this object is still alive.  As a result, tasks posted from the child
   // thread to the encoder or device thread should use base::Unretained(this),
   // and tasks posted the other way should use |weak_this_|.
   base::WeakPtrFactory<ExynosVideoEncodeAccelerator> weak_this_ptr_factory_;
   base::WeakPtr<ExynosVideoEncodeAccelerator> weak_this_;
 
   // To expose client callbacks from VideoEncodeAccelerator.
   // NOTE: all calls to these objects *MUST* be executed on
   // child_message_loop_proxy_.
-  base::WeakPtrFactory<Client> client_ptr_factory_;
+  scoped_ptr<base::WeakPtrFactory<Client> > client_ptr_factory_;
   base::WeakPtr<Client> client_;
 
   //
   // Encoder state, owned and operated by encoder_thread_.
   // Before encoder_thread_ has started, the encoder state is managed by
   // the child (main) thread.  After encoder_thread_ has started, the encoder
   // thread should be the only one managing these.
   //
 
   // This thread services tasks posted from the VEA API entry points by the
@@ skipping 82 matching lines @@
   // eventfd fd to signal device poll thread when its poll() should be
   // interrupted.
   int device_poll_interrupt_fd_;
 
   DISALLOW_COPY_AND_ASSIGN(ExynosVideoEncodeAccelerator);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_GPU_MEDIA_EXYNOS_VIDEO_ENCODE_ACCELERATOR_H_