media: Create Mojo StructTraits for VideoFrame

media/mojo/common/mojo_shared_buffer_video_frame.h

 // Copyright 2016 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_MOJO_COMMON_MOJO_SHARED_BUFFER_VIDEO_FRAME_H_
 #define MEDIA_MOJO_COMMON_MOJO_SHARED_BUFFER_VIDEO_FRAME_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 54 matching lines @@
       size_t v_offset,
       int32_t y_stride,
       int32_t u_stride,
       int32_t v_stride,
       base::TimeDelta timestamp);
 
   // Returns the offsets relative to the start of |shared_buffer| for the
   // |plane| specified.
   size_t PlaneOffset(size_t plane) const;
 
+  // Returns the mojo shared memory handle. This object continues to own the
+  // handle. Caller should call duplicate the handle if they want to keep a

[dcheng, 2017/06/05 20:33:01]

Nit: delete "call"

[sandersd (OOO until July 31), 2017/06/06 01:05:47]

Done.

+  // copy of the shared memory.
+  const mojo::SharedBufferHandle& Handle() const;
+
+  // Returns the size of the shared memory.
+  size_t MappedSize() const;
+
   // Sets the callback to be called to free the shared buffer. If not null,
   // it is called on destruction, and is passed ownership of |handle|.
   void SetMojoSharedBufferDoneCB(
       const MojoSharedBufferDoneCB& mojo_shared_buffer_done_cb);
 
  private:
   // mojo::TypeConverter added as a friend so that MojoSharedBufferVideoFrame
   // can be transferred across a mojo connection.
   friend struct mojo::TypeConverter<mojo::StructPtr<mojom::VideoFrame>,
                                     scoped_refptr<VideoFrame>>;
@@ skipping 10 matching lines @@
 
   // Initializes the MojoSharedBufferVideoFrame by creating a mapping onto
   // the shared memory, and then setting the strides and offsets as specified.
   bool Init(int32_t y_stride,
             int32_t u_stride,
             int32_t v_stride,
             size_t y_offset,
             size_t u_offset,
             size_t v_offset);
 
-  // Returns the mojo shared memory handle. This object continues to own the
-  // handle. Caller should call duplicate the handle if they want to keep a
-  // copy of the shared memory.
-  const mojo::SharedBufferHandle& Handle() const;
-
-  // Returns the size of the shared memory.
-  size_t MappedSize() const;
-
   uint8_t* shared_buffer_data() {
     return reinterpret_cast<uint8_t*>(shared_buffer_mapping_.get());
   };
 
   mojo::ScopedSharedBufferHandle shared_buffer_handle_;
   mojo::ScopedSharedBufferMapping shared_buffer_mapping_;
   size_t shared_buffer_size_;
   size_t offsets_[kMaxPlanes];
   MojoSharedBufferDoneCB mojo_shared_buffer_done_cb_;
 
   DISALLOW_COPY_AND_ASSIGN(MojoSharedBufferVideoFrame);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_MOJO_COMMON_MOJO_SHARED_BUFFER_VIDEO_FRAME_H_