PPAPI: implement GetSupportedPreviewSizes

content/renderer/pepper/content_renderer_pepper_host_factory.cc

 // Copyright (c) 2012 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 "content/renderer/pepper/content_renderer_pepper_host_factory.h"
 
 #include "base/logging.h"
 #include "base/strings/string_util.h"
 #include "content/common/content_switches_internal.h"
 #include "content/public/common/content_client.h"
 #include "content/public/renderer/content_renderer_client.h"
 #include "content/renderer/pepper/pepper_audio_input_host.h"
 #include "content/renderer/pepper/pepper_compositor_host.h"
 #include "content/renderer/pepper/pepper_file_chooser_host.h"
 #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"
 #include "content/renderer/pepper/pepper_video_capture_host.h"
 #include "content/renderer/pepper/pepper_video_decoder_host.h"
 #include "content/renderer/pepper/pepper_video_destination_host.h"
 #include "content/renderer/pepper/pepper_video_source_host.h"
 #include "content/renderer/pepper/pepper_websocket_host.h"
 #include "content/renderer/pepper/ppb_image_data_impl.h"
 #include "content/renderer/pepper/renderer_ppapi_host_impl.h"
@@ skipping 165 matching lines @@
             new PepperVideoCaptureHost(host_, instance, resource);
         if (!host->Init()) {
           delete host;
           return scoped_ptr<ResourceHost>();
         }
         return scoped_ptr<ResourceHost>(host);
       }
     }
   }
 
+  // Private interfaces.
+  if (GetPermissions().HasPermission(ppapi::PERMISSION_PRIVATE)) {
+    switch (message.type()) {
+      case PpapiHostMsg_ImageCapture_Create::ID: {
+        scoped_ptr<PepperImageCaptureHost> host(
+            new PepperImageCaptureHost(host_, instance, resource));
+        return host->Init() ? scoped_ptr<ResourceHost>(host.Pass())

[dcheng, 2015/02/12 21:47:20]

return host->Init() ? host : nullptr;

should work.

[Justin Chuang, 2015/02/16 19:15:23]

Actually no, it needs to upcast to ResourceHost in scoped_ptr. There are several
places of similar code in this file.

[dcheng, 2015/02/16 19:24:22]

return host->Init() ? host.Pass() : nullptr;

should still work.

[Justin Chuang, 2015/02/16 19:54:22]

You're right. Pass() properly handles upcast. cl upload again.

+                            : scoped_ptr<ResourceHost>();
+      }
+    }
+  }
+
   return scoped_ptr<ResourceHost>();
 }
 
 const ppapi::PpapiPermissions&
 ContentRendererPepperHostFactory::GetPermissions() const {
   return host_->GetPpapiHost()->permissions();
 }
 
 }  // namespace content