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

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

Issue 1731483003: chrome: Add out-of-line copy ctors for complex classes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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) 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_BUNDLE_INSTALLER_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_BUNDLE_INSTALLER_H_
6 #define CHROME_BROWSER_EXTENSIONS_BUNDLE_INSTALLER_H_ 6 #define CHROME_BROWSER_EXTENSIONS_BUNDLE_INSTALLER_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include <string> 10 #include <string>
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
54 struct Item { 54 struct Item {
55 // Items are in the PENDING state until they've been installed, or the 55 // Items are in the PENDING state until they've been installed, or the
56 // install has failed or been canceled. 56 // install has failed or been canceled.
57 enum State { 57 enum State {
58 STATE_PENDING, 58 STATE_PENDING,
59 STATE_INSTALLED, 59 STATE_INSTALLED,
60 STATE_FAILED 60 STATE_FAILED
61 }; 61 };
62 62
63 Item(); 63 Item();
64 Item(const Item& other);
64 ~Item(); 65 ~Item();
65 66
66 // Gets the localized name, formatted for display in the bubble. 67 // Gets the localized name, formatted for display in the bubble.
67 base::string16 GetNameForDisplay() const; 68 base::string16 GetNameForDisplay() const;
68 69
69 std::string id; 70 std::string id;
70 std::string manifest; 71 std::string manifest;
71 std::string localized_name; 72 std::string localized_name;
72 GURL icon_url; 73 GURL icon_url;
73 SkBitmap icon; 74 SkBitmap icon;
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
208 base::Closure install_callback_; 209 base::Closure install_callback_;
209 210
210 base::WeakPtrFactory<BundleInstaller> weak_factory_; 211 base::WeakPtrFactory<BundleInstaller> weak_factory_;
211 212
212 DISALLOW_COPY_AND_ASSIGN(BundleInstaller); 213 DISALLOW_COPY_AND_ASSIGN(BundleInstaller);
213 }; 214 };
214 215
215 } // namespace extensions 216 } // namespace extensions
216 217
217 #endif // CHROME_BROWSER_EXTENSIONS_BUNDLE_INSTALLER_H_ 218 #endif // CHROME_BROWSER_EXTENSIONS_BUNDLE_INSTALLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698