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

Side by Side Diff: chrome/browser/download/download_crx_util.h

Issue 10388252: Refactoring ExtenionInstallUI to abstract the Browser references. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Minor clean-ups Created 8 years, 7 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
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 // Download code which handles CRX files (extensions, themes, apps, ...). 5 // Download code which handles CRX files (extensions, themes, apps, ...).
6 6
7 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_CRX_UTIL_H_ 7 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_CRX_UTIL_H_
8 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_CRX_UTIL_H_ 8 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_CRX_UTIL_H_
9 9
10 #pragma once 10 #pragma once
11 11
12 #include "base/basictypes.h" 12 #include "base/basictypes.h"
13 #include "base/memory/ref_counted.h" 13 #include "base/memory/ref_counted.h"
14 14
15 class CrxInstaller; 15 class CrxInstaller;
16 class ExtensionInstallUI; 16 class ExtensionInstallPrompt;
17 class Profile; 17 class Profile;
18 18
19 namespace content { 19 namespace content {
20 class DownloadItem; 20 class DownloadItem;
21 } 21 }
22 22
23 namespace download_crx_util { 23 namespace download_crx_util {
24 24
25 // Allow tests to install a mock extension install UI object, to fake 25 // Allow tests to install a mock extension install Prompt object, to fake
Yoyo Zhou 2012/05/23 23:17:49 nit: just ExtensionInstallPrompt
Jay Civelli 2012/05/30 20:20:35 Done.
26 // user clicks on the permissions dialog. Each installed mock object 26 // user clicks on the permissions dialog. Each installed mock object
27 // is only used once. If you want to return a mock for two different 27 // is only used once. If you want to return a mock for two different
28 // installs, you need to call this function once before the first 28 // installs, you need to call this function once before the first
29 // install, and again after the first install and before the second. 29 // install, and again after the first install and before the second.
30 void SetMockInstallUIForTesting(ExtensionInstallUI* mock_ui); 30 void SetMockInstallPromptForTesting(ExtensionInstallPrompt* mock_prompt);
31 31
32 // Start installing a downloaded item item as a CRX (extension, theme, app, 32 // Start installing a downloaded item item as a CRX (extension, theme, app,
33 // ...). The installer does work on the file thread, so the installation 33 // ...). The installer does work on the file thread, so the installation
34 // is not complete when this function returns. Returns the object managing 34 // is not complete when this function returns. Returns the object managing
35 // the installation. 35 // the installation.
36 scoped_refptr<CrxInstaller> OpenChromeExtension( 36 scoped_refptr<CrxInstaller> OpenChromeExtension(
37 Profile* profile, 37 Profile* profile,
38 const content::DownloadItem& download_item); 38 const content::DownloadItem& download_item);
39 39
40 } // namespace download_crx_util 40 } // namespace download_crx_util
41 41
42 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_CRX_UTIL_H_ 42 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_CRX_UTIL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698