[Sync] Replace AttachmentIdSet with AttachmentIdList in interfaces.

Today both types are used on AttachmentService and other interfaces.
In this change I make all interfaces use AttachmentIdList.
AttachmentIdSet is only used in implementations and unittests.

R=maniscalco@chromium.org
BUG=
TEST=No observable behavior change.

Review URL: https://codereview.chromium.org/1035573002

Cr-Commit-Position: refs/heads/master@{#322130}

1 files changed, 2 insertions, 2 deletions

diff --git a/sync/internal_api/attachments/attachment_service_proxy_unittest.cc b/sync/internal_api/attachments/attachment_service_proxy_unittest.cc
index a66b39f..fe52f46 100644
--- a/sync/internal_api/attachments/attachment_service_proxy_unittest.cc
+++ b/sync/internal_api/attachments/attachment_service_proxy_unittest.cc
@@ -46,7 +46,7 @@ class StubAttachmentService : public AttachmentService,
                    base::Passed(&attachments)));
   }
 
-  void UploadAttachments(const AttachmentIdSet& attachments_ids) override {
+  void UploadAttachments(const AttachmentIdList& attachments_ids) override {
     CalledOnValidThread();
     Increment();
   }
@@ -135,7 +135,7 @@ class AttachmentServiceProxyTest : public testing::Test,
 // thread.
 TEST_F(AttachmentServiceProxyTest, MethodsAreProxied) {
   proxy->GetOrDownloadAttachments(AttachmentIdList(), callback_get_or_download);
-  proxy->UploadAttachments(AttachmentIdSet());
+  proxy->UploadAttachments(AttachmentIdList());
   // Wait for the posted calls to execute in the stub thread.
   WaitForStubThread();
   EXPECT_EQ(2, stub->GetCallCount());