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

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

Issue 629603002: replace OVERRIDE and FINAL with override and final in chrome/browser/[r-z]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase on master Created 6 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_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(ProfileSyncService* service);
24 virtual ~UpdatedProgressMarkerChecker(); 24 virtual ~UpdatedProgressMarkerChecker();
25 25
26 virtual bool IsExitConditionSatisfied() OVERRIDE; 26 virtual bool IsExitConditionSatisfied() override;
27 virtual std::string GetDebugMessage() const OVERRIDE; 27 virtual std::string GetDebugMessage() const override;
28 }; 28 };
29 29
30 #endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_UPDATED_PROGRESS_MARKER_CHECKER_ H_ 30 #endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_UPDATED_PROGRESS_MARKER_CHECKER_ H_
OLDNEW
« no previous file with comments | « chrome/browser/sync/test/integration/typed_urls_helper.cc ('k') | chrome/browser/sync/test/test_http_bridge_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698