[PPAPI] API definition for video media stream artifacts

ppapi/cpp/video_frame.h

Index: ppapi/cpp/video_frame.h
diff --git a/ppapi/cpp/video_frame.h b/ppapi/cpp/video_frame.h
new file mode 100644
index 0000000000000000000000000000000000000000..666212a19c86e743144c5ae1527fa943492aff72
--- /dev/null
+++ b/ppapi/cpp/video_frame.h
@@ -0,0 +1,68 @@
+// Copyright (c) 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 PPAPI_CPP_VIDEO_FRAME_H_
+#define PPAPI_CPP_VIDEO_FRAME_H_
+
+#include "ppapi/c/ppb_video_frame.h"
+#include "ppapi/cpp/resource.h"
+#include "ppapi/cpp/size.h"
+
+namespace pp {
+
+class VideoFrame : public Resource {
+ public:
+  /// Default constructor for creating an is_null()
+  /// <code>VideoFrame</code> object.
+  VideoFrame();
+
+  /// A constructor used when you have received a <code>PP_Resource</code> as a
+  /// return value that has had 1 ref added for you.
+  ///
+  /// @para[in] resource A <code>PPB_VideoFrame</code> resource.
+  VideoFrame(PassRef, PP_Resource resource);
+
+  virtual ~VideoFrame();
+
+  /// Gets timestamp of the video frame.
+  ///
+  /// @return A <code>PP_TimeDelta</code> containing timestamp of the video
+  /// frame. Given in seconds since the start of the containing video stream.
+  PP_TimeDelta GetTimestamp() const;
+
+  /// Sets the timestamp of the video frame. Given in seconds since the
+  /// start of the containing video stream.
+  ///
+  /// @param[in] timestamp A <code>PP_TimeDelta</code> containing the timestamp
+  /// of the video frame. Given in seconds since the start of the containing
+  /// video stream.
+  void SetTimestamp(PP_TimeDelta timestamp);
+
+  /// Gets format of the video frame.
+  ///
+  /// @return A <code>PP_VideoFrame_Format</code> containing the format of the
+  /// video frame.
+  PP_VideoFrame_Format GetFormat() const;
+
+  /// Gets size of the video frame.
+  ///
+  /// @param[out] size A <code>PP_Size</code>

[yzshen1, 2013/12/26 17:52:22]

Please update the comment, it is not PP_Size here.

[Peng, 2013/12/26 18:33:41]

Done.

+  ///
+  /// @return True if gets size successfully, otherwise false  is returned.

[yzshen1, 2013/12/26 17:52:22]

extra space before 'is'.

[Peng, 2013/12/26 18:33:41]

Done.

+  bool GetSize(Size* size) const;
+
+  /// Gets data buffer for video frame pixels.
+  ///
+  /// @return A pointer to the beginning of data buffer.
+  void* GetDataBuffer();
+
+  /// Gets size of data buffer.
+  ///
+  /// @return The size of the data buffer.
+  uint32_t GetDataBufferSize() const;
+};
+
+}  // namespace pp
+
+#endif  // PPAPI_CPP_VIDEO_FRAME_H_