PPAPI: implement GetSupportedPreviewSizes

ppapi/cpp/private/camera_capabilities_private.h

 /* 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.
  */
 
 #ifndef PPAPI_CPP_PRIVATE_CAMERA_CAPABILITIES_PRIVATE_H_
 #define PPAPI_CPP_PRIVATE_CAMERA_CAPABILITIES_PRIVATE_H_
 
+#include <vector>
+
 #include "ppapi/c/private/ppb_camera_capabilities_private.h"
 #include "ppapi/cpp/resource.h"
 #include "ppapi/cpp/size.h"
 
 /// @file
 /// This file defines the CameraCapabilities_Private interface for
 /// establishing an image capture configuration resource within the browser.
 namespace pp {
 
 /// The <code>CameraCapabilities_Private</code> interface contains methods for
@@ skipping 31 matching lines @@
   CameraCapabilities_Private(PassRef, PP_Resource resource);
 
   // Destructor.
   ~CameraCapabilities_Private();
 
   /// GetSupportedPreviewSizes() returns the supported preview sizes for the
   /// given <code>CameraCapabilities_Private</code>.
   ///
   /// @param[out] A vector of <code>Size</code> corresponding to the
   /// supported preview sizes in pixels.
-  void GetSupportedPreviewSizes(std::vector<Size>* preview_sizes);
+  void GetSupportedPreviewSizes(std::vector<Size>& preview_sizes);

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

Google c++ style guide says output should use pointer.

http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Function_Para...

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

Done. Test plugin code needs to be updated, too.

 
   /// GetSupportedJpegSize() returns the supported JPEG sizes for the given
   /// <code>CameraCapabilities_Private</code>.
   ///
   /// @param[out] A vector of <code>Size</code> corresponding to the
   /// supported JPEG image sizes in pixels.
-  void GetSupportedJpegSizes(std::vector<Size>* jpeg_sizes);
+  void GetSupportedJpegSizes(std::vector<Size>& jpeg_sizes);
 
   /// IsCameraCapabilities() determines if the given resource is a
   /// <code>CameraCapabilities_Private</code>.
   ///
   /// @param[in] resource A <code>Resource</code> corresponding to an image
   /// capture capabilities resource.
   ///
   /// @return true if the given resource is an <code>
   /// CameraCapabilities_Private</code> resource, otherwise false.
   static bool IsCameraCapabilities(const Resource& resource);
 };
 
 }  // namespace pp
 
 #endif  /* PPAPI_CPP_PRIVATE_CAMERA_CAPABILITIES_PRIVATE_H_ */
-