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

Side by Side Diff: components/sync/engine/attachments/fake_attachment_downloader.h

Issue 2915453002: Deprecate NonThreadSafe in components/sync in favor of SequenceChecker. (Closed)
Patch Set: fix comment Created 3 years, 6 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 #ifndef COMPONENTS_SYNC_ENGINE_ATTACHMENTS_FAKE_ATTACHMENT_DOWNLOADER_H_ 5 #ifndef COMPONENTS_SYNC_ENGINE_ATTACHMENTS_FAKE_ATTACHMENT_DOWNLOADER_H_
6 #define COMPONENTS_SYNC_ENGINE_ATTACHMENTS_FAKE_ATTACHMENT_DOWNLOADER_H_ 6 #define COMPONENTS_SYNC_ENGINE_ATTACHMENTS_FAKE_ATTACHMENT_DOWNLOADER_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "base/threading/non_thread_safe.h" 9 #include "base/sequence_checker.h"
10 #include "components/sync/engine/attachments/attachment_downloader.h" 10 #include "components/sync/engine/attachments/attachment_downloader.h"
11 11
12 namespace syncer { 12 namespace syncer {
13 13
14 // FakeAttachmentDownloader is for tests. For every request it posts a success 14 // FakeAttachmentDownloader is for tests. For every request it posts a success
15 // callback with empty attachment. 15 // callback with empty attachment.
16 class FakeAttachmentDownloader : public AttachmentDownloader, 16 class FakeAttachmentDownloader : public AttachmentDownloader {
17 public base::NonThreadSafe {
18 public: 17 public:
19 FakeAttachmentDownloader(); 18 FakeAttachmentDownloader();
20 ~FakeAttachmentDownloader() override; 19 ~FakeAttachmentDownloader() override;
21 20
22 // AttachmentDownloader implementation. 21 // AttachmentDownloader implementation.
23 void DownloadAttachment(const AttachmentId& attachment_id, 22 void DownloadAttachment(const AttachmentId& attachment_id,
24 const DownloadCallback& callback) override; 23 const DownloadCallback& callback) override;
25 24
26 private: 25 private:
26 SEQUENCE_CHECKER(sequence_checker_);
27
27 DISALLOW_COPY_AND_ASSIGN(FakeAttachmentDownloader); 28 DISALLOW_COPY_AND_ASSIGN(FakeAttachmentDownloader);
28 }; 29 };
29 30
30 } // namespace syncer 31 } // namespace syncer
31 32
32 #endif // COMPONENTS_SYNC_ENGINE_ATTACHMENTS_FAKE_ATTACHMENT_DOWNLOADER_H_ 33 #endif // COMPONENTS_SYNC_ENGINE_ATTACHMENTS_FAKE_ATTACHMENT_DOWNLOADER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698