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

Side by Side Diff: chrome/installer/util/google_update_settings.h

Issue 974423002: Revert of Profile_Metrics integration with Keystone (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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/chrome_browser.gypi ('k') | chrome/installer/util/google_update_settings.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_UTIL_GOOGLE_UPDATE_SETTINGS_H_ 5 #ifndef CHROME_INSTALLER_UTIL_GOOGLE_UPDATE_SETTINGS_H_
6 #define CHROME_INSTALLER_UTIL_GOOGLE_UPDATE_SETTINGS_H_ 6 #define CHROME_INSTALLER_UTIL_GOOGLE_UPDATE_SETTINGS_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 200 matching lines...) Expand 10 before | Expand all | Expand 10 after
211 // archive_type: tells whether this is incremental install or not. 211 // archive_type: tells whether this is incremental install or not.
212 // install_return_code: if 0, means installation was successful. 212 // install_return_code: if 0, means installation was successful.
213 // value: current value of Google Update "ap" key. 213 // value: current value of Google Update "ap" key.
214 // Returns true if |value| is modified. 214 // Returns true if |value| is modified.
215 static bool UpdateGoogleUpdateApKey(installer::ArchiveType archive_type, 215 static bool UpdateGoogleUpdateApKey(installer::ArchiveType archive_type,
216 int install_return_code, 216 int install_return_code,
217 installer::ChannelInfo* value); 217 installer::ChannelInfo* value);
218 218
219 // This method updates the values that report how many profiles are in use 219 // This method updates the values that report how many profiles are in use
220 // and how many of those are signed-in. 220 // and how many of those are signed-in.
221 static void UpdateProfileCounts(size_t profiles_active, 221 static void UpdateProfileCounts(int profiles_active, int profiles_signedin);
222 size_t profiles_signedin);
223 222
224 // For system-level installs, we need to be able to communicate the results 223 // For system-level installs, we need to be able to communicate the results
225 // of the Toast Experiments back to Google Update. The problem is just that 224 // of the Toast Experiments back to Google Update. The problem is just that
226 // the experiment is run in the context of the user, which doesn't have 225 // the experiment is run in the context of the user, which doesn't have
227 // write access to the HKLM key that Google Update expects the results in. 226 // write access to the HKLM key that Google Update expects the results in.
228 // However, when we are about to switch contexts from system to user, we can 227 // However, when we are about to switch contexts from system to user, we can
229 // duplicate the handle to the registry key and pass it (through handle 228 // duplicate the handle to the registry key and pass it (through handle
230 // inheritance) to the newly created child process that is launched as the 229 // inheritance) to the newly created child process that is launched as the
231 // user, allowing the child process to write to the key, with the 230 // user, allowing the child process to write to the key, with the
232 // WriteGoogleUpdateSystemClientKey function below. 231 // WriteGoogleUpdateSystemClientKey function below.
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
314 // this will do nothing to |experiment_labels|. This will return true if the 313 // this will do nothing to |experiment_labels|. This will return true if the
315 // label did not exist, or was successfully read. 314 // label did not exist, or was successfully read.
316 static bool ReadExperimentLabels(bool system_install, 315 static bool ReadExperimentLabels(bool system_install,
317 base::string16* experiment_labels); 316 base::string16* experiment_labels);
318 317
319 private: 318 private:
320 DISALLOW_IMPLICIT_CONSTRUCTORS(GoogleUpdateSettings); 319 DISALLOW_IMPLICIT_CONSTRUCTORS(GoogleUpdateSettings);
321 }; 320 };
322 321
323 #endif // CHROME_INSTALLER_UTIL_GOOGLE_UPDATE_SETTINGS_H_ 322 #endif // CHROME_INSTALLER_UTIL_GOOGLE_UPDATE_SETTINGS_H_
OLDNEW
« no previous file with comments | « chrome/chrome_browser.gypi ('k') | chrome/installer/util/google_update_settings.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698