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

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

Issue 10826188: Sharing shell_util_unittest code to verify shortcuts (Closed) Base URL: https://src.chromium.org/chrome/trunk/src/
Patch Set: 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
« no previous file with comments | « no previous file | chrome/installer/util/shell_util.cc » ('j') | chrome/installer/util/shell_util.cc » ('J')
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 // 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 421 matching lines...) Expand 10 before | Expand all | Expand 10 after
432 // should call GetCurrentInstallationSuffix(). 432 // should call GetCurrentInstallationSuffix().
433 static bool GetOldUserSpecificRegistrySuffix(string16* suffix); 433 static bool GetOldUserSpecificRegistrySuffix(string16* suffix);
434 434
435 // Returns the base32 encoding (using the [A-Z2-7] alphabet) of |bytes|. 435 // Returns the base32 encoding (using the [A-Z2-7] alphabet) of |bytes|.
436 // |size| is the length of |bytes|. 436 // |size| is the length of |bytes|.
437 // Note: This method does not suffix the output with '=' signs as technically 437 // Note: This method does not suffix the output with '=' signs as technically
438 // required by the base32 standard for inputs that aren't a multiple of 5 438 // required by the base32 standard for inputs that aren't a multiple of 5
439 // bytes. 439 // bytes.
440 static string16 ByteArrayToBase32(const uint8* bytes, size_t size); 440 static string16 ByteArrayToBase32(const uint8* bytes, size_t size);
441 441
442 // Verify that a shortcut exists with expected information (for testing)
gab 2012/08/07 19:18:25 s/with/with the/ Remove "(for testing)". Add '.'
Halli 2012/08/07 20:52:16 Done.
443 // |exe_path| shortcut's exe
444 // |shortcut| path to shortcut
445 // |description| shortcut's description
446 // |icon_index| in master prefs
Halli 2012/08/07 17:16:52 I got presubmit warnings about using wide strings,
gab 2012/08/07 19:18:25 No, replace all std::wstring by string16.
gab 2012/08/07 19:18:25 |icon_index| The icon's index in the exe.
Halli 2012/08/07 20:52:16 Done.
Halli 2012/08/07 20:52:16 Done.
447 static bool VerifyChromeShortcut(const std::wstring& exe_path,
448 const std::wstring& shortcut,
gab 2012/08/07 19:18:25 ident to match variable above
Halli 2012/08/07 20:52:16 Done.
449 const std::wstring& description,
450 int icon_index);
442 private: 451 private:
443 DISALLOW_COPY_AND_ASSIGN(ShellUtil); 452 DISALLOW_COPY_AND_ASSIGN(ShellUtil);
444 }; 453 };
445 454
446 455
447 #endif // CHROME_INSTALLER_UTIL_SHELL_UTIL_H_ 456 #endif // CHROME_INSTALLER_UTIL_SHELL_UTIL_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/installer/util/shell_util.cc » ('j') | chrome/installer/util/shell_util.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698