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

Unified Diff: chrome/browser/net/chrome_network_delegate.cc

Issue 7346024: Get rid of the ProfileId. It was added for ceee. I reverted the original change, since it led to ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 5 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
« no previous file with comments | « chrome/browser/net/chrome_network_delegate.h ('k') | chrome/browser/profiles/profile.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/chrome_network_delegate.cc
===================================================================
--- chrome/browser/net/chrome_network_delegate.cc (revision 92309)
+++ chrome/browser/net/chrome_network_delegate.cc (working copy)
@@ -26,14 +26,14 @@
// the proxy extension API.
void ForwardProxyErrors(net::URLRequest* request,
ExtensionEventRouterForwarder* event_router,
- ProfileId profile_id) {
+ void* profile) {
if (request->status().status() == net::URLRequestStatus::FAILED) {
switch (request->status().os_error()) {
case net::ERR_PROXY_AUTH_UNSUPPORTED:
case net::ERR_PROXY_CONNECTION_FAILED:
case net::ERR_TUNNEL_CONNECTION_FAILED:
ExtensionProxyEventRouter::GetInstance()->OnProxyError(
- event_router, profile_id, request->status().os_error());
+ event_router, profile, request->status().os_error());
}
}
}
@@ -43,10 +43,10 @@
ChromeNetworkDelegate::ChromeNetworkDelegate(
ExtensionEventRouterForwarder* event_router,
ExtensionInfoMap* extension_info_map,
- ProfileId profile_id,
+ void* profile,
BooleanPrefMember* enable_referrers)
: event_router_(event_router),
- profile_id_(profile_id),
+ profile_(profile),
extension_info_map_(extension_info_map),
enable_referrers_(enable_referrers) {
DCHECK(event_router);
@@ -71,7 +71,7 @@
if (!enable_referrers_->GetValue())
request->set_referrer(std::string());
return ExtensionWebRequestEventRouter::GetInstance()->OnBeforeRequest(
- profile_id_, extension_info_map_.get(), request, callback, new_url);
+ profile_, extension_info_map_.get(), request, callback, new_url);
}
int ChromeNetworkDelegate::OnBeforeSendHeaders(
@@ -79,7 +79,7 @@
net::CompletionCallback* callback,
net::HttpRequestHeaders* headers) {
return ExtensionWebRequestEventRouter::GetInstance()->OnBeforeSendHeaders(
- profile_id_, extension_info_map_.get(), request, callback, headers);
+ profile_, extension_info_map_.get(), request, callback, headers);
}
void ChromeNetworkDelegate::OnRequestSent(
@@ -87,21 +87,21 @@
const net::HostPortPair& socket_address,
const net::HttpRequestHeaders& headers) {
ExtensionWebRequestEventRouter::GetInstance()->OnRequestSent(
- profile_id_, extension_info_map_.get(), request_id,
+ profile_, extension_info_map_.get(), request_id,
socket_address, headers);
}
void ChromeNetworkDelegate::OnBeforeRedirect(net::URLRequest* request,
const GURL& new_location) {
ExtensionWebRequestEventRouter::GetInstance()->OnBeforeRedirect(
- profile_id_, extension_info_map_.get(), request, new_location);
+ profile_, extension_info_map_.get(), request, new_location);
}
void ChromeNetworkDelegate::OnResponseStarted(net::URLRequest* request) {
ExtensionWebRequestEventRouter::GetInstance()->OnResponseStarted(
- profile_id_, extension_info_map_.get(), request);
- ForwardProxyErrors(request, event_router_.get(), profile_id_);
+ profile_, extension_info_map_.get(), request);
+ ForwardProxyErrors(request, event_router_.get(), profile_);
}
void ChromeNetworkDelegate::OnRawBytesRead(const net::URLRequest& request,
@@ -116,27 +116,27 @@
request->response_headers()->response_code());
if (!is_redirect) {
ExtensionWebRequestEventRouter::GetInstance()->OnCompleted(
- profile_id_, extension_info_map_.get(), request);
+ profile_, extension_info_map_.get(), request);
}
} else if (request->status().status() == net::URLRequestStatus::FAILED) {
ExtensionWebRequestEventRouter::GetInstance()->OnErrorOccurred(
- profile_id_, extension_info_map_.get(), request);
+ profile_, extension_info_map_.get(), request);
}
- ForwardProxyErrors(request, event_router_.get(), profile_id_);
+ ForwardProxyErrors(request, event_router_.get(), profile_);
}
void ChromeNetworkDelegate::OnURLRequestDestroyed(net::URLRequest* request) {
ExtensionWebRequestEventRouter::GetInstance()->OnURLRequestDestroyed(
- profile_id_, request);
+ profile_, request);
}
void ChromeNetworkDelegate::OnHttpTransactionDestroyed(uint64 request_id) {
ExtensionWebRequestEventRouter::GetInstance()->OnHttpTransactionDestroyed(
- profile_id_, request_id);
+ profile_, request_id);
}
void ChromeNetworkDelegate::OnPACScriptError(int line_number,
const string16& error) {
ExtensionProxyEventRouter::GetInstance()->OnPACScriptError(
- event_router_.get(), profile_id_, line_number, error);
+ event_router_.get(), profile_, line_number, error);
}
« no previous file with comments | « chrome/browser/net/chrome_network_delegate.h ('k') | chrome/browser/profiles/profile.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698