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

Side by Side Diff: chrome/browser/extensions/extension_service.h

Issue 671873004: Migrates legacy packaged app data when it's upgraded to a platform app (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changes based on review comments Created 6 years, 1 month 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 (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 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_
6 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_ 6 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_
7 7
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 class CommandLine; 44 class CommandLine;
45 class SequencedTaskRunner; 45 class SequencedTaskRunner;
46 class Version; 46 class Version;
47 } 47 }
48 48
49 namespace content { 49 namespace content {
50 class DevToolsAgentHost; 50 class DevToolsAgentHost;
51 } 51 }
52 52
53 namespace extensions { 53 namespace extensions {
54 class AppDataMigrator;
54 class ComponentLoader; 55 class ComponentLoader;
55 class CrxInstaller; 56 class CrxInstaller;
56 class ExtensionActionStorageManager; 57 class ExtensionActionStorageManager;
57 class ExtensionDownloader; 58 class ExtensionDownloader;
58 class ExtensionDownloaderDelegate; 59 class ExtensionDownloaderDelegate;
59 class ExtensionErrorController; 60 class ExtensionErrorController;
60 class ExtensionRegistry; 61 class ExtensionRegistry;
61 class ExtensionSystem; 62 class ExtensionSystem;
62 class ExtensionUpdater; 63 class ExtensionUpdater;
63 class OneShotEvent; 64 class OneShotEvent;
(...skipping 647 matching lines...) Expand 10 before | Expand all | Expand 10 after
711 scoped_ptr<extensions::ExtensionActionStorageManager> 712 scoped_ptr<extensions::ExtensionActionStorageManager>
712 extension_action_storage_manager_; 713 extension_action_storage_manager_;
713 scoped_ptr<extensions::ManagementPolicy::Provider> 714 scoped_ptr<extensions::ManagementPolicy::Provider>
714 shared_module_policy_provider_; 715 shared_module_policy_provider_;
715 716
716 // The SharedModuleService used to check for import dependencies. 717 // The SharedModuleService used to check for import dependencies.
717 scoped_ptr<extensions::SharedModuleService> shared_module_service_; 718 scoped_ptr<extensions::SharedModuleService> shared_module_service_;
718 719
719 ObserverList<extensions::UpdateObserver, true> update_observers_; 720 ObserverList<extensions::UpdateObserver, true> update_observers_;
720 721
722 scoped_ptr<extensions::AppDataMigrator> app_data_migrator_;
benwells 2014/11/06 20:45:35 Nit: add a comment explaining briefly what it is f
723
721 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, 724 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
722 DestroyingProfileClearsExtensions); 725 DestroyingProfileClearsExtensions);
723 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, SetUnsetBlacklistInPrefs); 726 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, SetUnsetBlacklistInPrefs);
724 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, 727 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
725 BlacklistedExtensionWillNotInstall); 728 BlacklistedExtensionWillNotInstall);
726 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, 729 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
727 UnloadBlacklistedExtensionPolicy); 730 UnloadBlacklistedExtensionPolicy);
728 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, 731 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
729 WillNotLoadBlacklistedExtensionsFromDirectory); 732 WillNotLoadBlacklistedExtensionsFromDirectory);
730 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, ReloadBlacklistedExtension); 733 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, ReloadBlacklistedExtension);
731 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, BlacklistedInPrefsFromStartup); 734 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, BlacklistedInPrefsFromStartup);
732 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, 735 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
733 GreylistedExtensionDisabled); 736 GreylistedExtensionDisabled);
734 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, 737 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
735 GreylistDontEnableManuallyDisabled); 738 GreylistDontEnableManuallyDisabled);
736 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, 739 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
737 GreylistUnknownDontChange); 740 GreylistUnknownDontChange);
738 741
739 DISALLOW_COPY_AND_ASSIGN(ExtensionService); 742 DISALLOW_COPY_AND_ASSIGN(ExtensionService);
740 }; 743 };
741 744
742 #endif // CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_ 745 #endif // CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698