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

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

Issue 4687005: Track permissions granted to extensions in prefs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: missed updating a method call Created 10 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) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 <set> 7 #include <set>
8 8
9 #include "app/l10n_util.h" 9 #include "app/l10n_util.h"
10 #include "app/resource_bundle.h" 10 #include "app/resource_bundle.h"
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 } 74 }
75 return false; 75 return false;
76 } 76 }
77 77
78 CrxInstaller::CrxInstaller(const FilePath& install_directory, 78 CrxInstaller::CrxInstaller(const FilePath& install_directory,
79 ExtensionsService* frontend, 79 ExtensionsService* frontend,
80 ExtensionInstallUI* client) 80 ExtensionInstallUI* client)
81 : install_directory_(install_directory), 81 : install_directory_(install_directory),
82 install_source_(Extension::INTERNAL), 82 install_source_(Extension::INTERNAL),
83 delete_source_(false), 83 delete_source_(false),
84 allow_privilege_increase_(false),
85 is_gallery_install_(false), 84 is_gallery_install_(false),
86 create_app_shortcut_(false), 85 create_app_shortcut_(false),
87 frontend_(frontend), 86 frontend_(frontend),
88 client_(client), 87 client_(client),
89 apps_require_extension_mime_type_(false), 88 apps_require_extension_mime_type_(false),
90 allow_silent_install_(false) { 89 allow_silent_install_(false) {
91 extensions_enabled_ = frontend_->extensions_enabled(); 90 extensions_enabled_ = frontend_->extensions_enabled();
92 } 91 }
93 92
94 CrxInstaller::~CrxInstaller() { 93 CrxInstaller::~CrxInstaller() {
(...skipping 307 matching lines...) Expand 10 before | Expand all | Expand 10 after
402 401
403 void CrxInstaller::ReportSuccessFromUIThread() { 402 void CrxInstaller::ReportSuccessFromUIThread() {
404 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 403 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
405 404
406 // If there is a client, tell the client about installation. 405 // If there is a client, tell the client about installation.
407 if (client_) 406 if (client_)
408 client_->OnInstallSuccess(extension_.get()); 407 client_->OnInstallSuccess(extension_.get());
409 408
410 // Tell the frontend about the installation and hand off ownership of 409 // Tell the frontend about the installation and hand off ownership of
411 // extension_ to it. 410 // extension_ to it.
412 frontend_->OnExtensionInstalled(extension_, allow_privilege_increase_); 411 frontend_->OnExtensionInstalled(extension_);
413 extension_ = NULL; 412 extension_ = NULL;
414 413
415 // We're done. We don't post any more tasks to ourselves so we are deleted 414 // We're done. We don't post any more tasks to ourselves so we are deleted
416 // soon. 415 // soon.
417 } 416 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698