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

Unified Diff: google_apis/gcm/gcm_client_impl.cc

Issue 288433002: G-services settings v3 implementation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressing feedback from Jian Li Created 6 years, 7 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: google_apis/gcm/gcm_client_impl.cc
diff --git a/google_apis/gcm/gcm_client_impl.cc b/google_apis/gcm/gcm_client_impl.cc
index ed9838c9b4e4728eb6cce6264a2cfff8d4daac4f..ec07effc679a7cfaf7057436380468e00d7a5719 100644
--- a/google_apis/gcm/gcm_client_impl.cc
+++ b/google_apis/gcm/gcm_client_impl.cc
@@ -268,8 +268,8 @@ void GCMClientImpl::OnLoadCompleted(scoped_ptr<GCMStore::LoadResult> result) {
void GCMClientImpl::InitializeMCSClient(
scoped_ptr<GCMStore::LoadResult> result) {
std::vector<GURL> endpoints;
- endpoints.push_back(gservices_settings_.mcs_main_endpoint());
- endpoints.push_back(gservices_settings_.mcs_fallback_endpoint());
+ endpoints.push_back(gservices_settings_.GetMCSMainEndpoint());
+ endpoints.push_back(gservices_settings_.GetMCSFallbackEndpoint());
connection_factory_ = internals_builder_->BuildConnectionFactory(
endpoints,
kDefaultBackoffPolicy,
@@ -336,7 +336,7 @@ void GCMClientImpl::StartCheckin() {
account_ids_,
chrome_build_proto_);
checkin_request_.reset(
- new CheckinRequest(gservices_settings_.checkin_url(),
+ new CheckinRequest(gservices_settings_.GetCheckinURL(),
request_info,
kDefaultBackoffPolicy,
base::Bind(&GCMClientImpl::OnCheckinCompleted,
@@ -375,7 +375,7 @@ void GCMClientImpl::OnCheckinCompleted(
// First update G-services settings, as something might have changed.
if (gservices_settings_.UpdateFromCheckinResponse(checkin_response)) {
gcm_store_->SetGServicesSettings(
- gservices_settings_.GetSettingsMap(),
+ gservices_settings_.get_settings_map(),
gservices_settings_.digest(),
base::Bind(&GCMClientImpl::SetGServicesSettingsCallback,
weak_ptr_factory_.GetWeakPtr()));
@@ -415,7 +415,7 @@ void GCMClientImpl::SchedulePeriodicCheckin() {
}
base::TimeDelta GCMClientImpl::GetTimeToNextCheckin() const {
- return last_checkin_time_ + gservices_settings_.checkin_interval() -
+ return last_checkin_time_ + gservices_settings_.GetCheckinInterval() -
clock_->Now();
}
@@ -473,7 +473,7 @@ void GCMClientImpl::Register(const std::string& app_id,
DCHECK_EQ(0u, pending_registration_requests_.count(app_id));
RegistrationRequest* registration_request =
- new RegistrationRequest(gservices_settings_.registration_url(),
+ new RegistrationRequest(gservices_settings_.GetRegistrationURL(),
request_info,
kDefaultBackoffPolicy,
base::Bind(&GCMClientImpl::OnRegisterCompleted,
@@ -547,16 +547,15 @@ void GCMClientImpl::Unregister(const std::string& app_id) {
device_checkin_info_.secret,
app_id);
- UnregistrationRequest* unregistration_request =
- new UnregistrationRequest(
- gservices_settings_.registration_url(),
- request_info,
- kDefaultBackoffPolicy,
- base::Bind(&GCMClientImpl::OnUnregisterCompleted,
- weak_ptr_factory_.GetWeakPtr(),
- app_id),
- url_request_context_getter_,
- &recorder_);
+ UnregistrationRequest* unregistration_request = new UnregistrationRequest(
+ gservices_settings_.GetRegistrationURL(),
+ request_info,
+ kDefaultBackoffPolicy,
+ base::Bind(&GCMClientImpl::OnUnregisterCompleted,
+ weak_ptr_factory_.GetWeakPtr(),
+ app_id),
+ url_request_context_getter_,
+ &recorder_);
pending_unregistration_requests_[app_id] = unregistration_request;
unregistration_request->Start();
}

Powered by Google App Engine
This is Rietveld 408576698