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

Unified Diff: base/file_version_info_win.h

Issue 6897016: Base: Fix FileVersionInfo::CreateFileVersionInfoForCurrentModule so that (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 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 side-by-side diff with in-line comments
Download patch
Index: base/file_version_info_win.h
===================================================================
--- base/file_version_info_win.h (revision 82561)
+++ base/file_version_info_win.h (working copy)
@@ -16,10 +16,10 @@
struct tagVS_FIXEDFILEINFO;
typedef tagVS_FIXEDFILEINFO VS_FIXEDFILEINFO;
-class BASE_API FileVersionInfoWin : public FileVersionInfo {
+class FileVersionInfoWin : public FileVersionInfo {
wtc 2011/04/22 20:00:09 Why do you change from exporting the class to expo
rvargas (doing something else) 2011/04/22 21:40:05 Because I cannot derive from FileVersionInfo unles
public:
- FileVersionInfoWin(void* data, int language, int code_page);
- ~FileVersionInfoWin();
+ BASE_API FileVersionInfoWin(void* data, int language, int code_page);
+ BASE_API ~FileVersionInfoWin();
// Accessors to the different version properties.
// Returns an empty string if the property is not found.
@@ -41,11 +41,11 @@
virtual bool is_official_build();
// Lets you access other properties not covered above.
- bool GetValue(const wchar_t* name, std::wstring* value);
+ BASE_API bool GetValue(const wchar_t* name, std::wstring* value);
// Similar to GetValue but returns a wstring (empty string if the property
// does not exist).
- std::wstring GetStringValue(const wchar_t* name);
+ BASE_API std::wstring GetStringValue(const wchar_t* name);
// Get the fixed file info if it exists. Otherwise NULL
VS_FIXEDFILEINFO* fixed_file_info() { return fixed_file_info_; }

Powered by Google App Engine
This is Rietveld 408576698