Ensure ChromeURLRequestContext finds out first about extension loading

Ensure ChromeURLRequestContext finds out first about extension loading.

This prevents races from arising where extension renderers may try to navigate to extension urls before the request context knows how to map the urls to extension resources.

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

[Matt Perry, 2009-10-01 22:04:25]

http://codereview.chromium.org/255047/diff/1/6
File chrome/browser/net/chrome_url_request_context.h (right):

http://codereview.chromium.org/255047/diff/1/6#newcode87
Line 87: void OnNewExtensions(ExtensionPaths* new_paths);
I think this takes a map for legacy reasons. Now it looks like it only ever
handles 1 extension at a time, so let's change it to:
  OnLoadedExtension(string id, filepath path);

[rafaelw, 2009-10-01 22:32:45]

http://codereview.chromium.org/255047/diff/1/6
File chrome/browser/net/chrome_url_request_context.h (right):

http://codereview.chromium.org/255047/diff/1/6#newcode87
Line 87: void OnNewExtensions(ExtensionPaths* new_paths);
On 2009/10/01 22:04:25, Matt Perry wrote:
> I think this takes a map for legacy reasons. Now it looks like it only ever
> handles 1 extension at a time, so let's change it to:
>   OnLoadedExtension(string id, filepath path);

Done.

[Matt Perry, 2009-10-01 22:33:59]

ah, much better. LGTM

http://codereview.chromium.org/255047/diff/4001/4005
File chrome/browser/net/chrome_url_request_context.cc (right):

http://codereview.chromium.org/255047/diff/4001/4005#newcode523
Line 523: if (!is_off_the_record_) {
nit: remove braces