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

Side by Side Diff: components/browser_sync/profile_sync_service_unittest.cc

Issue 2926553004: Remove trailing semicolon at the end of a method definition (Closed)
Patch Set: rebase Created 3 years, 6 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "components/browser_sync/profile_sync_service.h" 5 #include "components/browser_sync/profile_sync_service.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/callback.h" 9 #include "base/callback.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 const char kGaiaId[] = "12345"; 53 const char kGaiaId[] = "12345";
54 const char kEmail[] = "test_user@gmail.com"; 54 const char kEmail[] = "test_user@gmail.com";
55 55
56 class FakeDataTypeManager : public syncer::DataTypeManager { 56 class FakeDataTypeManager : public syncer::DataTypeManager {
57 public: 57 public:
58 using ConfigureCalled = base::Callback<void(syncer::ConfigureReason)>; 58 using ConfigureCalled = base::Callback<void(syncer::ConfigureReason)>;
59 59
60 explicit FakeDataTypeManager(const ConfigureCalled& configure_called) 60 explicit FakeDataTypeManager(const ConfigureCalled& configure_called)
61 : configure_called_(configure_called) {} 61 : configure_called_(configure_called) {}
62 62
63 ~FakeDataTypeManager() override{}; 63 ~FakeDataTypeManager() override {}
64 64
65 void Configure(syncer::ModelTypeSet desired_types, 65 void Configure(syncer::ModelTypeSet desired_types,
66 syncer::ConfigureReason reason) override { 66 syncer::ConfigureReason reason) override {
67 DCHECK(!configure_called_.is_null()); 67 DCHECK(!configure_called_.is_null());
68 configure_called_.Run(reason); 68 configure_called_.Run(reason);
69 } 69 }
70 70
71 void ReenableType(syncer::ModelType type) override {} 71 void ReenableType(syncer::ModelType type) override {}
72 void ResetDataTypeErrors() override {} 72 void ResetDataTypeErrors() override {}
73 void PurgeForMigration(syncer::ModelTypeSet undesired_types, 73 void PurgeForMigration(syncer::ModelTypeSet undesired_types,
74 syncer::ConfigureReason reason) override {} 74 syncer::ConfigureReason reason) override {}
75 void Stop() override{}; 75 void Stop() override {}
76 ModelTypeSet GetActiveDataTypes() const override { return ModelTypeSet(); } 76 ModelTypeSet GetActiveDataTypes() const override { return ModelTypeSet(); }
77 bool IsNigoriEnabled() const override { return true; } 77 bool IsNigoriEnabled() const override { return true; }
78 State state() const override { return syncer::DataTypeManager::CONFIGURED; } 78 State state() const override { return syncer::DataTypeManager::CONFIGURED; }
79 79
80 private: 80 private:
81 ConfigureCalled configure_called_; 81 ConfigureCalled configure_called_;
82 }; 82 };
83 83
84 ACTION_P(ReturnNewDataTypeManager, configure_called) { 84 ACTION_P(ReturnNewDataTypeManager, configure_called) {
85 return new FakeDataTypeManager(configure_called); 85 return new FakeDataTypeManager(configure_called);
(...skipping 905 matching lines...) Expand 10 before | Expand all | Expand 10 after
991 // Progress the controller to RUNNING first, which is how the service 991 // Progress the controller to RUNNING first, which is how the service
992 // determines whether a type is enabled. 992 // determines whether a type is enabled.
993 controller->StartAssociating(base::Bind(&DoNothing)); 993 controller->StartAssociating(base::Bind(&DoNothing));
994 controller->FinishStart(DataTypeController::OK_FIRST_RUN); 994 controller->FinishStart(DataTypeController::OK_FIRST_RUN);
995 service()->RegisterDataTypeController(std::move(controller)); 995 service()->RegisterDataTypeController(std::move(controller));
996 EXPECT_NE(nullptr, service()->GetOpenTabsUIDelegate()); 996 EXPECT_NE(nullptr, service()->GetOpenTabsUIDelegate());
997 } 997 }
998 998
999 } // namespace 999 } // namespace
1000 } // namespace browser_sync 1000 } // namespace browser_sync
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/tabs/tab_unittest.cc ('k') | components/history/core/browser/typed_url_sync_bridge_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698