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

chrome/browser/sync/glue/typed_url_change_processor.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/sync/glue/typed_url_change_processor.h"
 
 #include "base/location.h"
 #include "base/metrics/histogram.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/string_util.h"
@@ skipping 148 matching lines @@
   }
 
   if (std::find_if(visit_vector.begin(), visit_vector.end(),
                    [](const history::VisitRow& visit) {
                      return ui::PageTransitionCoreTypeIs(
                          visit.transition, ui::PAGE_TRANSITION_TYPED);
                    }) == visit_vector.end())
     // This URL has no TYPED visits, don't sync it.
     return false;
 
-  syncer::ReadNode typed_url_root(trans);
-  if (typed_url_root.InitTypeRoot(syncer::TYPED_URLS) !=
-          syncer::BaseNode::INIT_OK) {
-    syncer::SyncError error(FROM_HERE,
-                            syncer::SyncError::DATATYPE_ERROR,
-                            "No top level folder",
-                            syncer::TYPED_URLS);
-    error_handler()->OnSingleDataTypeUnrecoverableError(error);
-    return false;
-  }
-
   if (model_associator_->ShouldIgnoreUrl(url.url()))
     return true;
 
   DCHECK(!visit_vector.empty());
   std::string tag = url.url().spec();
   syncer::WriteNode update_node(trans);
   syncer::BaseNode::InitByLookupResult result =
       update_node.InitByClientTagLookup(syncer::TYPED_URLS, tag);
   if (result == syncer::BaseNode::INIT_OK) {
     model_associator_->WriteToSyncNode(url, visit_vector, &update_node);
   } else if (result == syncer::BaseNode::INIT_FAILED_DECRYPT_IF_NECESSARY) {
     syncer::SyncError error(FROM_HERE,
                             syncer::SyncError::DATATYPE_ERROR,
                             "Failed to decrypt.",
                             syncer::TYPED_URLS);
     error_handler()->OnSingleDataTypeUnrecoverableError(error);
     return false;
   } else {
     syncer::WriteNode create_node(trans);
     syncer::WriteNode::InitUniqueByCreationResult result =
-        create_node.InitUniqueByCreation(syncer::TYPED_URLS,
-                                         typed_url_root, tag);
+        create_node.InitUniqueByCreation(syncer::TYPED_URLS, tag);
     if (result != syncer::WriteNode::INIT_SUCCESS) {
 
       syncer::SyncError error(FROM_HERE,
                               syncer::SyncError::DATATYPE_ERROR,
                               "Failed to create sync node",
                               syncer::TYPED_URLS);
       error_handler()->OnSingleDataTypeUnrecoverableError(error);
       return false;
     }
 
@@ skipping 134 matching lines @@
 }
 
 void TypedUrlChangeProcessor::StopObserving() {
   DCHECK(backend_loop_ == base::MessageLoop::current());
   DCHECK(history_backend_);
   DCHECK(profile_);
   history_backend_observer_.RemoveAll();
 }
 
 }  // namespace browser_sync