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

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

Issue 10213010: Don't register or use the DelegateExecute verb handler if it isn't present. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: logging tweak Created 8 years, 8 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 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 virtual int GetIconIndex(); 115 virtual int GetIconIndex();
116 116
117 virtual bool GetChromeChannel(std::wstring* channel); 117 virtual bool GetChromeChannel(std::wstring* channel);
118 118
119 // Returns true if the distribution includes a DelegateExecute verb handler, 119 // Returns true if the distribution includes a DelegateExecute verb handler,
120 // and provides the COM registration data if so: 120 // and provides the COM registration data if so:
121 // |handler_class_uuid| is the CommandExecuteImpl class UUID. 121 // |handler_class_uuid| is the CommandExecuteImpl class UUID.
122 // |type_lib_uuid| and |type_lib_version| identify its type library. 122 // |type_lib_uuid| and |type_lib_version| identify its type library.
123 // |interface_uuid| is the ICommandExecuteImpl interface UUID. 123 // |interface_uuid| is the ICommandExecuteImpl interface UUID.
124 // Only non-null parameters will be set, others will be ignored. 124 // Only non-null parameters will be set, others will be ignored.
125 // Implementations that only provide a DelegateExecute handler for use on
126 // certain OS versions must only return true when run on those supported
127 // systems.
125 virtual bool GetDelegateExecuteHandlerData(string16* handler_class_uuid, 128 virtual bool GetDelegateExecuteHandlerData(string16* handler_class_uuid,
126 string16* type_lib_uuid, 129 string16* type_lib_uuid,
127 string16* type_lib_version, 130 string16* type_lib_version,
128 string16* interface_uuid); 131 string16* interface_uuid);
129 132
130 virtual void UpdateInstallStatus(bool system_install, 133 virtual void UpdateInstallStatus(bool system_install,
131 installer::ArchiveType archive_type, 134 installer::ArchiveType archive_type,
132 installer::InstallStatus install_status); 135 installer::InstallStatus install_status);
133 136
134 // Gets the experiment details for a given language-brand combo. If |flavor| 137 // Gets the experiment details for a given language-brand combo. If |flavor|
(...skipping 27 matching lines...) Expand all
162 165
163 const Type type_; 166 const Type type_;
164 167
165 private: 168 private:
166 BrowserDistribution(); 169 BrowserDistribution();
167 170
168 DISALLOW_COPY_AND_ASSIGN(BrowserDistribution); 171 DISALLOW_COPY_AND_ASSIGN(BrowserDistribution);
169 }; 172 };
170 173
171 #endif // CHROME_INSTALLER_UTIL_BROWSER_DISTRIBUTION_H_ 174 #endif // CHROME_INSTALLER_UTIL_BROWSER_DISTRIBUTION_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698