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

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

Issue 1684123004: Bypass the DataReductionProxy for all POST requests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Resubmitting with upstream branch set to branch from issue 1680893002 Created 4 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/net/predictor.cc
diff --git a/chrome/browser/net/predictor.cc b/chrome/browser/net/predictor.cc
index 25a309a92a68d06e11ddf253de79a27129a63298..203e42c4851356e4b5764887c0c3057c23d1dfa1 100644
--- a/chrome/browser/net/predictor.cc
+++ b/chrome/browser/net/predictor.cc
@@ -1009,21 +1009,21 @@ void Predictor::LookupFinished(LookupRequest* request, const GURL& url,
info->SetNoSuchNameState();
}
}
bool Predictor::WouldLikelyProxyURL(const GURL& url) {
if (!proxy_service_)
return false;
net::ProxyInfo info;
bool synchronous_success = proxy_service_->TryResolveProxySynchronously(
- url, net::LOAD_NORMAL, &info, NULL, net::BoundNetLog());
+ url, net::LOAD_NORMAL, &info, nullptr, "", net::BoundNetLog());
bengr 2016/02/16 20:00:13 Since this entire file uses NULL, I would keep the
RyanSturm 2016/02/17 21:46:10 Done.
return synchronous_success && !info.is_direct();
}
UrlInfo* Predictor::AppendToResolutionQueue(
const GURL& url,
UrlInfo::ResolutionMotivation motivation) {
DCHECK_CURRENTLY_ON(BrowserThread::IO);
DCHECK(url.has_host());

Powered by Google App Engine
This is Rietveld 408576698