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

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

Issue 11131002: Make show_dialog_callback optional in ExtensionInstallPrompt (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | chrome/browser/extensions/extension_install_prompt.cc » ('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 (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 #include "chrome/browser/extensions/crx_installer.h" 5 #include "chrome/browser/extensions/crx_installer.h"
6 6
7 #include <map> 7 #include <map>
8 #include <set> 8 #include <set>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
99 apps_require_extension_mime_type_(false), 99 apps_require_extension_mime_type_(false),
100 allow_silent_install_(false), 100 allow_silent_install_(false),
101 install_cause_(extension_misc::INSTALL_CAUSE_UNSET), 101 install_cause_(extension_misc::INSTALL_CAUSE_UNSET),
102 creation_flags_(Extension::NO_FLAGS), 102 creation_flags_(Extension::NO_FLAGS),
103 off_store_install_allow_reason_(OffStoreInstallDisallowed), 103 off_store_install_allow_reason_(OffStoreInstallDisallowed),
104 did_handle_successfully_(true), 104 did_handle_successfully_(true),
105 record_oauth2_grant_(false), 105 record_oauth2_grant_(false),
106 error_on_unsupported_requirements_(false), 106 error_on_unsupported_requirements_(false),
107 requirements_checker_(new extensions::RequirementsChecker()), 107 requirements_checker_(new extensions::RequirementsChecker()),
108 has_requirement_errors_(false) { 108 has_requirement_errors_(false) {
109 show_dialog_callback_ =
110 ExtensionInstallPrompt::GetDefaultShowDialogCallback();
111 if (!approval) 109 if (!approval)
112 return; 110 return;
113 111
114 CHECK(profile_->IsSameProfile(approval->profile)); 112 CHECK(profile_->IsSameProfile(approval->profile));
115 if (client_) { 113 if (client_) {
116 client_->install_ui()->SetUseAppInstalledBubble( 114 client_->install_ui()->SetUseAppInstalledBubble(
117 approval->use_app_installed_bubble); 115 approval->use_app_installed_bubble);
118 client_->install_ui()->SetSkipPostInstallUI(approval->skip_post_install_ui); 116 client_->install_ui()->SetSkipPostInstallUI(approval->skip_post_install_ui);
119 } 117 }
120 118
(...skipping 533 matching lines...) Expand 10 before | Expand all | Expand 10 after
654 // is problematic because they don't know anything about the 652 // is problematic because they don't know anything about the
655 // extension before it is unpacked, so they cannot filter based 653 // extension before it is unpacked, so they cannot filter based
656 // on the extension. 654 // on the extension.
657 content::NotificationService::current()->Notify( 655 content::NotificationService::current()->Notify(
658 chrome::NOTIFICATION_CRX_INSTALLER_DONE, 656 chrome::NOTIFICATION_CRX_INSTALLER_DONE,
659 content::Source<CrxInstaller>(this), 657 content::Source<CrxInstaller>(this),
660 content::Details<const Extension>(extension)); 658 content::Details<const Extension>(extension));
661 } 659 }
662 660
663 } // namespace extensions 661 } // namespace extensions
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/extensions/extension_install_prompt.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698