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

Side by Side Diff: base/test/test_suite.cc

Issue 10004001: Add virtual and OVERRIDE to base/ implementation files (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Win Fix -> Missing header Created 8 years, 8 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 | Annotate | Revision Log
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 "base/test/test_suite.h" 5 #include "base/test/test_suite.h"
6 6
7 #include "base/at_exit.h" 7 #include "base/at_exit.h"
8 #include "base/base_paths.h" 8 #include "base/base_paths.h"
9 #include "base/base_switches.h" 9 #include "base/base_switches.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 21 matching lines...) Expand all
32 #endif 32 #endif
33 33
34 #if defined(TOOLKIT_USES_GTK) 34 #if defined(TOOLKIT_USES_GTK)
35 #include <gtk/gtk.h> 35 #include <gtk/gtk.h>
36 #endif 36 #endif
37 37
38 namespace { 38 namespace {
39 39
40 class MaybeTestDisabler : public testing::EmptyTestEventListener { 40 class MaybeTestDisabler : public testing::EmptyTestEventListener {
41 public: 41 public:
42 virtual void OnTestStart(const testing::TestInfo& test_info) { 42 virtual void OnTestStart(const testing::TestInfo& test_info) OVERRIDE {
43 ASSERT_FALSE(TestSuite::IsMarkedMaybe(test_info)) 43 ASSERT_FALSE(TestSuite::IsMarkedMaybe(test_info))
44 << "Probably the OS #ifdefs don't include all of the necessary " 44 << "Probably the OS #ifdefs don't include all of the necessary "
45 "platforms.\nPlease ensure that no tests have the MAYBE_ prefix " 45 "platforms.\nPlease ensure that no tests have the MAYBE_ prefix "
46 "after the code is preprocessed."; 46 "after the code is preprocessed.";
47 } 47 }
48 }; 48 };
49 49
50 class TestClientInitializer : public testing::EmptyTestEventListener { 50 class TestClientInitializer : public testing::EmptyTestEventListener {
51 public: 51 public:
52 TestClientInitializer() 52 TestClientInitializer()
(...skipping 222 matching lines...) Expand 10 before | Expand all | Expand 10 after
275 #endif 275 #endif
276 276
277 CatchMaybeTests(); 277 CatchMaybeTests();
278 ResetCommandLine(); 278 ResetCommandLine();
279 279
280 TestTimeouts::Initialize(); 280 TestTimeouts::Initialize();
281 } 281 }
282 282
283 void TestSuite::Shutdown() { 283 void TestSuite::Shutdown() {
284 } 284 }
OLDNEW
« no previous file with comments | « base/system_monitor/system_monitor_unittest.cc ('k') | base/threading/non_thread_safe_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698