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

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

Issue 2354613002: [Sync] Fix namespaces for the browser_sync component. (Closed)
Patch Set: Address comments. Created 4 years, 3 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_UPDATED_PROGRESS_MARKER_CHECKER_H_ 5 #ifndef CHROME_BROWSER_SYNC_TEST_INTEGRATION_UPDATED_PROGRESS_MARKER_CHECKER_H_
6 #define CHROME_BROWSER_SYNC_TEST_INTEGRATION_UPDATED_PROGRESS_MARKER_CHECKER_H_ 6 #define CHROME_BROWSER_SYNC_TEST_INTEGRATION_UPDATED_PROGRESS_MARKER_CHECKER_H_
7 7
8 #include "chrome/browser/sync/test/integration/single_client_status_change_check er.h" 8 #include "chrome/browser/sync/test/integration/single_client_status_change_check er.h"
9 9
10 // Waits until the latest progress markers are available. 10 // Waits until the latest progress markers are available.
11 // 11 //
12 // There are several limitations to this checker: 12 // There are several limitations to this checker:
13 // - It assumes that this client is the only one committing at this time. 13 // - It assumes that this client is the only one committing at this time.
14 // - It relies on the test-only 'self-notify' to trigger an extra GetUpdate 14 // - It relies on the test-only 'self-notify' to trigger an extra GetUpdate
15 // cycle after every commit. 15 // cycle after every commit.
16 // - It's flaky. In some rare cases, the IsExitConditionSatisifed() call could 16 // - It's flaky. In some rare cases, the IsExitConditionSatisifed() call could
17 // return a false positive. See comments in the .cc file for details. 17 // return a false positive. See comments in the .cc file for details.
18 // 18 //
19 // Because of these limitations, we intend to eventually migrate all tests off 19 // Because of these limitations, we intend to eventually migrate all tests off
20 // of this checker. Please do not use it in new tests. 20 // of this checker. Please do not use it in new tests.
21 class UpdatedProgressMarkerChecker : public SingleClientStatusChangeChecker { 21 class UpdatedProgressMarkerChecker : public SingleClientStatusChangeChecker {
22 public: 22 public:
23 explicit UpdatedProgressMarkerChecker(ProfileSyncService* service); 23 explicit UpdatedProgressMarkerChecker(
24 browser_sync::ProfileSyncService* service);
24 ~UpdatedProgressMarkerChecker() override; 25 ~UpdatedProgressMarkerChecker() override;
25 26
26 bool IsExitConditionSatisfied() override; 27 bool IsExitConditionSatisfied() override;
27 std::string GetDebugMessage() const override; 28 std::string GetDebugMessage() const override;
28 }; 29 };
29 30
30 #endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_UPDATED_PROGRESS_MARKER_CHECKER_ H_ 31 #endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_UPDATED_PROGRESS_MARKER_CHECKER_ H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698