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

Unified Diff: components/sync_driver/data_type_manager_impl_unittest.cc

Issue 1380693004: [Sync] Fixing sync_driver lint violations and adding PRESUBMIT lint check. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@cross
Patch Set: Removing lint fixes for revisit files, removing revisit patch dep. Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/sync_driver/data_type_manager_impl.cc ('k') | components/sync_driver/data_type_status_table.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync_driver/data_type_manager_impl_unittest.cc
diff --git a/components/sync_driver/data_type_manager_impl_unittest.cc b/components/sync_driver/data_type_manager_impl_unittest.cc
index dd7028bc2d8d72525b16c7f32ef4904951e99e93..0374618d86ab08b764fd2264e67614e39fcd0588 100644
--- a/components/sync_driver/data_type_manager_impl_unittest.cc
+++ b/components/sync_driver/data_type_manager_impl_unittest.cc
@@ -115,11 +115,11 @@ class FakeBackendDataTypeConfigurer : public BackendDataTypeConfigurer {
void ActivateNonBlockingDataType(
syncer::ModelType type,
scoped_ptr<syncer_v2::ActivationContext> activation_context) override {
- // TODO (stanisc): crbug.com/515962: Add test coverage.
+ // TODO(stanisc): crbug.com/515962: Add test coverage.
}
void DeactivateNonBlockingDataType(syncer::ModelType type) override {
- // TODO (stanisc): crbug.com/515962: Add test coverage.
+ // TODO(stanisc): crbug.com/515962: Add test coverage.
}
base::Callback<void(ModelTypeSet, ModelTypeSet)> last_ready_task() const {
@@ -228,7 +228,7 @@ FakeDataTypeEncryptionHandler::GetEncryptedDataTypes() const {
return encrypted_types_;
}
-} // namespace
+} // namespace
class TestDataTypeManager : public DataTypeManagerImpl {
public:
@@ -279,13 +279,13 @@ class SyncDataTypeManagerImplTest : public testing::Test {
protected:
void SetUp() override {
- dtm_.reset(
- new TestDataTypeManager(
- syncer::WeakHandle<syncer::DataTypeDebugInfoListener>(),
- &configurer_,
- &controllers_,
- &encryption_handler_,
- &observer_));
+ dtm_.reset(
+ new TestDataTypeManager(
+ syncer::WeakHandle<syncer::DataTypeDebugInfoListener>(),
+ &configurer_,
+ &controllers_,
+ &encryption_handler_,
+ &observer_));
}
void SetConfigureStartExpectation() {
« no previous file with comments | « components/sync_driver/data_type_manager_impl.cc ('k') | components/sync_driver/data_type_status_table.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698