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

Unified Diff: chrome/browser/extensions/extension_sorting_unittest.cc

Issue 12212047: Linux/ChromeOS Chromium style checker cleanup, chrome/browser/extensions edition. (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: Created 7 years, 10 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
Index: chrome/browser/extensions/extension_sorting_unittest.cc
===================================================================
--- chrome/browser/extensions/extension_sorting_unittest.cc (revision 181040)
+++ chrome/browser/extensions/extension_sorting_unittest.cc (working copy)
@@ -534,10 +534,10 @@
: public extensions::ExtensionPrefsPrepopulatedTest {
public :
ExtensionSortingEnsureValidOrdinals() {}
- ~ExtensionSortingEnsureValidOrdinals() {}
+ virtual ~ExtensionSortingEnsureValidOrdinals() {}
- virtual void Initialize() {}
- virtual void Verify() {
+ virtual void Initialize() OVERRIDE {}
+ virtual void Verify() OVERRIDE {
ExtensionSorting* extension_sorting = prefs()->extension_sorting();
// Give ext1 invalid ordinals and then check that EnsureValidOrdinals fixes
@@ -895,7 +895,7 @@
}
protected:
- virtual void SetupUserOrdinals() {
+ virtual void SetupUserOrdinals() OVERRIDE {
user_page_ordinal_ = default_page_ordinal_.CreateAfter();
user_app_launch_ordinal_ = default_app_launch_ordinal_.CreateBefore();
@@ -931,7 +931,7 @@
}
protected:
- virtual void SetupUserOrdinals() {
+ virtual void SetupUserOrdinals() OVERRIDE {
other_app_ = prefs_.AddApp("other_app");
// Creates a collision.
ExtensionSorting* extension_sorting = prefs()->extension_sorting();
« no previous file with comments | « chrome/browser/extensions/extension_service_unittest.cc ('k') | chrome/browser/extensions/extension_startup_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698