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

Side by Side Diff: chrome/browser/sync/test/integration/multi_client_status_change_checker.h

Issue 2354613002: [Sync] Fix namespaces for the browser_sync component. (Closed)
Patch Set: Address comments. Created 4 years, 2 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 CHROME_BROWSER_SYNC_TEST_INTEGRATION_MULTI_CLIENT_STATUS_CHANGE_CHECKER_ H_ 5 #ifndef CHROME_BROWSER_SYNC_TEST_INTEGRATION_MULTI_CLIENT_STATUS_CHANGE_CHECKER_ H_
6 #define CHROME_BROWSER_SYNC_TEST_INTEGRATION_MULTI_CLIENT_STATUS_CHANGE_CHECKER_ H_ 6 #define CHROME_BROWSER_SYNC_TEST_INTEGRATION_MULTI_CLIENT_STATUS_CHANGE_CHECKER_ H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
11 #include "base/time/time.h" 11 #include "base/time/time.h"
12 #include "chrome/browser/sync/test/integration/status_change_checker.h" 12 #include "chrome/browser/sync/test/integration/status_change_checker.h"
13 #include "components/sync/driver/sync_service_observer.h" 13 #include "components/sync/driver/sync_service_observer.h"
14 14
15 namespace browser_sync {
15 class ProfileSyncService; 16 class ProfileSyncService;
17 } // namespace browser_sync
16 18
17 // This class provides some common functionality for StatusChangeCheckers that 19 // This class provides some common functionality for StatusChangeCheckers that
18 // observe many ProfileSyncServices. This class is abstract. Its descendants 20 // observe many ProfileSyncServices. This class is abstract. Its descendants
19 // are expected to provide additional functionality. 21 // are expected to provide additional functionality.
20 class MultiClientStatusChangeChecker : public StatusChangeChecker, 22 class MultiClientStatusChangeChecker : public StatusChangeChecker,
21 public sync_driver::SyncServiceObserver { 23 public sync_driver::SyncServiceObserver {
22 public: 24 public:
23 explicit MultiClientStatusChangeChecker( 25 explicit MultiClientStatusChangeChecker(
24 std::vector<ProfileSyncService*> services); 26 std::vector<browser_sync::ProfileSyncService*> services);
25 ~MultiClientStatusChangeChecker() override; 27 ~MultiClientStatusChangeChecker() override;
26 28
27 // Called when waiting times out. 29 // Called when waiting times out.
28 void OnTimeout(); 30 void OnTimeout();
29 31
30 // Blocks until the exit condition is satisfied or a timeout occurs. 32 // Blocks until the exit condition is satisfied or a timeout occurs.
31 void Wait(); 33 void Wait();
32 34
33 // sync_driver::SyncServiceObserver implementation. 35 // sync_driver::SyncServiceObserver implementation.
34 void OnStateChanged() override; 36 void OnStateChanged() override;
35 37
36 // StatusChangeChecker implementations and stubs. 38 // StatusChangeChecker implementations and stubs.
37 bool IsExitConditionSatisfied() override = 0; 39 bool IsExitConditionSatisfied() override = 0;
38 std::string GetDebugMessage() const override = 0; 40 std::string GetDebugMessage() const override = 0;
39 41
40 protected: 42 protected:
41 const std::vector<ProfileSyncService*>& services() { return services_; } 43 const std::vector<browser_sync::ProfileSyncService*>& services() {
44 return services_;
45 }
42 46
43 private: 47 private:
44 std::vector<ProfileSyncService*> services_; 48 std::vector<browser_sync::ProfileSyncService*> services_;
45 }; 49 };
46 50
47 #endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_MULTI_CLIENT_STATUS_CHANGE_CHECK ER_H_ 51 #endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_MULTI_CLIENT_STATUS_CHANGE_CHECK ER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698