Implement Invalidator::Acknowledge

chrome/browser/chrome_to_mobile_service.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chrome_to_mobile_service.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/guid.h"
@@ skipping 413 matching lines @@
   UpdateCommandState();
 }
 
 void ChromeToMobileService::OnIncomingInvalidation(
     const syncer::ObjectIdInvalidationMap& invalidation_map,
     syncer::IncomingInvalidationSource source) {
   DCHECK_EQ(1U, invalidation_map.size());
   DCHECK_EQ(1U, invalidation_map.count(invalidation::ObjectId(
       ipc::invalidation::ObjectSource::CHROME_COMPONENTS,
       kSyncInvalidationObjectIdChromeToMobileDeviceList)));
+  // TODO(msw): Unit tests do not provide profiles; see http://crbug.com/122183

[akalin, 2012/10/19 13:27:16]

I think it should be acked only when the device search request finishes, no? 
Maybe add a TODO / file a bug with C2M folks.

[dcheng, 2012/10/19 19:38:11]

Done. Added a TODO and filed a crbug.

+  ProfileSyncService* profile_sync_service =
+      profile_ ? ProfileSyncServiceFactory::GetForProfile(profile_) : NULL;
+  if (profile_sync_service) {
+    profile_sync_service->AcknowledgeInvalidation(
+        invalidation_map.begin()->first,
+        invalidation_map.begin()->second.ack_handle);
+  }
   RequestDeviceSearch();
 }
 
 const std::string& ChromeToMobileService::GetAccessTokenForTest() const {
   return access_token_;
 }
 
 void ChromeToMobileService::SetAccessTokenForTest(
     const std::string& access_token) {
   access_token_ = access_token;
@@ skipping 257 matching lines @@
 
       // Report failure below and ignore the second response.
       request_observer_map_.erase(other);
       break;
     }
   }
 
   if (observer.get())
     observer->OnSendComplete(success);
 }