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

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

Issue 2530053003: chrome: Cleanup class/struct forward declarations (Closed)
Patch Set: Rebase + address comment Created 4 years 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_STATUS_CHANGE_CHECKER_H_ 5 #ifndef CHROME_BROWSER_SYNC_TEST_INTEGRATION_STATUS_CHANGE_CHECKER_H_
6 #define CHROME_BROWSER_SYNC_TEST_INTEGRATION_STATUS_CHANGE_CHECKER_H_ 6 #define CHROME_BROWSER_SYNC_TEST_INTEGRATION_STATUS_CHANGE_CHECKER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/time/time.h" 10 #include "base/time/time.h"
11 11
12 class ProfileSyncServiceHarness;
13
14 // Interface for a helper class that can pump the message loop while waiting 12 // Interface for a helper class that can pump the message loop while waiting
15 // for a certain state transition to take place. 13 // for a certain state transition to take place.
16 // 14 //
17 // This is a template that should be filled in by child classes so they can 15 // This is a template that should be filled in by child classes so they can
18 // observe specific kinds of changes and await specific conditions. 16 // observe specific kinds of changes and await specific conditions.
19 // 17 //
20 // The instances of this class are intended to be single-use. It doesn't make 18 // The instances of this class are intended to be single-use. It doesn't make
21 // sense to call StartBlockingWait() more than once. 19 // sense to call StartBlockingWait() more than once.
22 class StatusChangeChecker { 20 class StatusChangeChecker {
23 public: 21 public:
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
62 // true. 60 // true.
63 void CheckExitCondition(); 61 void CheckExitCondition();
64 62
65 // Called when the blocking wait timeout is exceeded. 63 // Called when the blocking wait timeout is exceeded.
66 void OnTimeout(); 64 void OnTimeout();
67 65
68 bool timed_out_; 66 bool timed_out_;
69 }; 67 };
70 68
71 #endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_STATUS_CHANGE_CHECKER_H_ 69 #endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_STATUS_CHANGE_CHECKER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698