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

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

Issue 660643002: [Refactor] Move creating a browser if necessary to ExtensionInstallDialogView (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 (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/unpacked_installer.h" 5 #include "chrome/browser/extensions/unpacked_installer.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/files/file_util.h" 9 #include "base/files/file_util.h"
10 #include "base/strings/string_util.h" 10 #include "base/strings/string_util.h"
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
65 }; 65 };
66 66
67 SimpleExtensionLoadPrompt::SimpleExtensionLoadPrompt( 67 SimpleExtensionLoadPrompt::SimpleExtensionLoadPrompt(
68 const Extension* extension, 68 const Extension* extension,
69 Profile* profile, 69 Profile* profile,
70 const base::Closure& callback) 70 const base::Closure& callback)
71 : extension_(extension), callback_(callback) { 71 : extension_(extension), callback_(callback) {
72 scoped_ptr<extensions::ExtensionInstallUI> ui( 72 scoped_ptr<extensions::ExtensionInstallUI> ui(
73 extensions::CreateExtensionInstallUI(profile)); 73 extensions::CreateExtensionInstallUI(profile));
74 install_ui_.reset(new ExtensionInstallPrompt( 74 install_ui_.reset(new ExtensionInstallPrompt(
75 profile, ui->GetDefaultInstallDialogParent(), NULL)); 75 profile, ui->GetDefaultInstallDialogParent()));
76 } 76 }
77 77
78 SimpleExtensionLoadPrompt::~SimpleExtensionLoadPrompt() { 78 SimpleExtensionLoadPrompt::~SimpleExtensionLoadPrompt() {
79 } 79 }
80 80
81 void SimpleExtensionLoadPrompt::ShowPrompt() { 81 void SimpleExtensionLoadPrompt::ShowPrompt() {
82 switch (ExtensionInstallPrompt::g_auto_confirm_for_tests) { 82 switch (ExtensionInstallPrompt::g_auto_confirm_for_tests) {
83 case ExtensionInstallPrompt::NONE: 83 case ExtensionInstallPrompt::NONE:
84 install_ui_->ConfirmInstall( 84 install_ui_->ConfirmInstall(
85 this, 85 this,
(...skipping 273 matching lines...) Expand 10 before | Expand all | Expand 10 after
359 359
360 PermissionsUpdater perms_updater(service_weak_->profile()); 360 PermissionsUpdater perms_updater(service_weak_->profile());
361 perms_updater.InitializePermissions(extension()); 361 perms_updater.InitializePermissions(extension());
362 perms_updater.GrantActivePermissions(extension()); 362 perms_updater.GrantActivePermissions(extension());
363 363
364 service_weak_->OnExtensionInstalled( 364 service_weak_->OnExtensionInstalled(
365 extension(), syncer::StringOrdinal(), kInstallFlagInstallImmediately); 365 extension(), syncer::StringOrdinal(), kInstallFlagInstallImmediately);
366 } 366 }
367 367
368 } // namespace extensions 368 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/extensions/external_install_error.cc ('k') | chrome/browser/extensions/webstore_install_with_prompt.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698