Replace calls to 3-arg SyncData::CreateLocalData with 5-arg version.

chrome/browser/history/history_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.
 
 // History unit tests come in two flavors:
 //
 // 1. The more complicated style is that the unit test creates a full history
 //    service. This spawns a background thread for the history backend, and
 //    all communication is asynchronous. This is useful for testing more
 //    complicated things or end-to-end behavior.
@@ skipping 42 matching lines @@
 #include "chrome/browser/history/page_usage_data.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/thumbnail_score.h"
 #include "chrome/tools/profiles/thumbnail-inl.h"
 #include "content/public/browser/download_item.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_source.h"
 #include "sql/connection.h"
 #include "sql/statement.h"
+#include "sync/api/attachments/attachment_id.h"
+#include "sync/api/attachments/attachment_service_proxy_for_test.h"
 #include "sync/api/fake_sync_change_processor.h"
 #include "sync/api/sync_change.h"
 #include "sync/api/sync_change_processor.h"
 #include "sync/api/sync_change_processor_wrapper_for_test.h"
 #include "sync/api/sync_error.h"
 #include "sync/api/sync_error_factory.h"
 #include "sync/api/sync_merge_result.h"
 #include "sync/protocol/history_delete_directive_specifics.pb.h"
 #include "sync/protocol/sync.pb.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 1553 matching lines @@
   // 1st directive.
   sync_pb::EntitySpecifics entity_specs;
   sync_pb::GlobalIdDirective* global_id_directive =
       entity_specs.mutable_history_delete_directive()
           ->mutable_global_id_directive();
   global_id_directive->add_global_id(
       (base::Time::UnixEpoch() + base::TimeDelta::FromMicroseconds(6))
       .ToInternalValue());
   global_id_directive->set_start_time_usec(3);
   global_id_directive->set_end_time_usec(10);
-  directives.push_back(
-      syncer::SyncData::CreateRemoteData(1, entity_specs, base::Time()));
+  directives.push_back(syncer::SyncData::CreateRemoteData(
+      1,
+      entity_specs,
+      base::Time(),
+      syncer::AttachmentIdList(),
+      syncer::AttachmentServiceProxyForTest::Create()));
 
   // 2nd directive.
   global_id_directive->Clear();
   global_id_directive->add_global_id(
       (base::Time::UnixEpoch() + base::TimeDelta::FromMicroseconds(17))
       .ToInternalValue());
   global_id_directive->set_start_time_usec(13);
   global_id_directive->set_end_time_usec(19);
-  directives.push_back(
-      syncer::SyncData::CreateRemoteData(2, entity_specs, base::Time()));
+  directives.push_back(syncer::SyncData::CreateRemoteData(
+      2,
+      entity_specs,
+      base::Time(),
+      syncer::AttachmentIdList(),
+      syncer::AttachmentServiceProxyForTest::Create()));
 
   syncer::FakeSyncChangeProcessor change_processor;
   EXPECT_FALSE(
       history_service_->MergeDataAndStartSyncing(
                             syncer::HISTORY_DELETE_DIRECTIVES,
                             directives,
                             scoped_ptr<syncer::SyncChangeProcessor>(
                                 new syncer::SyncChangeProcessorWrapperForTest(
                                     &change_processor)),
                             scoped_ptr<syncer::SyncErrorFactory>())
@@ skipping 48 matching lines @@
   EXPECT_EQ(10, query_url_row_.visit_count());
 
   syncer::SyncDataList directives;
   // 1st directive.
   sync_pb::EntitySpecifics entity_specs;
   sync_pb::TimeRangeDirective* time_range_directive =
       entity_specs.mutable_history_delete_directive()
           ->mutable_time_range_directive();
   time_range_directive->set_start_time_usec(2);
   time_range_directive->set_end_time_usec(5);
-  directives.push_back(syncer::SyncData::CreateRemoteData(1,
-                                                          entity_specs,
-                                                          base::Time()));
+  directives.push_back(syncer::SyncData::CreateRemoteData(
+      1,
+      entity_specs,
+      base::Time(),
+      syncer::AttachmentIdList(),
+      syncer::AttachmentServiceProxyForTest::Create()));
 
   // 2nd directive.
   time_range_directive->Clear();
   time_range_directive->set_start_time_usec(8);
   time_range_directive->set_end_time_usec(10);
-  directives.push_back(syncer::SyncData::CreateRemoteData(2,
-                                                          entity_specs,
-                                                          base::Time()));
+  directives.push_back(syncer::SyncData::CreateRemoteData(
+      2,
+      entity_specs,
+      base::Time(),
+      syncer::AttachmentIdList(),
+      syncer::AttachmentServiceProxyForTest::Create()));
 
   syncer::FakeSyncChangeProcessor change_processor;
   EXPECT_FALSE(
       history_service_->MergeDataAndStartSyncing(
                             syncer::HISTORY_DELETE_DIRECTIVES,
                             directives,
                             scoped_ptr<syncer::SyncChangeProcessor>(
                                 new syncer::SyncChangeProcessorWrapperForTest(
                                     &change_processor)),
                             scoped_ptr<syncer::SyncErrorFactory>())
@@ skipping 89 matching lines @@
   std::vector<PageUsageData*> results;
   db_->QuerySegmentUsage(segment_time, 10, &results);
   ASSERT_EQ(1u, results.size());
   EXPECT_EQ(url, results[0]->GetURL());
   EXPECT_EQ(segment_id, results[0]->GetID());
   EXPECT_EQ(title, results[0]->GetTitle());
   STLDeleteElements(&results);
 }
 
 }  // namespace history