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

Side by Side Diff: chrome/browser/extensions/api/declarative_webrequest/webrequest_action.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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/extensions/api/declarative_webrequest/webrequest_action .h" 5 #include "chrome/browser/extensions/api/declarative_webrequest/webrequest_action .h"
6 6
7 #include <limits> 7 #include <limits>
8 8
9 #include "base/lazy_instance.h" 9 #include "base/lazy_instance.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 764 matching lines...) Expand 10 before | Expand all | Expand 10 after
775 WebRequestAddResponseHeaderAction::GetType() const { 775 WebRequestAddResponseHeaderAction::GetType() const {
776 return WebRequestAction::ACTION_ADD_RESPONSE_HEADER; 776 return WebRequestAction::ACTION_ADD_RESPONSE_HEADER;
777 } 777 }
778 778
779 LinkedPtrEventResponseDelta 779 LinkedPtrEventResponseDelta
780 WebRequestAddResponseHeaderAction::CreateDelta( 780 WebRequestAddResponseHeaderAction::CreateDelta(
781 const WebRequestRule::RequestData& request_data, 781 const WebRequestRule::RequestData& request_data,
782 const std::string& extension_id, 782 const std::string& extension_id,
783 const base::Time& extension_install_time) const { 783 const base::Time& extension_install_time) const {
784 CHECK(request_data.stage & GetStages()); 784 CHECK(request_data.stage & GetStages());
785 net::HttpResponseHeaders* headers = 785 const net::HttpResponseHeaders* headers =
786 request_data.original_response_headers; 786 request_data.original_response_headers;
787 if (!headers) 787 if (!headers)
788 return LinkedPtrEventResponseDelta(NULL); 788 return LinkedPtrEventResponseDelta(NULL);
789 789
790 // Don't generate the header if it exists already. 790 // Don't generate the header if it exists already.
791 if (headers->HasHeaderValue(name_, value_)) 791 if (headers->HasHeaderValue(name_, value_))
792 return LinkedPtrEventResponseDelta(NULL); 792 return LinkedPtrEventResponseDelta(NULL);
793 793
794 LinkedPtrEventResponseDelta result( 794 LinkedPtrEventResponseDelta result(
795 new helpers::EventResponseDelta(extension_id, extension_install_time)); 795 new helpers::EventResponseDelta(extension_id, extension_install_time));
(...skipping 24 matching lines...) Expand all
820 WebRequestRemoveResponseHeaderAction::GetType() const { 820 WebRequestRemoveResponseHeaderAction::GetType() const {
821 return WebRequestAction::ACTION_REMOVE_RESPONSE_HEADER; 821 return WebRequestAction::ACTION_REMOVE_RESPONSE_HEADER;
822 } 822 }
823 823
824 LinkedPtrEventResponseDelta 824 LinkedPtrEventResponseDelta
825 WebRequestRemoveResponseHeaderAction::CreateDelta( 825 WebRequestRemoveResponseHeaderAction::CreateDelta(
826 const WebRequestRule::RequestData& request_data, 826 const WebRequestRule::RequestData& request_data,
827 const std::string& extension_id, 827 const std::string& extension_id,
828 const base::Time& extension_install_time) const { 828 const base::Time& extension_install_time) const {
829 CHECK(request_data.stage & GetStages()); 829 CHECK(request_data.stage & GetStages());
830 net::HttpResponseHeaders* headers = 830 const net::HttpResponseHeaders* headers =
831 request_data.original_response_headers; 831 request_data.original_response_headers;
832 if (!headers) 832 if (!headers)
833 return LinkedPtrEventResponseDelta(NULL); 833 return LinkedPtrEventResponseDelta(NULL);
834 834
835 LinkedPtrEventResponseDelta result( 835 LinkedPtrEventResponseDelta result(
836 new helpers::EventResponseDelta(extension_id, extension_install_time)); 836 new helpers::EventResponseDelta(extension_id, extension_install_time));
837 void* iter = NULL; 837 void* iter = NULL;
838 std::string current_value; 838 std::string current_value;
839 while (headers->EnumerateHeader(&iter, name_, &current_value)) { 839 while (headers->EnumerateHeader(&iter, name_, &current_value)) {
840 if (has_value_ && 840 if (has_value_ &&
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
945 CHECK(request_data.stage & GetStages()); 945 CHECK(request_data.stage & GetStages());
946 LinkedPtrEventResponseDelta result( 946 LinkedPtrEventResponseDelta result(
947 new extension_web_request_api_helpers::EventResponseDelta( 947 new extension_web_request_api_helpers::EventResponseDelta(
948 extension_id, extension_install_time)); 948 extension_id, extension_install_time));
949 result->response_cookie_modifications.push_back( 949 result->response_cookie_modifications.push_back(
950 response_cookie_modification_); 950 response_cookie_modification_);
951 return result; 951 return result;
952 } 952 }
953 953
954 } // namespace extensions 954 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698