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

Unified Diff: chrome/browser/extensions/api/web_request/web_request_api.cc

Issue 10944013: Make original_response_headers parameter of NetworkDelegate::OnHeadersReceived const (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merged with ToT Created 8 years, 2 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/extensions/api/web_request/web_request_api.cc
diff --git a/chrome/browser/extensions/api/web_request/web_request_api.cc b/chrome/browser/extensions/api/web_request/web_request_api.cc
index e42452738d37b7f89ba39c8a9238c2cd3df5326d..4b0f4ec5fc4b43b7324aec313fbf7bf1ff06de0a 100644
--- a/chrome/browser/extensions/api/web_request/web_request_api.cc
+++ b/chrome/browser/extensions/api/web_request/web_request_api.cc
@@ -355,7 +355,7 @@ struct ExtensionWebRequestEventRouter::BlockedRequest {
// The response headers that were received from the server. Only valid for
// OnHeadersReceived.
- scoped_refptr<net::HttpResponseHeaders> original_response_headers;
+ scoped_refptr<const net::HttpResponseHeaders> original_response_headers;
// Location where to override response headers. Only valid for
// OnHeadersReceived.
@@ -669,7 +669,7 @@ int ExtensionWebRequestEventRouter::OnHeadersReceived(
ExtensionInfoMap* extension_info_map,
net::URLRequest* request,
const net::CompletionCallback& callback,
- net::HttpResponseHeaders* original_response_headers,
+ const net::HttpResponseHeaders* original_response_headers,
scoped_refptr<net::HttpResponseHeaders>* override_response_headers) {
// We hide events from the system context as well as sensitive requests.
if (!profile ||
@@ -1295,7 +1295,7 @@ helpers::EventResponseDelta* CalculateDelta(
response->cancel, old_headers, new_headers);
}
case ExtensionWebRequestEventRouter::kOnHeadersReceived: {
- net::HttpResponseHeaders* old_headers =
+ const net::HttpResponseHeaders* old_headers =
blocked_request->original_response_headers.get();
helpers::ResponseHeaders* new_headers =
response->response_headers.get();
@@ -1476,7 +1476,7 @@ bool ExtensionWebRequestEventRouter::ProcessDeclarativeRules(
const std::string& event_name,
net::URLRequest* request,
extensions::RequestStage request_stage,
- net::HttpResponseHeaders* original_response_headers) {
+ const net::HttpResponseHeaders* original_response_headers) {
// Rules of the current |profile| may apply but we need to check also whether
// there are applicable rules from extensions whose background page
// spans from regular to incognito mode.

Powered by Google App Engine
This is Rietveld 408576698