When allowing an extension file:/// URL access, grant its process access to that scheme.

When allowing an extension file:/// URL access, grant its process access to that scheme.

This allows XHRs from the background page to file:/// URLs to succeed. XHRs from
content scripts still fail, since they're running in a regular renderer process,
and there is currently no mechanism for only whitelisting the isolated world
within it for the file: scheme.

Also re-enable ExtensionApiTest.CrossOriginXHRContentScript since it doesn't
appear to have flaked recently.

BUG=104547, 96725
R=aa@chromium.org


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110614

[Aaron Boodman, 2011-11-17 21:31:23]

lgtm

http://codereview.chromium.org/8587052/diff/1/chrome/browser/renderer_host/ch...
File chrome/browser/renderer_host/chrome_render_view_host_observer.cc (right):

http://codereview.chromium.org/8587052/diff/1/chrome/browser/renderer_host/ch...
chrome/browser/renderer_host/chrome_render_view_host_observer.cc:119: if (type
== Extension::TYPE_EXTENSION && profile_->GetExtensionService()->
Weird indent - can you put profile on the next line?

[Mihai Parparita -not on Chrome, 2011-11-17 23:14:03]

http://codereview.chromium.org/8587052/diff/1/chrome/browser/renderer_host/ch...
File chrome/browser/renderer_host/chrome_render_view_host_observer.cc (right):

http://codereview.chromium.org/8587052/diff/1/chrome/browser/renderer_host/ch...
chrome/browser/renderer_host/chrome_render_view_host_observer.cc:119: if (type
== Extension::TYPE_EXTENSION && profile_->GetExtensionService()->
On 2011/11/17 21:31:23, Aaron Boodman wrote:
> Weird indent - can you put profile on the next line?

Done.

[commit-bot: I haz the power, 2011-11-17 23:17:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mihaip@chromium.org/8587052/3001

[commit-bot: I haz the power, 2011-11-18 00:32:58]

Change committed as 110614