NOT FOR COMMIT: Add DeviceInfo type and ChangeProcessor

sync/internal_api/sync_manager_impl_unittest.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.
 
 // Unit tests for the SyncApi. Note that a lot of the underlying
 // functionality is provided by the Syncable layer, which has its own
 // unit tests. We'll test SyncApi specific things in this harness.
 
 #include <cstddef>
 #include <map>
@@ skipping 735 matching lines @@
     EXPECT_FALSE(js_backend_.IsInitialized());
 
     std::vector<ModelSafeWorker*> workers;
     ModelSafeRoutingInfo routing_info;
     GetModelSafeRoutingInfo(&routing_info);
 
     // Takes ownership of |fake_invalidator_|.
     sync_manager_.Init(temp_dir_.path(),
                        WeakHandle<JsEventHandler>(),
                        "bogus", 0, false,
-                       base::MessageLoopProxy::current(),
                        scoped_ptr<HttpPostProviderFactory>(
                            new TestHttpPostProviderFactory()),
                        workers, &extensions_activity_monitor_, this,
                        credentials,
                        scoped_ptr<Invalidator>(fake_invalidator_),
                        "", "",  // bootstrap tokens
                        scoped_ptr<InternalComponentsFactory>(GetFactory()),
                        &encryptor_,
                        &handler_,
                        NULL);
@@ skipping 17 matching lines @@
     sync_manager_.ShutdownOnSyncThread();
     // We can't assert that |sync_manager_| isn't registered with
     // |fake_invalidator_| anymore because |fake_invalidator_| is now
     // destroyed.
     fake_invalidator_ = NULL;
     PumpLoop();
   }
 
   void GetModelSafeRoutingInfo(ModelSafeRoutingInfo* out) {
     (*out)[NIGORI] = GROUP_PASSIVE;
+    (*out)[DEVICE_INFO] = GROUP_PASSIVE;
     (*out)[BOOKMARKS] = GROUP_PASSIVE;
     (*out)[THEMES] = GROUP_PASSIVE;
     (*out)[SESSIONS] = GROUP_PASSIVE;
     (*out)[PASSWORDS] = GROUP_PASSIVE;
     (*out)[PREFERENCES] = GROUP_PASSIVE;
   }
 
   virtual void OnChangesApplied(
       ModelType model_type,
       const BaseTransaction* trans,
@@ skipping 378 matching lines @@
     ListValue args;
     args.Append(Value::CreateStringValue("1"));
     SendJsMessage("getChildNodeIds",
                   JsArgList(&args), reply_handler.AsWeakHandle());
   }
 
   EXPECT_EQ(1u, return_args.Get().GetSize());
   const ListValue* nodes = NULL;
   ASSERT_TRUE(return_args.Get().GetList(0, &nodes));
   ASSERT_TRUE(nodes);
