Sync: Don't set/use Parent ID for non-hierarchical sync data types.

sync/internal_api/sync_rollback_manager_unittest.cc

 // Copyright 2014 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 "sync/internal_api/sync_rollback_manager.h"
 
 #include "base/files/scoped_temp_dir.h"
 #include "base/run_loop.h"
 #include "sync/internal_api/public/read_node.h"
 #include "sync/internal_api/public/read_transaction.h"
@@ skipping 74 matching lines @@
   MOCK_METHOD1(OnMigrationRequested, void(ModelTypeSet));;
   MOCK_METHOD1(OnProtocolEvent, void(const ProtocolEvent&));
 
   void OnConfigDone(bool success) {
     EXPECT_TRUE(success);
   }
 
   int64 CreateEntry(UserShare* user_share, ModelType type,
                     const std::string& client_tag) {
     WriteTransaction trans(FROM_HERE, user_share);
-    ReadNode type_root(&trans);
-    EXPECT_EQ(BaseNode::INIT_OK, type_root.InitTypeRoot(type));
-
     WriteNode node(&trans);
     EXPECT_EQ(WriteNode::INIT_SUCCESS,
-              node.InitUniqueByCreation(type, type_root, client_tag));
+              node.InitUniqueByCreation(type, client_tag));
     return node.GetEntry()->GetMetahandle();
   }
 
   void InitManager(SyncManager* manager, ModelTypeSet types,
                    TestChangeDelegate* delegate,
                    InternalComponentsFactory::StorageOption storage_option) {
     manager_ = manager;
     types_ = types;
 
     EXPECT_CALL(*this, OnInitializationComplete(_, _, _, _))
@@ skipping 148 matching lines @@
   // Test graceful shutdown on initialization failure.
   scoped_ptr<SyncRollbackManager> rollback_manager(
       new SyncRollbackManager);
   InitManager(rollback_manager.get(), ModelTypeSet(PREFERENCES), NULL,
               InternalComponentsFactory::STORAGE_ON_DISK);
 }
 
 }  // anonymous namespace
 
 }  // namespace syncer