PPAPI: implement GetSupportedPreviewSizes

content/renderer/pepper/content_renderer_pepper_host_factory.cc

Index: content/renderer/pepper/content_renderer_pepper_host_factory.cc
diff --git a/content/renderer/pepper/content_renderer_pepper_host_factory.cc b/content/renderer/pepper/content_renderer_pepper_host_factory.cc
index 67f49ad97877bf8cdcd6603a406e8da1d184b3e3..0e86a40894eec6e3bd871be47cf10508ad65399c 100644
--- a/content/renderer/pepper/content_renderer_pepper_host_factory.cc
+++ b/content/renderer/pepper/content_renderer_pepper_host_factory.cc
@@ -14,6 +14,7 @@
 #include "content/renderer/pepper/pepper_file_ref_renderer_host.h"
 #include "content/renderer/pepper/pepper_file_system_host.h"
 #include "content/renderer/pepper/pepper_graphics_2d_host.h"
+#include "content/renderer/pepper/pepper_image_capture_host.h"
 #include "content/renderer/pepper/pepper_media_stream_video_track_host.h"
 #include "content/renderer/pepper/pepper_plugin_instance_impl.h"
 #include "content/renderer/pepper/pepper_url_loader_host.h"
@@ -203,6 +204,18 @@ scoped_ptr<ResourceHost> ContentRendererPepperHostFactory::CreateResourceHost(
     }
   }
 
+  // Private interfaces.
+  if (GetPermissions().HasPermission(ppapi::PERMISSION_PRIVATE)) {
+    switch (message.type()) {

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

There is only one case. Using if is enough.

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

I think it's okay because the whole file does this.

+      case PpapiHostMsg_ImageCapture_Create::ID: {
+        scoped_ptr<PepperImageCaptureHost> host(
+            new PepperImageCaptureHost(host_, instance, resource));
+        return host->Init() ? scoped_ptr<ResourceHost>(host.Pass())
+                            : scoped_ptr<ResourceHost>();
+      }
+    }
+  }
+
   return scoped_ptr<ResourceHost>();
 }