Change the permission check for Pepper socket API to support both the public and private APIs.

Change the permission check for Pepper socket API to support both the public and private APIs.

BUG=247225
TEST=None

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

[yzshen1, 2013-06-13 23:54:03]

Hi,

Would you please take a look?

John: The changes to the content API:
content/public/browser/content_browser_client.{h,cc}

Bei: How the extension permission is checked:
chrome/browser/chrome_content_browser_client.cc

Victor: Everything else.

Thanks!

[jam, 2013-06-14 00:20:08]

On 2013/06/13 23:54:03, yzshen1 wrote:
> Hi,
> 
> Would you please take a look?
> 
> John: The changes to the content API:
> content/public/browser/content_browser_client.{h,cc}
lgtm
> 
> Bei: How the extension permission is checked:
> chrome/browser/chrome_content_browser_client.cc
> 
> Victor: Everything else.
> 
> Thanks!

[victorhsieh, 2013-06-14 15:22:07]

lgtm

[yzshen1, 2013-06-14 16:28:13]

Hi, Jonathan.

Would you please do an OWNER review for:
android_webview/browser/aw_content_browser_client.{h,cc}

Thanks!

[joth, 2013-06-14 18:27:56]

lgtm

[Bei Zhang, 2013-06-14 18:31:16]

lgtm

[commit-bot: I haz the power, 2013-06-14 18:38:55]

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

[commit-bot: I haz the power, 2013-06-14 23:14:13]

Retried try job too often on win7_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-06-14 23:17:26]

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

[commit-bot: I haz the power, 2013-06-15 01:26:41]

Change committed as 206533