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

Side by Side Diff: chrome/browser/apps/shortcut_manager.h

Issue 668903002: Standardize usage of virtual/override/final in chrome/browser/apps/ (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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 CHROME_BROWSER_APPS_SHORTCUT_MANAGER_H_ 5 #ifndef CHROME_BROWSER_APPS_SHORTCUT_MANAGER_H_
6 #define CHROME_BROWSER_APPS_SHORTCUT_MANAGER_H_ 6 #define CHROME_BROWSER_APPS_SHORTCUT_MANAGER_H_
7 7
8 #include "base/memory/weak_ptr.h" 8 #include "base/memory/weak_ptr.h"
9 #include "base/scoped_observer.h" 9 #include "base/scoped_observer.h"
10 #include "chrome/browser/profiles/profile_info_cache_observer.h" 10 #include "chrome/browser/profiles/profile_info_cache_observer.h"
(...skipping 14 matching lines...) Expand all
25 25
26 // This class manages the installation of shortcuts for platform apps. 26 // This class manages the installation of shortcuts for platform apps.
27 class AppShortcutManager : public KeyedService, 27 class AppShortcutManager : public KeyedService,
28 public extensions::ExtensionRegistryObserver, 28 public extensions::ExtensionRegistryObserver,
29 public ProfileInfoCacheObserver { 29 public ProfileInfoCacheObserver {
30 public: 30 public:
31 static void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry); 31 static void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry);
32 32
33 explicit AppShortcutManager(Profile* profile); 33 explicit AppShortcutManager(Profile* profile);
34 34
35 virtual ~AppShortcutManager(); 35 ~AppShortcutManager() override;
36 36
37 // Updates all shortcuts if kAppShortcutsVersion in prefs is less than 37 // Updates all shortcuts if kAppShortcutsVersion in prefs is less than
38 // kCurrentAppShortcutsVersion. 38 // kCurrentAppShortcutsVersion.
39 void UpdateShortcutsForAllAppsIfNeeded(); 39 void UpdateShortcutsForAllAppsIfNeeded();
40 40
41 // extensions::ExtensionRegistryObserver. 41 // extensions::ExtensionRegistryObserver.
42 virtual void OnExtensionWillBeInstalled( 42 void OnExtensionWillBeInstalled(content::BrowserContext* browser_context,
43 content::BrowserContext* browser_context, 43 const extensions::Extension* extension,
44 const extensions::Extension* extension, 44 bool is_update,
45 bool is_update, 45 bool from_ephemeral,
46 bool from_ephemeral, 46 const std::string& old_name) override;
47 const std::string& old_name) override; 47 void OnExtensionUninstalled(content::BrowserContext* browser_context,
48 virtual void OnExtensionUninstalled( 48 const extensions::Extension* extension,
49 content::BrowserContext* browser_context, 49 extensions::UninstallReason reason) override;
50 const extensions::Extension* extension,
51 extensions::UninstallReason reason) override;
52 50
53 // ProfileInfoCacheObserver. 51 // ProfileInfoCacheObserver.
54 virtual void OnProfileWillBeRemoved( 52 void OnProfileWillBeRemoved(const base::FilePath& profile_path) override;
55 const base::FilePath& profile_path) override;
56 53
57 private: 54 private:
58 void DeleteApplicationShortcuts(const extensions::Extension* extension); 55 void DeleteApplicationShortcuts(const extensions::Extension* extension);
59 56
60 Profile* profile_; 57 Profile* profile_;
61 bool is_profile_info_cache_observer_; 58 bool is_profile_info_cache_observer_;
62 PrefService* prefs_; 59 PrefService* prefs_;
63 60
64 ScopedObserver<extensions::ExtensionRegistry, 61 ScopedObserver<extensions::ExtensionRegistry,
65 extensions::ExtensionRegistryObserver> 62 extensions::ExtensionRegistryObserver>
66 extension_registry_observer_; 63 extension_registry_observer_;
67 64
68 base::WeakPtrFactory<AppShortcutManager> weak_ptr_factory_; 65 base::WeakPtrFactory<AppShortcutManager> weak_ptr_factory_;
69 66
70 DISALLOW_COPY_AND_ASSIGN(AppShortcutManager); 67 DISALLOW_COPY_AND_ASSIGN(AppShortcutManager);
71 }; 68 };
72 69
73 #endif // CHROME_BROWSER_APPS_SHORTCUT_MANAGER_H_ 70 #endif // CHROME_BROWSER_APPS_SHORTCUT_MANAGER_H_
OLDNEW
« no previous file with comments | « chrome/browser/apps/per_app_settings_service_factory.h ('k') | chrome/browser/apps/shortcut_manager_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698