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

Side by Side Diff: chrome/installer/mini_installer/configuration.h

Issue 1277833002: Revert of Return Windows error code when create-process fails. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
« no previous file with comments | « chrome/installer/mini_installer.gypi ('k') | chrome/installer/mini_installer/configuration.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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_INSTALLER_MINI_INSTALLER_CONFIGURATION_H_ 5 #ifndef CHROME_INSTALLER_MINI_INSTALLER_CONFIGURATION_H_
6 #define CHROME_INSTALLER_MINI_INSTALLER_CONFIGURATION_H_ 6 #define CHROME_INSTALLER_MINI_INSTALLER_CONFIGURATION_H_
7 7
8 #include <windows.h> 8 #include <windows.h>
9 9
10 #include "chrome/installer/mini_installer/mini_string.h"
11
12 namespace mini_installer { 10 namespace mini_installer {
13 11
14 // A simple container of the mini_installer's configuration, as dictated by the 12 // A simple container of the mini_installer's configuration, as dictated by the
15 // command line used to invoke it. 13 // command line used to invoke it.
16 class Configuration { 14 class Configuration {
17 public: 15 public:
18 enum Operation { 16 enum Operation {
19 INSTALL_PRODUCT, 17 INSTALL_PRODUCT,
20 CLEANUP, 18 CLEANUP,
21 }; 19 };
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 const wchar_t* chrome_app_guid_; 66 const wchar_t* chrome_app_guid_;
69 const wchar_t* command_line_; 67 const wchar_t* command_line_;
70 int argument_count_; 68 int argument_count_;
71 Operation operation_; 69 Operation operation_;
72 bool has_chrome_; 70 bool has_chrome_;
73 bool has_chrome_frame_; 71 bool has_chrome_frame_;
74 bool is_multi_install_; 72 bool is_multi_install_;
75 bool is_system_level_; 73 bool is_system_level_;
76 const wchar_t* previous_version_; 74 const wchar_t* previous_version_;
77 75
78 protected:
79 typedef StackString<128> ValueString;
80
81 // Virtual for testing.
82 virtual bool ReadClientStateRegistryValue(
83 const HKEY root_key, const wchar_t* app_guid,
84 LONG* retval, ValueString& value);
85
86 private: 76 private:
87 Configuration(const Configuration&); 77 Configuration(const Configuration&);
88 Configuration& operator=(const Configuration&); 78 Configuration& operator=(const Configuration&);
89
90 void SetChromeAppGuid();
91 }; 79 };
92 80
93 } // namespace mini_installer 81 } // namespace mini_installer
94 82
95 #endif // CHROME_INSTALLER_MINI_INSTALLER_CONFIGURATION_H_ 83 #endif // CHROME_INSTALLER_MINI_INSTALLER_CONFIGURATION_H_
OLDNEW
« no previous file with comments | « chrome/installer/mini_installer.gypi ('k') | chrome/installer/mini_installer/configuration.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698