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

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

Issue 10446111: Abstract suffixing logic away from GetApplicationName (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 2012 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 a class that contains various method related to branding. 5 // This file declares a class that contains various method related to branding.
6 6
7 #ifndef CHROME_INSTALLER_UTIL_BROWSER_DISTRIBUTION_H_ 7 #ifndef CHROME_INSTALLER_UTIL_BROWSER_DISTRIBUTION_H_
8 #define CHROME_INSTALLER_UTIL_BROWSER_DISTRIBUTION_H_ 8 #define CHROME_INSTALLER_UTIL_BROWSER_DISTRIBUTION_H_
9 #pragma once 9 #pragma once
10 10
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
60 static BrowserDistribution* GetSpecificDistribution(Type type); 60 static BrowserDistribution* GetSpecificDistribution(Type type);
61 61
62 Type GetType() const { return type_; } 62 Type GetType() const { return type_; }
63 63
64 virtual void DoPostUninstallOperations(const Version& version, 64 virtual void DoPostUninstallOperations(const Version& version,
65 const FilePath& local_data_path, 65 const FilePath& local_data_path,
66 const string16& distribution_data); 66 const string16& distribution_data);
67 67
68 virtual string16 GetAppGuid(); 68 virtual string16 GetAppGuid();
69 69
70 // Returns the name by which the program is registered with Default Programs. 70 // Returns the unsuffixed application name of this program.
grt (UTC plus 2) 2012/06/08 19:54:21 I think the mention of "Default Programs" was usef
gab 2012/06/11 16:04:40 Done.
71 // This is not a localized string suitable for presenting to a user. 71 // IMPORTANT: This should only be called by the installer which needs to make
72 virtual string16 GetApplicationName(); 72 // decisions on the suffixing of the upcoming install (or by logging code that
73 // doesn't care about the suffixes), not by external callers at run-time.
74 virtual string16 GetBaseAppName();
73 75
74 // Returns the localized name of the program. 76 // Returns the localized name of the program.
75 virtual string16 GetAppShortCutName(); 77 virtual string16 GetAppShortCutName();
76 78
77 virtual string16 GetAlternateApplicationName(); 79 virtual string16 GetAlternateApplicationName();
78 80
79 virtual string16 GetBrowserAppId(); 81 virtual string16 GetBrowserAppId();
80 82
81 virtual string16 GetInstallSubDir(); 83 virtual string16 GetInstallSubDir();
82 84
(...skipping 79 matching lines...) Expand 10 before | Expand all | Expand 10 after
162 164
163 const Type type_; 165 const Type type_;
164 166
165 private: 167 private:
166 BrowserDistribution(); 168 BrowserDistribution();
167 169
168 DISALLOW_COPY_AND_ASSIGN(BrowserDistribution); 170 DISALLOW_COPY_AND_ASSIGN(BrowserDistribution);
169 }; 171 };
170 172
171 #endif // CHROME_INSTALLER_UTIL_BROWSER_DISTRIBUTION_H_ 173 #endif // CHROME_INSTALLER_UTIL_BROWSER_DISTRIBUTION_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698