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

Side by Side Diff: chrome/browser/extensions/webstore_installer.h

Issue 10833058: Make DownloadItem derive SupportsUserData instead of re-implementing it. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge Created 8 years, 4 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 #ifndef CHROME_BROWSER_EXTENSIONS_WEBSTORE_INSTALLER_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_WEBSTORE_INSTALLER_H_
6 #define CHROME_BROWSER_EXTENSIONS_WEBSTORE_INSTALLER_H_ 6 #define CHROME_BROWSER_EXTENSIONS_WEBSTORE_INSTALLER_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 const std::string& error) = 0; 50 const std::string& error) = 0;
51 51
52 protected: 52 protected:
53 virtual ~Delegate() {} 53 virtual ~Delegate() {}
54 }; 54 };
55 55
56 // Contains information about what parts of the extension install process can 56 // Contains information about what parts of the extension install process can
57 // be skipped or modified. If one of these is present, it means that a CRX 57 // be skipped or modified. If one of these is present, it means that a CRX
58 // download was initiated by WebstoreInstaller. The Approval instance should 58 // download was initiated by WebstoreInstaller. The Approval instance should
59 // be checked further for additional details. 59 // be checked further for additional details.
60 struct Approval : public content::DownloadItem::ExternalData { 60 struct Approval : public base::SupportsUserData::Data {
61 static scoped_ptr<Approval> CreateWithInstallPrompt(Profile* profile); 61 static scoped_ptr<Approval> CreateWithInstallPrompt(Profile* profile);
62 static scoped_ptr<Approval> CreateWithNoInstallPrompt( 62 static scoped_ptr<Approval> CreateWithNoInstallPrompt(
63 Profile* profile, 63 Profile* profile,
64 const std::string& extension_id, 64 const std::string& extension_id,
65 scoped_ptr<base::DictionaryValue> parsed_manifest); 65 scoped_ptr<base::DictionaryValue> parsed_manifest);
66 66
67 virtual ~Approval(); 67 virtual ~Approval();
68 68
69 // The extension id that was approved for installation. 69 // The extension id that was approved for installation.
70 std::string extension_id; 70 std::string extension_id;
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 // transitions to state REMOVING. 161 // transitions to state REMOVING.
162 content::DownloadItem* download_item_; 162 content::DownloadItem* download_item_;
163 int flags_; 163 int flags_;
164 scoped_ptr<Approval> approval_; 164 scoped_ptr<Approval> approval_;
165 GURL download_url_; 165 GURL download_url_;
166 }; 166 };
167 167
168 } // namespace extensions 168 } // namespace extensions
169 169
170 #endif // CHROME_BROWSER_EXTENSIONS_WEBSTORE_INSTALLER_H_ 170 #endif // CHROME_BROWSER_EXTENSIONS_WEBSTORE_INSTALLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698