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

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

Issue 10453041: Support for interactive set-chrome-as-default in Windows. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed for unittests + a typo or two. Created 8 years, 6 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 // This file declares methods that are useful for integrating Chrome in 5 // This file declares methods that are useful for integrating Chrome in
6 // Windows shell. These methods are all static and currently part of 6 // Windows shell. These methods are all static and currently part of
7 // ShellUtil class. 7 // ShellUtil class.
8 8
9 #ifndef CHROME_INSTALLER_UTIL_SHELL_UTIL_H_ 9 #ifndef CHROME_INSTALLER_UTIL_SHELL_UTIL_H_
10 #define CHROME_INSTALLER_UTIL_SHELL_UTIL_H_ 10 #define CHROME_INSTALLER_UTIL_SHELL_UTIL_H_
(...skipping 219 matching lines...) Expand 10 before | Expand all | Expand 10 after
230 // level or user level. If value has ShellChange::SYSTEM_LEVEL 230 // level or user level. If value has ShellChange::SYSTEM_LEVEL
231 // we should be running as admin user. 231 // we should be running as admin user.
232 // chrome_exe: The chrome.exe path to register as default browser. 232 // chrome_exe: The chrome.exe path to register as default browser.
233 // elevate_if_not_admin: On Vista if user is not admin, try to elevate for 233 // elevate_if_not_admin: On Vista if user is not admin, try to elevate for
234 // Chrome registration. 234 // Chrome registration.
235 static bool MakeChromeDefault(BrowserDistribution* dist, 235 static bool MakeChromeDefault(BrowserDistribution* dist,
236 int shell_change, 236 int shell_change,
237 const string16& chrome_exe, 237 const string16& chrome_exe,
238 bool elevate_if_not_admin); 238 bool elevate_if_not_admin);
239 239
240 // Shows to the user a system dialog where Chrome can be set as the
241 // default browser. This is intended for Windows 8, but is known to work
242 // on Windows 7, too. This is a blocking call.
gab 2012/05/31 22:17:20 Please change "This is intended for Windows 8, but
grt (UTC plus 2) 2012/06/01 14:36:39 Agreed. If you'd like to leave a note for posteri
motek. 2012/06/01 20:50:17 Done.
motek. 2012/06/01 20:50:17 Ack.
243 //
244 // Parameters as MakeChromeDefault.
245 static bool ShowMakeChromeDefaultSystemUI(BrowserDistribution* dist,
246 int shell_change,
grt (UTC plus 2) 2012/06/01 14:36:39 is this parameter needed? my understanding is tha
motek. 2012/06/01 20:50:17 Done.
247 const string16& chrome_exe,
248 bool elevate_if_not_admin);
249
240 // Make Chrome the default application for a protocol. 250 // Make Chrome the default application for a protocol.
241 // chrome_exe: The chrome.exe path to register as default browser. 251 // chrome_exe: The chrome.exe path to register as default browser.
242 // protocol: The protocol to register as the default handler for. 252 // protocol: The protocol to register as the default handler for.
243 static bool MakeChromeDefaultProtocolClient(BrowserDistribution* dist, 253 static bool MakeChromeDefaultProtocolClient(BrowserDistribution* dist,
244 const string16& chrome_exe, 254 const string16& chrome_exe,
245 const string16& protocol); 255 const string16& protocol);
246 256
247 // This method adds Chrome to the list that shows up in Add/Remove Programs-> 257 // This method adds Chrome to the list that shows up in Add/Remove Programs->
248 // Set Program Access and Defaults and also creates Chrome ProgIds under 258 // Set Program Access and Defaults and also creates Chrome ProgIds under
249 // Software\Classes. This method requires write access to HKLM so is just 259 // Software\Classes. This method requires write access to HKLM so is just
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
344 const string16& icon_path, 354 const string16& icon_path,
345 int icon_index, 355 int icon_index,
346 uint32 options); 356 uint32 options);
347 357
348 private: 358 private:
349 DISALLOW_COPY_AND_ASSIGN(ShellUtil); 359 DISALLOW_COPY_AND_ASSIGN(ShellUtil);
350 }; 360 };
351 361
352 362
353 #endif // CHROME_INSTALLER_UTIL_SHELL_UTIL_H_ 363 #endif // CHROME_INSTALLER_UTIL_SHELL_UTIL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698