Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 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 CHROME_BROWSER_SYNC_ENGINE_DOWNLOAD_UPDATES_COMMAND_H_ | 5 #ifndef CHROME_BROWSER_SYNC_ENGINE_DOWNLOAD_UPDATES_COMMAND_H_ |
| 6 #define CHROME_BROWSER_SYNC_ENGINE_DOWNLOAD_UPDATES_COMMAND_H_ | 6 #define CHROME_BROWSER_SYNC_ENGINE_DOWNLOAD_UPDATES_COMMAND_H_ |
| 7 #pragma once | 7 #pragma once |
| 8 | 8 |
| 9 #include "base/basictypes.h" | 9 #include "base/basictypes.h" |
| 10 #include "base/compiler_specific.h" | 10 #include "base/compiler_specific.h" |
| (...skipping 22 matching lines...) Expand all Loading... | |
| 33 // session can be found in the Syncer logic. When looping, the | 33 // session can be found in the Syncer logic. When looping, the |
| 34 // DownloadUpdatesCommand consumes the information stored by the | 34 // DownloadUpdatesCommand consumes the information stored by the |
| 35 // StoreTimestampsCommand. | 35 // StoreTimestampsCommand. |
| 36 // | 36 // |
| 37 // In practice, DownloadUpdatesCommand should loop until all updates are | 37 // In practice, DownloadUpdatesCommand should loop until all updates are |
| 38 // downloaded for all enabled datatypes (i.e., until the server indicates | 38 // downloaded for all enabled datatypes (i.e., until the server indicates |
| 39 // changes_remaining == 0 in the GetUpdates response), or until an error | 39 // changes_remaining == 0 in the GetUpdates response), or until an error |
| 40 // is encountered. | 40 // is encountered. |
| 41 class DownloadUpdatesCommand : public SyncerCommand { | 41 class DownloadUpdatesCommand : public SyncerCommand { |
| 42 public: | 42 public: |
| 43 DownloadUpdatesCommand(); | 43 explicit DownloadUpdatesCommand(bool include_syncable_bookmarks); |
|
Nicolas Zea
2012/02/27 20:03:12
Comment about the parameter.
akalin
2012/02/27 22:04:18
Done.
| |
| 44 virtual ~DownloadUpdatesCommand(); | 44 virtual ~DownloadUpdatesCommand(); |
| 45 | 45 |
| 46 // SyncerCommand implementation. | 46 // SyncerCommand implementation. |
| 47 virtual SyncerError ExecuteImpl(sessions::SyncSession* session) OVERRIDE; | 47 virtual SyncerError ExecuteImpl(sessions::SyncSession* session) OVERRIDE; |
| 48 | 48 |
| 49 private: | 49 private: |
| 50 FRIEND_TEST_ALL_PREFIXES(DownloadUpdatesCommandTest, VerifyAppendDebugInfo); | 50 FRIEND_TEST_ALL_PREFIXES(DownloadUpdatesCommandTest, VerifyAppendDebugInfo); |
| 51 void AppendClientDebugInfoIfNeeded(sessions::SyncSession* session, | 51 void AppendClientDebugInfoIfNeeded(sessions::SyncSession* session, |
| 52 sync_pb::DebugInfo* debug_info); | 52 sync_pb::DebugInfo* debug_info); |
| 53 | |
| 54 const bool include_syncable_bookmarks_; | |
| 55 | |
| 53 DISALLOW_COPY_AND_ASSIGN(DownloadUpdatesCommand); | 56 DISALLOW_COPY_AND_ASSIGN(DownloadUpdatesCommand); |
| 54 }; | 57 }; |
| 55 | 58 |
| 56 } // namespace browser_sync | 59 } // namespace browser_sync |
| 57 | 60 |
| 58 #endif // CHROME_BROWSER_SYNC_ENGINE_DOWNLOAD_UPDATES_COMMAND_H_ | 61 #endif // CHROME_BROWSER_SYNC_ENGINE_DOWNLOAD_UPDATES_COMMAND_H_ |
| 59 | 62 |
| OLD | NEW |