PPAPI: implement GetSupportedPreviewSizes

ppapi/proxy/camera_capabilities_resource.cc

Index: ppapi/proxy/camera_capabilities_resource.cc
diff --git a/ppapi/proxy/camera_capabilities_resource.cc b/ppapi/proxy/camera_capabilities_resource.cc
new file mode 100644
index 0000000000000000000000000000000000000000..99bc67a7dced88167d6074ef50fc97a12a41a567
--- /dev/null
+++ b/ppapi/proxy/camera_capabilities_resource.cc
@@ -0,0 +1,57 @@
+// 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.
+
+#include "base/logging.h"
+#include "ppapi/proxy/camera_capabilities_resource.h"
+
+namespace ppapi {
+namespace proxy {
+
+CameraCapabilitiesResource::CameraCapabilitiesResource(PP_Instance instance)
+    : Resource(OBJECT_IS_PROXY, instance),
+      num_preview_sizes_(0),
+      num_jpeg_sizes_(0) {
+}
+
+CameraCapabilitiesResource::~CameraCapabilitiesResource() {
+}
+
+thunk::PPB_CameraCapabilities_API*
+CameraCapabilitiesResource::AsPPB_CameraCapabilities_API() {
+  return this;
+}
+
+void CameraCapabilitiesResource::GetSupportedPreviewSizes(
+    int32_t* array_size,
+    PP_Size** preview_sizes) {
+  *array_size = num_preview_sizes_;
+  *preview_sizes = preview_sizes_.get();
+}
+
+void CameraCapabilitiesResource::GetSupportedJpegSizes(int32_t* array_size,
+                                                       PP_Size** jpeg_sizes) {
+  *array_size = num_jpeg_sizes_;
+  *jpeg_sizes = jpeg_sizes_.get();
+}
+
+void CameraCapabilitiesResource::SetPreviewSizes(
+    const std::vector<PP_Size>& sizes) {

[wuchengli, 2015/02/02 14:42:33]

A reminder. You mentioned you'd move this to thunk layer.

[Justin Chuang, 2015/02/03 12:32:08]

Done. I was wrong. The added argument is still added at to the constructor of
this class. Also remove Create() method in PPB_CameraCapabilities_Private()
because it's not needed. (similar example: PPB_NetworkList)

+  num_preview_sizes_ = sizes.size();
+  DCHECK(!preview_sizes_.get()); // only initialized once
+  preview_sizes_.reset(new PP_Size[num_preview_sizes_]);
+  for (int i = 0; i < num_preview_sizes_; i++)
+    preview_sizes_[i] = sizes[i];
+}
+
+void CameraCapabilitiesResource::SetJpegSizes(
+    const std::vector<PP_Size>& sizes) {
+  num_jpeg_sizes_ = sizes.size();
+  DCHECK(!jpeg_sizes_.get()); // only initialized once
+  jpeg_sizes_.reset(new PP_Size[num_jpeg_sizes_]);
+  for (int i = 0; i < num_jpeg_sizes_; i++)
+    jpeg_sizes_[i] = sizes[i];
+}
+
+}  // namespace proxy
+}  // namespace ppapi