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

Side by Side Diff: extensions/browser/null_app_sorting.h

Issue 664933004: Standardize usage of virtual/override/final in extensions/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 unified diff | Download patch
« no previous file with comments | « extensions/browser/mock_extension_system.h ('k') | extensions/browser/process_manager.h » ('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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #ifndef EXTENSIONS_BROWSER_NULL_APP_SORTING_H_ 5 #ifndef EXTENSIONS_BROWSER_NULL_APP_SORTING_H_
6 #define EXTENSIONS_BROWSER_NULL_APP_SORTING_H_ 6 #define EXTENSIONS_BROWSER_NULL_APP_SORTING_H_
7 7
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "extensions/browser/app_sorting.h" 10 #include "extensions/browser/app_sorting.h"
11 11
12 namespace extensions { 12 namespace extensions {
13 13
14 // An AppSorting that doesn't provide any ordering. 14 // An AppSorting that doesn't provide any ordering.
15 class NullAppSorting : public AppSorting { 15 class NullAppSorting : public AppSorting {
16 public: 16 public:
17 NullAppSorting(); 17 NullAppSorting();
18 virtual ~NullAppSorting(); 18 ~NullAppSorting() override;
19 19
20 // AppSorting overrides: 20 // AppSorting overrides:
21 virtual void SetExtensionScopedPrefs(ExtensionScopedPrefs* prefs) override; 21 void SetExtensionScopedPrefs(ExtensionScopedPrefs* prefs) override;
22 virtual void SetExtensionSyncService( 22 void SetExtensionSyncService(
23 ExtensionSyncService* extension_sync_service) override; 23 ExtensionSyncService* extension_sync_service) override;
24 virtual void Initialize(const ExtensionIdList& extension_ids) override; 24 void Initialize(const ExtensionIdList& extension_ids) override;
25 virtual void FixNTPOrdinalCollisions() override; 25 void FixNTPOrdinalCollisions() override;
26 virtual void EnsureValidOrdinals( 26 void EnsureValidOrdinals(
27 const std::string& extension_id, 27 const std::string& extension_id,
28 const syncer::StringOrdinal& suggested_page) override; 28 const syncer::StringOrdinal& suggested_page) override;
29 virtual void OnExtensionMoved( 29 void OnExtensionMoved(const std::string& moved_extension_id,
30 const std::string& moved_extension_id, 30 const std::string& predecessor_extension_id,
31 const std::string& predecessor_extension_id, 31 const std::string& successor_extension_id) override;
32 const std::string& successor_extension_id) override; 32 syncer::StringOrdinal GetAppLaunchOrdinal(
33 virtual syncer::StringOrdinal GetAppLaunchOrdinal(
34 const std::string& extension_id) const override; 33 const std::string& extension_id) const override;
35 virtual void SetAppLaunchOrdinal( 34 void SetAppLaunchOrdinal(
36 const std::string& extension_id, 35 const std::string& extension_id,
37 const syncer::StringOrdinal& new_app_launch_ordinal) override; 36 const syncer::StringOrdinal& new_app_launch_ordinal) override;
38 virtual syncer::StringOrdinal CreateFirstAppLaunchOrdinal( 37 syncer::StringOrdinal CreateFirstAppLaunchOrdinal(
39 const syncer::StringOrdinal& page_ordinal) const override; 38 const syncer::StringOrdinal& page_ordinal) const override;
40 virtual syncer::StringOrdinal CreateNextAppLaunchOrdinal( 39 syncer::StringOrdinal CreateNextAppLaunchOrdinal(
41 const syncer::StringOrdinal& page_ordinal) const override; 40 const syncer::StringOrdinal& page_ordinal) const override;
42 virtual syncer::StringOrdinal CreateFirstAppPageOrdinal() const override; 41 syncer::StringOrdinal CreateFirstAppPageOrdinal() const override;
43 virtual syncer::StringOrdinal GetNaturalAppPageOrdinal() const override; 42 syncer::StringOrdinal GetNaturalAppPageOrdinal() const override;
44 virtual syncer::StringOrdinal GetPageOrdinal( 43 syncer::StringOrdinal GetPageOrdinal(
45 const std::string& extension_id) const override; 44 const std::string& extension_id) const override;
46 virtual void SetPageOrdinal( 45 void SetPageOrdinal(const std::string& extension_id,
47 const std::string& extension_id, 46 const syncer::StringOrdinal& new_page_ordinal) override;
48 const syncer::StringOrdinal& new_page_ordinal) override; 47 void ClearOrdinals(const std::string& extension_id) override;
49 virtual void ClearOrdinals(const std::string& extension_id) override; 48 int PageStringOrdinalAsInteger(
50 virtual int PageStringOrdinalAsInteger(
51 const syncer::StringOrdinal& page_ordinal) const override; 49 const syncer::StringOrdinal& page_ordinal) const override;
52 virtual syncer::StringOrdinal PageIntegerAsStringOrdinal( 50 syncer::StringOrdinal PageIntegerAsStringOrdinal(size_t page_index) override;
53 size_t page_index) override; 51 void SetExtensionVisible(const std::string& extension_id,
54 virtual void SetExtensionVisible(const std::string& extension_id, 52 bool visible) override;
55 bool visible) override;
56 53
57 private: 54 private:
58 DISALLOW_COPY_AND_ASSIGN(NullAppSorting); 55 DISALLOW_COPY_AND_ASSIGN(NullAppSorting);
59 }; 56 };
60 57
61 } // namespace extensions 58 } // namespace extensions
62 59
63 #endif // EXTENSIONS_BROWSER_NULL_APP_SORTING_H_ 60 #endif // EXTENSIONS_BROWSER_NULL_APP_SORTING_H_
OLDNEW
« no previous file with comments | « extensions/browser/mock_extension_system.h ('k') | extensions/browser/process_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698