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

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

Issue 11359081: Lazy initialization for ProcessesEventRouter. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 | 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 #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 <list> 8 #include <list>
9 #include <map> 9 #include <map>
10 #include <set> 10 #include <set>
(...skipping 862 matching lines...) Expand 10 before | Expand all | Expand 10 after
873 // Keeps track of app notifications. 873 // Keeps track of app notifications.
874 scoped_refptr<extensions::AppNotificationManager> app_notification_manager_; 874 scoped_refptr<extensions::AppNotificationManager> app_notification_manager_;
875 875
876 // Keeps track of favicon-sized omnibox icons for extensions. 876 // Keeps track of favicon-sized omnibox icons for extensions.
877 ExtensionIconManager omnibox_icon_manager_; 877 ExtensionIconManager omnibox_icon_manager_;
878 ExtensionIconManager omnibox_popup_icon_manager_; 878 ExtensionIconManager omnibox_popup_icon_manager_;
879 879
880 // Flag to make sure event routers are only initialized once. 880 // Flag to make sure event routers are only initialized once.
881 bool event_routers_initialized_; 881 bool event_routers_initialized_;
882 882
883 // TODO(yoz): None of these should be owned by ExtensionService.
884 // crbug.com/159265
883 scoped_ptr<HistoryExtensionEventRouter> history_event_router_; 885 scoped_ptr<HistoryExtensionEventRouter> history_event_router_;
884 886
885 scoped_ptr<extensions::BrowserEventRouter> browser_event_router_; 887 scoped_ptr<extensions::BrowserEventRouter> browser_event_router_;
886 888
887 scoped_ptr<extensions::WindowEventRouter> window_event_router_; 889 scoped_ptr<extensions::WindowEventRouter> window_event_router_;
888 890
889 scoped_ptr<extensions::PreferenceEventRouter> preference_event_router_; 891 scoped_ptr<extensions::PreferenceEventRouter> preference_event_router_;
890 892
891 scoped_ptr<BookmarkExtensionEventRouter> bookmark_event_router_; 893 scoped_ptr<BookmarkExtensionEventRouter> bookmark_event_router_;
892 894
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
951 #endif 953 #endif
952 954
953 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, 955 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
954 InstallAppsWithUnlimtedStorage); 956 InstallAppsWithUnlimtedStorage);
955 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest, 957 FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
956 InstallAppsAndCheckStorageProtection); 958 InstallAppsAndCheckStorageProtection);
957 DISALLOW_COPY_AND_ASSIGN(ExtensionService); 959 DISALLOW_COPY_AND_ASSIGN(ExtensionService);
958 }; 960 };
959 961
960 #endif // CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_ 962 #endif // CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698