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

Side by Side Diff: sync/engine/syncer_proto_util_unittest.cc

Issue 792343004: Standardize usage of virtual/override/final specifiers in sync/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 12 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
« no previous file with comments | « sync/engine/sync_scheduler_unittest.cc ('k') | sync/engine/syncer_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "sync/engine/syncer_proto_util.h" 5 #include "sync/engine/syncer_proto_util.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/basictypes.h" 9 #include "base/basictypes.h"
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 166 matching lines...) Expand 10 before | Expand all | Expand 10 after
177 two_name_response.set_name(oxyphen); 177 two_name_response.set_name(oxyphen);
178 two_name_response.set_non_unique_name(neuro); 178 two_name_response.set_non_unique_name(neuro);
179 179
180 const std::string name_a = 180 const std::string name_a =
181 SyncerProtoUtil::NameFromSyncEntity(two_name_entity); 181 SyncerProtoUtil::NameFromSyncEntity(two_name_entity);
182 EXPECT_EQ(neuro, name_a); 182 EXPECT_EQ(neuro, name_a);
183 } 183 }
184 184
185 class SyncerProtoUtilTest : public testing::Test { 185 class SyncerProtoUtilTest : public testing::Test {
186 public: 186 public:
187 virtual void SetUp() { 187 void SetUp() override { dir_maker_.SetUp(); }
188 dir_maker_.SetUp();
189 }
190 188
191 virtual void TearDown() { 189 void TearDown() override { dir_maker_.TearDown(); }
192 dir_maker_.TearDown();
193 }
194 190
195 syncable::Directory* directory() { 191 syncable::Directory* directory() {
196 return dir_maker_.directory(); 192 return dir_maker_.directory();
197 } 193 }
198 194
199 protected: 195 protected:
200 base::MessageLoop message_loop_; 196 base::MessageLoop message_loop_;
201 TestDirectorySetterUpper dir_maker_; 197 TestDirectorySetterUpper dir_maker_;
202 }; 198 };
203 199
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
302 dcm.set_send_error(false); 298 dcm.set_send_error(false);
303 EXPECT_TRUE(SyncerProtoUtil::PostAndProcessHeaders(&dcm, NULL, 299 EXPECT_TRUE(SyncerProtoUtil::PostAndProcessHeaders(&dcm, NULL,
304 msg, &response)); 300 msg, &response));
305 301
306 dcm.set_access_denied(true); 302 dcm.set_access_denied(true);
307 EXPECT_FALSE(SyncerProtoUtil::PostAndProcessHeaders(&dcm, NULL, 303 EXPECT_FALSE(SyncerProtoUtil::PostAndProcessHeaders(&dcm, NULL,
308 msg, &response)); 304 msg, &response));
309 } 305 }
310 306
311 } // namespace syncer 307 } // namespace syncer
OLDNEW
« no previous file with comments | « sync/engine/sync_scheduler_unittest.cc ('k') | sync/engine/syncer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698