move 'web_accessible_resources' parsing out of Extension class.

chrome/renderer/extensions/resource_request_policy.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/renderer/extensions/resource_request_policy.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/stringprintf.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_set.h"
 #include "chrome/common/extensions/manifest_url_handler.h"
+#include "chrome/common/extensions/manifest_web_handler.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/common/page_transition_types.h"
 #include "extensions/common/constants.h"
 #include "googleurl/src/gurl.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebString.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebConsoleMessage.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebDocument.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
 
 namespace extensions {
@@ skipping 20 matching lines @@
   // launchers.
   std::string resource_root_relative_path =
       resource_url.path().empty() ? "" : resource_url.path().substr(1);
   if (extension->is_hosted_app() &&
       !extension->icons().ContainsPath(resource_root_relative_path)) {
     LOG(ERROR) << "Denying load of " << resource_url.spec() << " from "
                << "hosted app.";
     return false;
   }
 
   // Disallow loading of extension resources which are not explicitely listed

[Yoyo Zhou, 2013/01/10 00:37:58]

nit: explicitely->explicitly

[Joe Thomas, 2013/01/10 20:56:18]

Done.

   // as web accessible if the manifest version is 2 or greater.
-  if (!extension->IsResourceWebAccessible(resource_url.path()) &&
+  if (!WebAccessibleResourcesInfo::IsResourceWebAccessible(
+          resource_url.path(), extension) &&
       !CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kDisableExtensionsResourceWhitelist)) {
     GURL frame_url = frame->document().url();
     GURL page_url = frame->top()->document().url();
 
     // Exceptions are:
     // - empty origin (needed for some edge cases when we have empty origins)
     bool is_empty_origin = frame_url.is_empty();
     // - extensions requesting their own resources (frame_url check is for
     //     images, page_url check is for iframes)
@@ skipping 45 matching lines @@
     return false;
   }
 
   return true;
 }
 
 ResourceRequestPolicy::ResourceRequestPolicy() {
 }
 
 }  // namespace extensions