PPAPI: implement GetSupportedPreviewSizes

ppapi/proxy/camera_capabilities_resource.cc

+// 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(

[wuchengli, 2015/01/19 14:05:03]

SetXXX usually implies this can be called more than once. This better be done in
the constructor. Same for SetJpegSizes.

[Justin Chuang, 2015/01/26 15:00:51]

You are right. Actually I found I should do it in thunk layer, not proxy layer.
So the code is wrong.  

Will update in later CL.

+    const std::vector<PP_Size>& sizes) {
+  num_preview_sizes_ = sizes.size();
+  DCHECK(!preview_sizes_.get()); // only initialized once

[wuchengli, 2015/01/19 14:05:03]

s/initialized/initialize/.

+  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