ppapi: PPB_VpnProvider: Implement permission check

chrome/browser/plugins/chrome_content_browser_client_plugins_part.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 CHROME_BROWSER_PLUGINS_CHROME_CONTENT_BROWSER_CLIENT_PLUGINS_PART_H_
 #define CHROME_BROWSER_PLUGINS_CHROME_CONTENT_BROWSER_CLIENT_PLUGINS_PART_H_
 
 #include "base/macros.h"
 #include "chrome/browser/chrome_content_browser_client_parts.h"
 #include "content/public/browser/browser_ppapi_host.h"
@@ skipping 14 matching lines @@
       const GURL& url,
       const std::set<std::string>& allowed_dev_channel_origins);
 
   static bool AllowPepperSocketAPI(
       content::BrowserContext* browser_context,
       const GURL& url,
       bool private_api,
       const content::SocketPermissionRequest* params,
       const std::set<std::string>& allowed_socket_origin);
 
+  static bool AllowPepperVpnProviderAPI(

[sky, 2016/05/17 19:08:37]

This sounds like it's allowing something, vs querying if something is enabled.
How about IsPepperVpnProviderAPIAllowed() ?

[adrian.belgun, 2016/05/17 20:13:32]

In this patch I was closely following the naming scheme and behavior of the
above AllowPepperSocketAPI(). Anyway, the new name works for me.

Changed as per request.

+      content::BrowserContext* browser_context,
+      const GURL& url);
+
   static bool IsPluginAllowedToCallRequestOSFileHandle(
       content::BrowserContext* browser_context,
       const GURL& url,
       const std::set<std::string>& allowed_file_handle_origins);
 
   static void DidCreatePpapiPlugin(content::BrowserPpapiHost* browser_host);
 
  private:
   // ChromeContentBrowserClientParts:
   void RenderProcessWillLaunch(content::RenderProcessHost* host) override;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeContentBrowserClientPluginsPart);
 };
 
 }  // namespace plugins
 
 #endif  // CHROME_BROWSER_PLUGINS_CHROME_CONTENT_BROWSER_CLIENT_PLUGINS_PART_H_