Add schema chrome-extension-resource:// for extension resources

chrome/browser/extensions/extension_protocols.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/extensions/extension_protocols.h"
 
 #include <algorithm>
 
 #include "base/compiler_specific.h"
 #include "base/file_path.h"
@@ skipping 222 matching lines @@
 
   virtual net::URLRequestJob* MaybeCreateJob(
       net::URLRequest* request) const OVERRIDE;
 
  private:
   const bool is_incognito_;
   ExtensionInfoMap* const extension_info_map_;
   DISALLOW_COPY_AND_ASSIGN(ExtensionProtocolHandler);
 };
 
-// Creates URLRequestJobs for extension:// URLs.
+// Creates URLRequestJobs for chrome-extension:// URLs.
 net::URLRequestJob*
 ExtensionProtocolHandler::MaybeCreateJob(net::URLRequest* request) const {
   // TODO(mpcomplete): better error code.
   if (!AllowExtensionResourceLoad(
            request, is_incognito_, extension_info_map_)) {
     return new net::URLRequestErrorJob(request, net::ERR_ADDRESS_UNREACHABLE);
   }
 
   // chrome-extension://extension-id/resource/path.js
   const std::string& extension_id = request->url().host();
@@ skipping 61 matching lines @@
     // Getting the file path will touch the file system.  Fixing
     // crbug.com/59849 would also fix this.  Suppress the error for now.
     base::ThreadRestrictions::ScopedAllowIO allow_io;
     resource_file_path = resource.GetFilePath();
   }
 
   return new URLRequestExtensionJob(request, resource_file_path,
                                     content_security_policy, send_cors_header);
 }
 
+class ExtensionResourceProtocolHandler
+    : public net::URLRequestJobFactory::ProtocolHandler {
+ public:
+  ExtensionResourceProtocolHandler() {}
+  virtual ~ExtensionResourceProtocolHandler() {}
+
+  virtual net::URLRequestJob* MaybeCreateJob(
+      net::URLRequest* request) const OVERRIDE;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ExtensionResourceProtocolHandler);
+};
+
+// Creates URLRequestJobs for chrome-extension-resource:// URLs.
+net::URLRequestJob*
+ExtensionResourceProtocolHandler::MaybeCreateJob(
+    net::URLRequest* request) const {
+  if (!request->url().has_host() || !request->url().has_path())
+    return NULL;
+
+  FilePath path;
+  PathService::Get(chrome::DIR_RESOURCES_EXTENSION, &path);

[miket_OOO, 2012/04/12 18:38:39]

Is this the EXT from the other CL? Did you want it to mean "extra" or
"extension"?

[Peng, 2012/04/12 20:13:22]

I renamed it from EXT to extension. Because it is for
chrome-externsion-resource.

And this CL replaces the old CL, because of some bug of codereview web site. I
can not upload the old CL now. :(

+  path = path.Append(request->url().host()).Append(request->url().path());

[miket_OOO, 2012/04/12 18:38:39]

I'm not sure whether request has already been sanitized or whether it's even a
potential user input, but please make sure it's safe to build a path out of
components taken from it.

[Peng, 2012/04/12 20:13:22]

Done

+
+  return new net::URLRequestFileJob(request, path);
+}
+
 }  // namespace
 
 net::URLRequestJobFactory::ProtocolHandler* CreateExtensionProtocolHandler(
     bool is_incognito,
     ExtensionInfoMap* extension_info_map) {
   return new ExtensionProtocolHandler(is_incognito, extension_info_map);
 }
+
+net::URLRequestJobFactory::ProtocolHandler*
+CreateExtensionResourceProtocolHandler(
+    bool is_incognito,
+    ExtensionInfoMap* extension_info_map) {
+  return new ExtensionResourceProtocolHandler();

[miket_OOO, 2012/04/12 18:38:39]

Is there a reason these arguments need to be in the signature, but unused in the
method body?

[Peng, 2012/04/12 20:13:22]

I copied & pasted it. I think it may be useful in future. Anyway I removed it
for now.

done

+}