| Index: trunk/src/chrome/browser/services/gcm/gcm_profile_service.cc
|
| ===================================================================
|
| --- trunk/src/chrome/browser/services/gcm/gcm_profile_service.cc (revision 264317)
|
| +++ trunk/src/chrome/browser/services/gcm/gcm_profile_service.cc (working copy)
|
| @@ -176,8 +176,7 @@
|
| void Send(const std::string& app_id,
|
| const std::string& receiver_id,
|
| const GCMClient::OutgoingMessage& message);
|
| - void GetGCMStatistics(bool clear_logs);
|
| - void SetGCMRecording(bool recording);
|
| + void RequestGCMStatistics();
|
|
|
| // For testing purpose. Can be called from UI thread. Use with care.
|
| GCMClient* gcm_client_for_testing() const { return gcm_client_.get(); }
|
| @@ -373,42 +372,23 @@
|
| gcm_client_->Send(app_id, receiver_id, message);
|
| }
|
|
|
| -void GCMProfileService::IOWorker::GetGCMStatistics(bool clear_logs) {
|
| +void GCMProfileService::IOWorker::RequestGCMStatistics() {
|
| DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
|
| gcm::GCMClient::GCMStatistics stats;
|
|
|
| if (gcm_client_.get()) {
|
| - if (clear_logs)
|
| - gcm_client_->ClearActivityLogs();
|
| + stats.gcm_client_created = true;
|
| stats = gcm_client_->GetStatistics();
|
| }
|
|
|
| content::BrowserThread::PostTask(
|
| content::BrowserThread::UI,
|
| FROM_HERE,
|
| - base::Bind(&GCMProfileService::GetGCMStatisticsFinished,
|
| + base::Bind(&GCMProfileService::RequestGCMStatisticsFinished,
|
| service_,
|
| stats));
|
| }
|
|
|
| -void GCMProfileService::IOWorker::SetGCMRecording(bool recording) {
|
| - DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
|
| - gcm::GCMClient::GCMStatistics stats;
|
| -
|
| - if (gcm_client_.get()) {
|
| - gcm_client_->SetRecording(recording);
|
| - stats = gcm_client_->GetStatistics();
|
| - stats.gcm_client_created = true;
|
| - }
|
| -
|
| - content::BrowserThread::PostTask(
|
| - content::BrowserThread::UI,
|
| - FROM_HERE,
|
| - base::Bind(&GCMProfileService::GetGCMStatisticsFinished,
|
| - service_,
|
| - stats));
|
| -}
|
| -
|
| std::string GCMProfileService::GetGCMEnabledStateString(GCMEnabledState state) {
|
| switch (state) {
|
| case GCMProfileService::ALWAYS_ENABLED:
|
| @@ -708,8 +688,8 @@
|
| return gcm_client_ready_;
|
| }
|
|
|
| -void GCMProfileService::GetGCMStatistics(
|
| - GetGCMStatisticsCallback callback, bool clear_logs) {
|
| +void GCMProfileService::RequestGCMStatistics(
|
| + RequestGCMStatisticsCallback callback) {
|
| DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
|
| DCHECK(!callback.is_null());
|
|
|
| @@ -717,24 +697,10 @@
|
| content::BrowserThread::PostTask(
|
| content::BrowserThread::IO,
|
| FROM_HERE,
|
| - base::Bind(&GCMProfileService::IOWorker::GetGCMStatistics,
|
| - io_worker_,
|
| - clear_logs));
|
| + base::Bind(&GCMProfileService::IOWorker::RequestGCMStatistics,
|
| + io_worker_));
|
| }
|
|
|
| -void GCMProfileService::SetGCMRecording(
|
| - GetGCMStatisticsCallback callback, bool recording) {
|
| - DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
|
| -
|
| - request_gcm_statistics_callback_ = callback;
|
| - content::BrowserThread::PostTask(
|
| - content::BrowserThread::IO,
|
| - FROM_HERE,
|
| - base::Bind(&GCMProfileService::IOWorker::SetGCMRecording,
|
| - io_worker_,
|
| - recording));
|
| -}
|
| -
|
| void GCMProfileService::Observe(int type,
|
| const content::NotificationSource& source,
|
| const content::NotificationDetails& details) {
|
| @@ -938,9 +904,10 @@
|
| return iter == app_handlers_.end() ? &default_app_handler_ : iter->second;
|
| }
|
|
|
| -void GCMProfileService::GetGCMStatisticsFinished(
|
| +void GCMProfileService::RequestGCMStatisticsFinished(
|
| GCMClient::GCMStatistics stats) {
|
| DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
|
| +
|
| request_gcm_statistics_callback_.Run(stats);
|
| }
|
|
|
|
|