| OLD | NEW | 
|---|
| 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_ | 
| OLD | NEW | 
|---|