Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4433)

Unified Diff: chrome/renderer/extensions/extension_resource_request_policy.cc

Issue 10821133: Move c/r/extensions/* into extensions namespace (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Latest master for cq Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/extensions/extension_resource_request_policy.cc
diff --git a/chrome/renderer/extensions/extension_resource_request_policy.cc b/chrome/renderer/extensions/extension_resource_request_policy.cc
deleted file mode 100644
index 9f1e8428b3fdd523a22297231fcbad0c2d721a9a..0000000000000000000000000000000000000000
--- a/chrome/renderer/extensions/extension_resource_request_policy.cc
+++ /dev/null
@@ -1,109 +0,0 @@
-// 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/extension_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/url_constants.h"
-#include "chrome/common/extensions/extension.h"
-#include "chrome/common/extensions/extension_set.h"
-#include "googleurl/src/gurl.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"
-#include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebString.h"
-
-using extensions::Extension;
-
-// static
-bool ExtensionResourceRequestPolicy::CanRequestResource(
- const GURL& resource_url,
- WebKit::WebFrame* frame,
- const ExtensionSet* loaded_extensions) {
- CHECK(resource_url.SchemeIs(chrome::kExtensionScheme));
-
- const Extension* extension =
- loaded_extensions->GetExtensionOrAppByURL(ExtensionURLInfo(resource_url));
- if (!extension) {
- // Allow the load in the case of a non-existent extension. We'll just get a
- // 404 from the browser process.
- return true;
- }
-
- // Disallow loading of packaged resources for hosted apps. We don't allow
- // hybrid hosted/packaged apps. The one exception is access to icons, since
- // some extensions want to be able to do things like create their own
- // 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
- // as web accessible if the manifest version is 2 or greater.
- if (!extension->IsResourceWebAccessible(resource_url.path()) &&
- !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)
- bool is_own_resource = frame_url.GetOrigin() == extension->url() ||
- page_url.GetOrigin() == extension->url();
- // - devtools (chrome-extension:// URLs are loaded into frames of devtools
- // to support the devtools extension APIs)
- bool is_dev_tools = page_url.SchemeIs(chrome::kChromeDevToolsScheme) &&
- !extension->devtools_url().is_empty();
-
- if (!is_empty_origin && !is_own_resource && !is_dev_tools) {
- std::string message = base::StringPrintf(
- "Denying load of %s. Resources must be listed in the "
- "web_accessible_resources manifest key in order to be loaded by "
- "pages outside the extension.",
- resource_url.spec().c_str());
- frame->addMessageToConsole(
- WebKit::WebConsoleMessage(WebKit::WebConsoleMessage::LevelError,
- WebKit::WebString::fromUTF8(message)));
- return false;
- }
- }
-
- return true;
-}
-
-// static
-bool ExtensionResourceRequestPolicy::CanRequestExtensionResourceScheme(
- const GURL& resource_url,
- WebKit::WebFrame* frame) {
- CHECK(resource_url.SchemeIs(chrome::kExtensionResourceScheme));
-
- GURL frame_url = frame->document().url();
- if (!frame_url.is_empty() &&
- !frame_url.SchemeIs(chrome::kExtensionScheme)) {
- std::string message = base::StringPrintf(
- "Denying load of %s. chrome-extension-resources:// can only be "
- "loaded from extensions.",
- resource_url.spec().c_str());
- frame->addMessageToConsole(
- WebKit::WebConsoleMessage(WebKit::WebConsoleMessage::LevelError,
- WebKit::WebString::fromUTF8(message)));
- return false;
- }
-
- return true;
-}
-
-ExtensionResourceRequestPolicy::ExtensionResourceRequestPolicy() {
-}
« no previous file with comments | « chrome/renderer/extensions/extension_resource_request_policy.h ('k') | chrome/renderer/extensions/json_schema_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698