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

Side by Side Diff: components/web_resource/web_resource_service.cc

Issue 900633002: Add missing locale parameter to promo service. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 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 "components/web_resource/web_resource_service.h" 5 #include "components/web_resource/web_resource_service.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "base/prefs/pref_service.h" 9 #include "base/prefs/pref_service.h"
10 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 prefs_->SetString(last_update_time_pref_name_, 100 prefs_->SetString(last_update_time_pref_name_,
101 base::DoubleToString(base::Time::Now().ToDoubleT())); 101 base::DoubleToString(base::Time::Now().ToDoubleT()));
102 102
103 // If we are still fetching data, exit. 103 // If we are still fetching data, exit.
104 if (in_fetch_) 104 if (in_fetch_)
105 return; 105 return;
106 in_fetch_ = true; 106 in_fetch_ = true;
107 107
108 GURL web_resource_server = 108 GURL web_resource_server =
109 application_locale_.empty() 109 application_locale_.empty()
110 ? google_util::AppendGoogleLocaleParam(web_resource_server_, 110 ? web_resource_server_
111 application_locale_) 111 : google_util::AppendGoogleLocaleParam(web_resource_server_,
112 : web_resource_server_; 112 application_locale_);
113 113
114 DVLOG(1) << "WebResourceService StartFetch " << web_resource_server; 114 DVLOG(1) << "WebResourceService StartFetch " << web_resource_server;
115 url_fetcher_.reset( 115 url_fetcher_.reset(
116 net::URLFetcher::Create(web_resource_server, net::URLFetcher::GET, this)); 116 net::URLFetcher::Create(web_resource_server, net::URLFetcher::GET, this));
117 // Do not let url fetcher affect existing state in system context 117 // Do not let url fetcher affect existing state in system context
118 // (by setting cookies, for example). 118 // (by setting cookies, for example).
119 url_fetcher_->SetLoadFlags(net::LOAD_DISABLE_CACHE | 119 url_fetcher_->SetLoadFlags(net::LOAD_DISABLE_CACHE |
120 net::LOAD_DO_NOT_SEND_COOKIES | 120 net::LOAD_DO_NOT_SEND_COOKIES |
121 net::LOAD_DO_NOT_SAVE_COOKIES); 121 net::LOAD_DO_NOT_SAVE_COOKIES);
122 url_fetcher_->SetRequestContext(request_context_.get()); 122 url_fetcher_->SetRequestContext(request_context_.get());
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
166 // Wait at least |start_fetch_delay_ms_|. 166 // Wait at least |start_fetch_delay_ms_|.
167 if (ms_until_update > start_fetch_delay_ms_) 167 if (ms_until_update > start_fetch_delay_ms_)
168 delay = ms_until_update; 168 delay = ms_until_update;
169 } 169 }
170 } 170 }
171 // Start fetch and wait for UpdateResourceCache. 171 // Start fetch and wait for UpdateResourceCache.
172 ScheduleFetch(delay); 172 ScheduleFetch(delay);
173 } 173 }
174 174
175 } // namespace web_resource 175 } // namespace web_resource
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698