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

Unified Diff: chrome/browser/renderer_host/resource_dispatcher_host.cc

Issue 661178: Continue removing bad dependency of chrome/common on chrome/browser... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years, 10 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/browser/renderer_host/resource_dispatcher_host.cc
===================================================================
--- chrome/browser/renderer_host/resource_dispatcher_host.cc (revision 40221)
+++ chrome/browser/renderer_host/resource_dispatcher_host.cc (working copy)
@@ -51,9 +51,9 @@
#include "chrome/browser/ssl/ssl_manager.h"
#include "chrome/browser/worker_host/worker_service.h"
#include "chrome/common/chrome_switches.h"
-#include "chrome/common/extensions/extension_l10n_util.h"
#include "chrome/common/notification_service.h"
#include "chrome/common/render_messages.h"
+#include "chrome/common/url_constants.h"
#include "net/base/auth.h"
#include "net/base/cert_status_flags.h"
#include "net/base/load_flags.h"
@@ -467,8 +467,9 @@
ResourceType::IsFrame(request_data.resource_type), // allow_download
request_data.host_renderer_id,
request_data.host_render_view_id);
- extension_l10n_util::ApplyMessageFilterPolicy(
- request_data.url, request_data.resource_type, extra_info);
+ ApplyExtensionMessageFilterPolicy(request_data.url,
+ request_data.resource_type,
+ extra_info);
SetRequestInfo(request, extra_info); // Request takes ownership.
chrome_browser_net::SetOriginProcessUniqueIDForRequest(
request_data.origin_child_id, request);
@@ -1738,6 +1739,7 @@
delete requests;
}
+// static
bool ResourceDispatcherHost::IsResourceDispatcherHostMessage(
const IPC::Message& message) {
switch (message.type()) {
@@ -1757,3 +1759,17 @@
return false;
}
+
+// static
+void ResourceDispatcherHost::ApplyExtensionMessageFilterPolicy(
+ const GURL& url,
+ const ResourceType::Type& resource_type,
+ ResourceDispatcherHostRequestInfo* request_info) {
+ // Apply filter only to chrome extension css files that don't have
+ // security filter already set.
+ if (url.SchemeIs(chrome::kExtensionScheme) &&
+ request_info->filter_policy() == FilterPolicy::DONT_FILTER &&
+ resource_type == ResourceType::STYLESHEET) {
+ request_info->set_filter_policy(FilterPolicy::FILTER_EXTENSION_MESSAGES);
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698