-  EXPECT_EQ(6u, nodes->GetSize());
+  EXPECT_EQ(7u, nodes->GetSize());
 }
 
 TEST_F(SyncManagerTest, GetChildNodeIdsFailure) {
   StrictMock<MockJsReplyHandler> reply_handler;
 
   ListValue empty_list_args;
   empty_list_args.Append(new ListValue());
 
   EXPECT_CALL(reply_handler,
               HandleJsReply("getChildNodeIds",
@@ skipping 139 matching lines @@
   TriggerOnIncomingNotificationForTest(model_types);
   SetJsEventHandler(WeakHandle<JsEventHandler>());
 
   TriggerOnIncomingNotificationForTest(empty_model_types);
   TriggerOnIncomingNotificationForTest(model_types);
 
   // Should trigger the replies.
   PumpLoop();
 }
 
+#if 0 // FIXME -> Move these

[Nicolas Zea, 2012/09/13 00:45:55]

did you mean to leave this?

[rlarocque, 2012/09/14 01:03:07]

Nope, that was a mistake.  The tests have already been moved to
synced_device_tracker_unittest.cc.

+// New client scenario: set device info when no previous info existed.
+TEST_F(SyncManagerTest, CreateNewDeviceInfo) {
+  UserShare* share = sync_manager_.GetUserShare();
+  const std::string version("v1");
+  const std::string session_name("session");
+  sync_pb::DeviceInfoSpecifics device_info;
+
+  ASSERT_FALSE(GetDeviceInfo(share->directory.get(), &device_info));
+
+  UpdateDeviceInfo(version, session_name);
+
+  ASSERT_TRUE(GetDeviceInfo(share->directory.get(), &device_info));
+
+  EXPECT_EQ(version, device_info.chrome_version());
+  EXPECT_EQ(session_name, device_info.name());
+  EXPECT_EQ(share->directory->cache_guid(), device_info.cache_guid());
+}
+
+// Restart scenario: update existing device info with identical data.
+TEST_F(SyncManagerTest, DontModifyExistingDeviceInfo) {
+  UserShare* share = sync_manager_.GetUserShare();
+  const std::string version("v1");
+  const std::string session_name("session");
+  sync_pb::DeviceInfoSpecifics old_device_info;
+  sync_pb::DeviceInfoSpecifics new_device_info;
+
+  UpdateDeviceInfo(version, session_name);
+  ASSERT_TRUE(GetDeviceInfo(share->directory.get(), &old_device_info));
+
+  // Pretend we committed by unsetting all unsynced bits.
+  share->directory->SetInvariantCheckLevel(syncable::OFF);
+  {
+    syncable::WriteTransaction trans(FROM_HERE, syncable::UNITTEST,
+                                     share->directory.get());
+    syncable::Directory::UnsyncedMetaHandles handles;
+    share->directory->GetUnsyncedMetaHandles(&trans, &handles);
+
+    for (syncable::Directory::UnsyncedMetaHandles::iterator it =
+         handles.begin(); it != handles.end(); ++it) {
+      syncable::MutableEntry entry(&trans, syncable::GET_BY_HANDLE, *it);
+      entry.Put(syncable::IS_UNSYNCED, false);
+    }
+
+    ASSERT_EQ(0, share->directory->unsynced_entity_count());
+  }
+
+  // Overwrite the device info with the same data as before.
+  UpdateDeviceInfo(version, session_name);
+  ASSERT_TRUE(GetDeviceInfo(share->directory.get(), &new_device_info));
+
+  // Ensure that this didn't count as a change worth syncing.
+  EXPECT_EQ(0, share->directory->unsynced_entity_count());
+  EXPECT_EQ(old_device_info.SerializeAsString(),
+            new_device_info.SerializeAsString());
+
+  EXPECT_EQ(old_device_info.chrome_version(), new_device_info.chrome_version());
+  EXPECT_EQ(old_device_info.name(), new_device_info.name());
+  EXPECT_EQ(old_device_info.cache_guid(), new_device_info.cache_guid());
+}
+
+// Upgrade scenario: update existing device info with new version.
+TEST_F(SyncManagerTest, UpdateExistingDeviceInfo) {
+  UserShare* share = sync_manager_.GetUserShare();
+  const std::string old_version("v1");
+  const std::string new_version("v2");
+  const std::string session_name("session");
+  sync_pb::DeviceInfoSpecifics device_info;
+
+  UpdateDeviceInfo(old_version, session_name);
+  UpdateDeviceInfo(new_version, session_name);
+
+  GetDeviceInfo(share->directory.get(), &device_info);
+
+  EXPECT_EQ(new_version, device_info.chrome_version());
+}
+#endif
+
 TEST_F(SyncManagerTest, RefreshEncryptionReady) {
   EXPECT_TRUE(SetUpEncryption(WRITE_TO_NIGORI, DEFAULT_ENCRYPTION));
   EXPECT_CALL(encryption_observer_, OnEncryptionComplete());
   EXPECT_CALL(encryption_observer_, OnCryptographerStateChanged(_));
   EXPECT_CALL(encryption_observer_, OnEncryptedTypesChanged(_, false));
 
   sync_manager_.GetEncryptionHandler()->Init();
   PumpLoop();
 
   const ModelTypeSet encrypted_types = GetEncryptedTypes();
@@ skipping 1524 matching lines @@
 
   // Verify only the non-disabled types remain after cleanup.
   sync_manager_.PurgeDisabledTypes(enabled_types, new_enabled_types);
   EXPECT_TRUE(new_enabled_types.Equals(
       Union(sync_manager_.InitialSyncEndedTypes(), partial_enabled_types)));
   EXPECT_TRUE(disabled_types.Equals(
       sync_manager_.GetTypesWithEmptyProgressMarkerToken(ModelTypeSet::All())));
 }
 
 }  // namespace