Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(280)

Side by Side Diff: sync/internal_api/attachments/fake_attachment_downloader.cc

Issue 982883002: [Sync] Add size and crc32c to AttachmentId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix argument evaluation order bug. Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "sync/internal_api/public/attachments/fake_attachment_downloader.h" 5 #include "sync/internal_api/public/attachments/fake_attachment_downloader.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "sync/internal_api/public/attachments/attachment_util.h" 9 #include "sync/internal_api/public/attachments/attachment_util.h"
10 10
11 namespace syncer { 11 namespace syncer {
12 12
13 FakeAttachmentDownloader::FakeAttachmentDownloader() { 13 FakeAttachmentDownloader::FakeAttachmentDownloader() {
14 } 14 }
15 15
16 FakeAttachmentDownloader::~FakeAttachmentDownloader() { 16 FakeAttachmentDownloader::~FakeAttachmentDownloader() {
17 DCHECK(CalledOnValidThread()); 17 DCHECK(CalledOnValidThread());
18 } 18 }
19 19
20 void FakeAttachmentDownloader::DownloadAttachment( 20 void FakeAttachmentDownloader::DownloadAttachment(
21 const AttachmentId& attachment_id, 21 const AttachmentId& attachment_id,
22 const DownloadCallback& callback) { 22 const DownloadCallback& callback) {
23 DCHECK(CalledOnValidThread()); 23 DCHECK(CalledOnValidThread());
24 // This is happy fake downloader, it always successfully downloads empty 24 // This is happy fake downloader, it always successfully downloads empty
25 // attachment. 25 // attachment.
26 scoped_refptr<base::RefCountedMemory> data(new base::RefCountedBytes()); 26 scoped_refptr<base::RefCountedMemory> data(new base::RefCountedBytes());
27 scoped_ptr<Attachment> attachment; 27 scoped_ptr<Attachment> attachment;
28 const uint32_t crc32c = ComputeCrc32c(data);
29 attachment.reset( 28 attachment.reset(
30 new Attachment(Attachment::CreateFromParts(attachment_id, data, crc32c))); 29 new Attachment(Attachment::CreateFromParts(attachment_id, data)));
31 base::MessageLoop::current()->PostTask( 30 base::MessageLoop::current()->PostTask(
32 FROM_HERE, 31 FROM_HERE,
33 base::Bind(callback, DOWNLOAD_SUCCESS, base::Passed(&attachment))); 32 base::Bind(callback, DOWNLOAD_SUCCESS, base::Passed(&attachment)));
34 } 33 }
35 34
36 } // namespace syncer 35 } // namespace syncer
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698