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

Side by Side Diff: chrome/browser/shell_integration.h

Issue 11712003: [Fixit-Dec-2012] Also add dual_mode to Start Menu shortcuts in MigrateChromiumShortcuts. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add space to test shortcut paths Created 7 years, 11 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
« no previous file with comments | « no previous file | chrome/browser/shell_integration_unittest.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_BROWSER_SHELL_INTEGRATION_H_ 5 #ifndef CHROME_BROWSER_SHELL_INTEGRATION_H_
6 #define CHROME_BROWSER_SHELL_INTEGRATION_H_ 6 #define CHROME_BROWSER_SHELL_INTEGRATION_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 149
150 // Returns the location (path and index) of the Chromium icon, (e.g., 150 // Returns the location (path and index) of the Chromium icon, (e.g.,
151 // "C:\path\to\chrome.exe,0"). This is used to specify the icon to use 151 // "C:\path\to\chrome.exe,0"). This is used to specify the icon to use
152 // for the taskbar group on Win 7. 152 // for the taskbar group on Win 7.
153 static string16 GetChromiumIconLocation(); 153 static string16 GetChromiumIconLocation();
154 154
155 // Migrates existing chrome shortcuts by tagging them with correct app id. 155 // Migrates existing chrome shortcuts by tagging them with correct app id.
156 // see http://crbug.com/28104 156 // see http://crbug.com/28104
157 static void MigrateChromiumShortcuts(); 157 static void MigrateChromiumShortcuts();
158 158
159 // Migrates all shortcuts in |path| which point to |chrome_exe| such that they
160 // have the appropriate AppUserModelId. Also makes sure those shortcuts have
161 // the dual_mode property set if such is requested by |check_dual_mode|.
162 // Returns the number of shortcuts migrated.
163 // This method should not be called prior to Windows 7.
164 // This method is only public for the sake of tests and shouldn't be called
165 // externally otherwise.
166 static int MigrateShortcutsInPathInternal(
brettw 2013/01/04 21:04:08 I'd align all these after the ( on different lines
gab 2013/01/04 21:23:09 Done.
167 const FilePath& chrome_exe, const FilePath& path, bool check_dual_mode);
168
159 // Returns the path to the Start Menu shortcut for the given Chrome. 169 // Returns the path to the Start Menu shortcut for the given Chrome.
160 static FilePath GetStartMenuShortcut(const FilePath& chrome_exe); 170 static FilePath GetStartMenuShortcut(const FilePath& chrome_exe);
161 #endif // defined(OS_WIN) 171 #endif // defined(OS_WIN)
162 172
163 // The current default web client application UI state. This is used when 173 // The current default web client application UI state. This is used when
164 // querying if Chrome is the default browser or the default handler 174 // querying if Chrome is the default browser or the default handler
165 // application for a url protocol, and communicates the state and result of 175 // application for a url protocol, and communicates the state and result of
166 // a request. 176 // a request.
167 enum DefaultWebClientUIState { 177 enum DefaultWebClientUIState {
168 STATE_PROCESSING, 178 STATE_PROCESSING,
(...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
301 // Set Chrome as the default handler for this protocol. 311 // Set Chrome as the default handler for this protocol.
302 virtual bool SetAsDefault(bool interactive_permitted) OVERRIDE; 312 virtual bool SetAsDefault(bool interactive_permitted) OVERRIDE;
303 313
304 std::string protocol_; 314 std::string protocol_;
305 315
306 DISALLOW_COPY_AND_ASSIGN(DefaultProtocolClientWorker); 316 DISALLOW_COPY_AND_ASSIGN(DefaultProtocolClientWorker);
307 }; 317 };
308 }; 318 };
309 319
310 #endif // CHROME_BROWSER_SHELL_INTEGRATION_H_ 320 #endif // CHROME_BROWSER_SHELL_INTEGRATION_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/shell_integration_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698