Add browser resource host for Apps v2 APIs in Pepper.

Add browser resource host for Apps v2 APIs in Pepper.

This CL doesn't include the logic of interacting with extension function dispatcher.

BUG=226303
TEST=None

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=199143

[yzshen1, 2013-05-07 19:44:01]

Hi, Raymes.

Would you please take a look? This is just the skeleton. Thanks!

[raymes, 2013-05-08 18:28:56]

lgtm

https://codereview.chromium.org/15039008/diff/1/chrome/browser/renderer_host/...
File
chrome/browser/renderer_host/pepper/pepper_extensions_common_message_filter.cc
(right):

https://codereview.chromium.org/15039008/diff/1/chrome/browser/renderer_host/...
chrome/browser/renderer_host/pepper/pepper_extensions_common_message_filter.cc:19:
PepperExtensionsCommonMessageFilter*
please add a // static comment

[yzshen1, 2013-05-08 19:38:18]

Thanks, Raymes!

https://codereview.chromium.org/15039008/diff/1/chrome/browser/renderer_host/...
File
chrome/browser/renderer_host/pepper/pepper_extensions_common_message_filter.cc
(right):

https://codereview.chromium.org/15039008/diff/1/chrome/browser/renderer_host/...
chrome/browser/renderer_host/pepper/pepper_extensions_common_message_filter.cc:19:
PepperExtensionsCommonMessageFilter*
On 2013/05/08 18:28:56, raymes wrote:
> please add a // static comment

Done.

[commit-bot: I haz the power, 2013-05-08 19:38:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yzshen@chromium.org/15039008/5001

[commit-bot: I haz the power, 2013-05-08 19:38:31]

Failed to apply patch for
chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file
chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
  Hunk #1 FAILED at 5.
  Hunk #2 succeeded at 42 (offset 1 line).
  1 out of 2 hunks FAILED -- saving rejects to file
chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc.rej

Patch:      
chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
Index: chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
diff --git
a/chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
b/chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
index
b90d1f1586a354e725b2c6d32592e98a5790dca8..21626362910497765828bc4cd2512e5b92636972
100644
--- a/chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
+++ b/chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
@@ -5,6 +5,7 @@
 #include
"chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.h"
 
 #include "chrome/browser/renderer_host/pepper/pepper_broker_message_filter.h"
+#include
"chrome/browser/renderer_host/pepper/pepper_extensions_common_message_filter.h"
 #include "chrome/browser/renderer_host/pepper/pepper_flash_browser_host.h"
 #include
"chrome/browser/renderer_host/pepper/pepper_flash_clipboard_message_filter.h"
 #include "chrome/browser/renderer_host/pepper/pepper_flash_device_id_host.h"
@@ -41,6 +42,22 @@ scoped_ptr<ResourceHost>
ChromeBrowserPepperHostFactory::CreateResourceHost(
   if (!host_->IsValidInstance(instance))
     return scoped_ptr<ResourceHost>();
 
+  // Dev interfaces.
+  if (host_->GetPpapiHost()->permissions().HasPermission(
+      ppapi::PERMISSION_DEV)) {
+    switch (message.type()) {
+      case PpapiHostMsg_ExtensionsCommon_Create::ID: {
+        scoped_refptr<ResourceMessageFilter> extensions_common_filter(
+            PepperExtensionsCommonMessageFilter::Create(host_, instance));
+        if (!extensions_common_filter.get())
+          return scoped_ptr<ResourceHost>();
+        return scoped_ptr<ResourceHost>(new MessageFilterHost(
+            host_->GetPpapiHost(), instance, params.pp_resource(),
+            extensions_common_filter));
+      }
+    }
+  }
+
   // Private interfaces.
   if (host_->GetPpapiHost()->permissions().HasPermission(
           ppapi::PERMISSION_PRIVATE)) {

[commit-bot: I haz the power, 2013-05-08 19:47:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yzshen@chromium.org/15039008/13001

[commit-bot: I haz the power, 2013-05-09 06:01:36]

Change committed as 199143