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

Side by Side Diff: chrome/browser/drive/drive_notification_manager.cc

Issue 15580002: Make use of InvalidationService (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 7 years, 6 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "chrome/browser/drive/drive_notification_manager.h" 5 #include "chrome/browser/drive/drive_notification_manager.h"
6 6
7 #include "base/metrics/histogram.h" 7 #include "base/metrics/histogram.h"
8 #include "chrome/browser/drive/drive_notification_observer.h" 8 #include "chrome/browser/drive/drive_notification_observer.h"
9 #include "chrome/browser/invalidation/invalidation_service.h"
10 #include "chrome/browser/invalidation/invalidation_service_factory.h"
9 #include "chrome/browser/profiles/profile.h" 11 #include "chrome/browser/profiles/profile.h"
10 #include "chrome/browser/sync/profile_sync_service.h"
11 #include "chrome/browser/sync/profile_sync_service_factory.h"
12 #include "google/cacheinvalidation/types.pb.h" 12 #include "google/cacheinvalidation/types.pb.h"
13 13
14 namespace drive { 14 namespace drive {
15 15
16 namespace { 16 namespace {
17 17
18 // The polling interval time is used when XMPP is disabled. 18 // The polling interval time is used when XMPP is disabled.
19 const int kFastPollingIntervalInSecs = 60; 19 const int kFastPollingIntervalInSecs = 60;
20 20
21 // The polling interval time is used when XMPP is enabled. Theoretically 21 // The polling interval time is used when XMPP is enabled. Theoretically
(...skipping 13 matching lines...) Expand all
35 polling_timer_(true /* retain_user_task */, false /* is_repeating */), 35 polling_timer_(true /* retain_user_task */, false /* is_repeating */),
36 weak_ptr_factory_(this) { 36 weak_ptr_factory_(this) {
37 RegisterDriveNotifications(); 37 RegisterDriveNotifications();
38 RestartPollingTimer(); 38 RestartPollingTimer();
39 } 39 }
40 40
41 DriveNotificationManager::~DriveNotificationManager() {} 41 DriveNotificationManager::~DriveNotificationManager() {}
42 42
43 void DriveNotificationManager::Shutdown() { 43 void DriveNotificationManager::Shutdown() {
44 // Unregister for Drive notifications. 44 // Unregister for Drive notifications.
45 ProfileSyncService* profile_sync_service = 45 invalidation::InvalidationService* invalidation_service =
46 ProfileSyncServiceFactory::GetForProfile(profile_); 46 invalidation::InvalidationServiceFactory::GetForProfile(profile_);
47 if (!profile_sync_service || !push_notification_registered_) { 47 if (!invalidation_service || !push_notification_registered_) {
48 return; 48 return;
49 } 49 }
50 50
51 profile_sync_service->UpdateRegisteredInvalidationIds( 51 // We unregister the handler without updating unregistering our IDs on
52 this, syncer::ObjectIdSet()); 52 // purpose. See the class comment on the InvalidationService interface for
53 profile_sync_service->UnregisterInvalidationHandler(this); 53 // more information.
54 invalidation_service->UnregisterInvalidationHandler(this);
54 } 55 }
55 56
56 void DriveNotificationManager::OnInvalidatorStateChange( 57 void DriveNotificationManager::OnInvalidatorStateChange(
57 syncer::InvalidatorState state) { 58 syncer::InvalidatorState state) {
58 push_notification_enabled_ = (state == syncer::INVALIDATIONS_ENABLED); 59 push_notification_enabled_ = (state == syncer::INVALIDATIONS_ENABLED);
59 if (push_notification_enabled_) { 60 if (push_notification_enabled_) {
60 DVLOG(1) << "XMPP Notifications enabled"; 61 DVLOG(1) << "XMPP Notifications enabled";
61 } else { 62 } else {
62 DVLOG(1) << "XMPP Notifications disabled (state=" << state << ")"; 63 DVLOG(1) << "XMPP Notifications disabled (state=" << state << ")";
63 } 64 }
64 FOR_EACH_OBSERVER(DriveNotificationObserver, observers_, 65 FOR_EACH_OBSERVER(DriveNotificationObserver, observers_,
65 OnPushNotificationEnabled(push_notification_enabled_)); 66 OnPushNotificationEnabled(push_notification_enabled_));
66 } 67 }
67 68
68 void DriveNotificationManager::OnIncomingInvalidation( 69 void DriveNotificationManager::OnIncomingInvalidation(
69 const syncer::ObjectIdInvalidationMap& invalidation_map) { 70 const syncer::ObjectIdInvalidationMap& invalidation_map) {
70 DVLOG(2) << "XMPP Drive Notification Received"; 71 DVLOG(2) << "XMPP Drive Notification Received";
71 DCHECK_EQ(1U, invalidation_map.size()); 72 DCHECK_EQ(1U, invalidation_map.size());
72 const invalidation::ObjectId object_id( 73 const invalidation::ObjectId object_id(
73 ipc::invalidation::ObjectSource::COSMO_CHANGELOG, 74 ipc::invalidation::ObjectSource::COSMO_CHANGELOG,
74 kDriveInvalidationObjectId); 75 kDriveInvalidationObjectId);
75 DCHECK_EQ(1U, invalidation_map.count(object_id)); 76 DCHECK_EQ(1U, invalidation_map.count(object_id));
76 77
77 // TODO(dcheng): Only acknowledge the invalidation once the fetch has 78 // TODO(dcheng): Only acknowledge the invalidation once the fetch has
78 // completed. http://crbug.com/156843 79 // completed. http://crbug.com/156843
79 ProfileSyncService* profile_sync_service = 80 invalidation::InvalidationService* invalidation_service =
80 ProfileSyncServiceFactory::GetForProfile(profile_); 81 invalidation::InvalidationServiceFactory::GetForProfile(profile_);
81 profile_sync_service->AcknowledgeInvalidation( 82 DCHECK(invalidation_service);
83 invalidation_service->AcknowledgeInvalidation(
82 invalidation_map.begin()->first, 84 invalidation_map.begin()->first,
83 invalidation_map.begin()->second.ack_handle); 85 invalidation_map.begin()->second.ack_handle);
84 86
85 NotifyObserversToUpdate(NOTIFICATION_XMPP); 87 NotifyObserversToUpdate(NOTIFICATION_XMPP);
86 } 88 }
87 89
88 void DriveNotificationManager::AddObserver( 90 void DriveNotificationManager::AddObserver(
89 DriveNotificationObserver* observer) { 91 DriveNotificationObserver* observer) {
90 observers_.AddObserver(observer); 92 observers_.AddObserver(observer);
91 } 93 }
(...skipping 29 matching lines...) Expand all
121 123
122 // Note that polling_timer_ is not a repeating timer. Restarting manually 124 // Note that polling_timer_ is not a repeating timer. Restarting manually
123 // here is better as XMPP may be received right before the polling timer is 125 // here is better as XMPP may be received right before the polling timer is
124 // fired (i.e. we don't notify observers twice in a row). 126 // fired (i.e. we don't notify observers twice in a row).
125 RestartPollingTimer(); 127 RestartPollingTimer();
126 } 128 }
127 129
128 void DriveNotificationManager::RegisterDriveNotifications() { 130 void DriveNotificationManager::RegisterDriveNotifications() {
129 DCHECK(!push_notification_enabled_); 131 DCHECK(!push_notification_enabled_);
130 132
131 ProfileSyncService* profile_sync_service = 133 invalidation::InvalidationService* invalidation_service =
132 ProfileSyncServiceFactory::GetForProfile(profile_); 134 invalidation::InvalidationServiceFactory::GetForProfile(profile_);
133 if (!profile_sync_service) 135 if (!invalidation_service)
134 return; 136 return;
135 137
136 profile_sync_service->RegisterInvalidationHandler(this); 138 invalidation_service->RegisterInvalidationHandler(this);
137 syncer::ObjectIdSet ids; 139 syncer::ObjectIdSet ids;
138 ids.insert(invalidation::ObjectId( 140 ids.insert(invalidation::ObjectId(
139 ipc::invalidation::ObjectSource::COSMO_CHANGELOG, 141 ipc::invalidation::ObjectSource::COSMO_CHANGELOG,
140 kDriveInvalidationObjectId)); 142 kDriveInvalidationObjectId));
141 profile_sync_service->UpdateRegisteredInvalidationIds(this, ids); 143 invalidation_service->UpdateRegisteredInvalidationIds(this, ids);
142 push_notification_registered_ = true; 144 push_notification_registered_ = true;
143 OnInvalidatorStateChange(profile_sync_service->GetInvalidatorState()); 145 OnInvalidatorStateChange(invalidation_service->GetInvalidatorState());
144 146
145 UMA_HISTOGRAM_BOOLEAN("Drive.PushNotificationRegistered", 147 UMA_HISTOGRAM_BOOLEAN("Drive.PushNotificationRegistered",
146 push_notification_registered_); 148 push_notification_registered_);
147 } 149 }
148 150
149 // static 151 // static
150 std::string DriveNotificationManager::NotificationSourceToString( 152 std::string DriveNotificationManager::NotificationSourceToString(
151 NotificationSource source) { 153 NotificationSource source) {
152 switch (source) { 154 switch (source) {
153 case NOTIFICATION_XMPP: 155 case NOTIFICATION_XMPP:
154 return "NOTIFICATION_XMPP"; 156 return "NOTIFICATION_XMPP";
155 case NOTIFICATION_POLLING: 157 case NOTIFICATION_POLLING:
156 return "NOTIFICATION_POLLING"; 158 return "NOTIFICATION_POLLING";
157 } 159 }
158 160
159 NOTREACHED(); 161 NOTREACHED();
160 return ""; 162 return "";
161 } 163 }
162 164
163 } // namespace drive 165 } // namespace drive
OLDNEW
« no previous file with comments | « chrome/browser/chrome_to_mobile_service_factory.cc ('k') | chrome/browser/drive/drive_notification_manager_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698