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

Side by Side Diff: chrome/installer/setup/install.h

Issue 2456853002: Remove shortcut path fixups introduced to repair breakages caused by r378802. (Closed)
Patch Set: Created 4 years, 1 month 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/setup/install.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 // This file contains the specification of setup main functions. 5 // This file contains the specification of setup main functions.
6 6
7 #ifndef CHROME_INSTALLER_SETUP_INSTALL_H_ 7 #ifndef CHROME_INSTALLER_SETUP_INSTALL_H_
8 #define CHROME_INSTALLER_SETUP_INSTALL_H_ 8 #define CHROME_INSTALLER_SETUP_INSTALL_H_
9 9
10 #include <vector> 10 #include <vector>
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 43
44 enum InstallShortcutLevel { 44 enum InstallShortcutLevel {
45 // Install shortcuts for the current user only. 45 // Install shortcuts for the current user only.
46 CURRENT_USER, 46 CURRENT_USER,
47 // Install global shortcuts visible to all users. Note: the Quick Launch 47 // Install global shortcuts visible to all users. Note: the Quick Launch
48 // and taskbar pin shortcuts are still installed per-user (as they have no 48 // and taskbar pin shortcuts are still installed per-user (as they have no
49 // all-users version). 49 // all-users version).
50 ALL_USERS, 50 ALL_USERS,
51 }; 51 };
52 52
53 // Sets |new_target_path| as the new target path of all shortcuts in the
54 // location specified by |shortcut_location| and |dist| which either:
55 // - Point to a file rooted at |old_target_dir| whose name ends in
56 // |old_target_name_suffix|, or,
57 // - Have an icon rooted at |old_target_dir|.
58 void UpdatePerUserShortcutsInLocation(
59 const ShellUtil::ShortcutLocation shortcut_location,
60 BrowserDistribution* dist,
61 const base::FilePath& old_target_dir,
62 const base::FilePath& old_target_name_suffix,
63 const base::FilePath& new_target_path);
64
65 // Escape |att_value| as per the XML AttValue production 53 // Escape |att_value| as per the XML AttValue production
66 // (http://www.w3.org/TR/2008/REC-xml-20081126/#NT-AttValue) for a value in 54 // (http://www.w3.org/TR/2008/REC-xml-20081126/#NT-AttValue) for a value in
67 // single quotes. 55 // single quotes.
68 void EscapeXmlAttributeValueInSingleQuotes(base::string16* att_value); 56 void EscapeXmlAttributeValueInSingleQuotes(base::string16* att_value);
69 57
70 // Creates VisualElementsManifest.xml beside chrome.exe in |src_path| if 58 // Creates VisualElementsManifest.xml beside chrome.exe in |src_path| if
71 // |src_path|\VisualElements exists. 59 // |src_path|\VisualElements exists.
72 // Returns true unless the manifest is supposed to be created, but fails to be. 60 // Returns true unless the manifest is supposed to be created, but fails to be.
73 bool CreateVisualElementsManifest(const base::FilePath& src_path, 61 bool CreateVisualElementsManifest(const base::FilePath& src_path,
74 const base::Version& version); 62 const base::Version& version);
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 // Shortcut creation is skipped if the First Run beacon is present (unless 136 // Shortcut creation is skipped if the First Run beacon is present (unless
149 // |force| is set to true). 137 // |force| is set to true).
150 // |chrome| The installed product (must be a browser). 138 // |chrome| The installed product (must be a browser).
151 void HandleActiveSetupForBrowser(const base::FilePath& installation_root, 139 void HandleActiveSetupForBrowser(const base::FilePath& installation_root,
152 const Product& chrome, 140 const Product& chrome,
153 bool force); 141 bool force);
154 142
155 } // namespace installer 143 } // namespace installer
156 144
157 #endif // CHROME_INSTALLER_SETUP_INSTALL_H_ 145 #endif // CHROME_INSTALLER_SETUP_INSTALL_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/installer/setup/install.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698