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

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

Issue 64953004: Split extensions::MenuManager instance out from ExtensionService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: unit tests compile Created 7 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) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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 #include "chrome/browser/extensions/extension_service.h" 5 #include "chrome/browser/extensions/extension_service.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <iterator> 8 #include <iterator>
9 #include <set> 9 #include <set>
10 10
(...skipping 314 matching lines...) Expand 10 before | Expand all | Expand 10 after
325 extension_prefs_(extension_prefs), 325 extension_prefs_(extension_prefs),
326 blacklist_(blacklist), 326 blacklist_(blacklist),
327 settings_frontend_(extensions::SettingsFrontend::Create(profile)), 327 settings_frontend_(extensions::SettingsFrontend::Create(profile)),
328 extension_sync_service_(NULL), 328 extension_sync_service_(NULL),
329 pending_extension_manager_(*this), 329 pending_extension_manager_(*this),
330 install_directory_(install_directory), 330 install_directory_(install_directory),
331 extensions_enabled_(extensions_enabled), 331 extensions_enabled_(extensions_enabled),
332 show_extensions_prompts_(true), 332 show_extensions_prompts_(true),
333 install_updates_when_idle_(true), 333 install_updates_when_idle_(true),
334 ready_(ready), 334 ready_(ready),
335 menu_manager_(profile),
336 update_once_all_providers_are_ready_(false), 335 update_once_all_providers_are_ready_(false),
337 browser_terminating_(false), 336 browser_terminating_(false),
338 installs_delayed_for_gc_(false), 337 installs_delayed_for_gc_(false),
339 is_first_run_(false) { 338 is_first_run_(false) {
340 #if defined(OS_CHROMEOS) 339 #if defined(OS_CHROMEOS)
341 disable_garbage_collection_ = false; 340 disable_garbage_collection_ = false;
342 #endif 341 #endif
343 CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 342 CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
344 343
345 // Figure out if extension installation should be enabled. 344 // Figure out if extension installation should be enabled.
(...skipping 2459 matching lines...) Expand 10 before | Expand all | Expand 10 after
2805 } 2804 }
2806 2805
2807 void ExtensionService::AddUpdateObserver(extensions::UpdateObserver* observer) { 2806 void ExtensionService::AddUpdateObserver(extensions::UpdateObserver* observer) {
2808 update_observers_.AddObserver(observer); 2807 update_observers_.AddObserver(observer);
2809 } 2808 }
2810 2809
2811 void ExtensionService::RemoveUpdateObserver( 2810 void ExtensionService::RemoveUpdateObserver(
2812 extensions::UpdateObserver* observer) { 2811 extensions::UpdateObserver* observer) {
2813 update_observers_.RemoveObserver(observer); 2812 update_observers_.RemoveObserver(observer);
2814 } 2813 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698