Unify the version string to be displayed on "About Chromium" dialog.

chrome/browser/ui/views/about_chrome_view.h

Index: chrome/browser/ui/views/about_chrome_view.h
diff --git a/chrome/browser/ui/views/about_chrome_view.h b/chrome/browser/ui/views/about_chrome_view.h
index 35c02ec60ef439917ad5493809340c57a5406bca..7a4426a6ea7c328547dc0eb4afc8ebbc0a7c7d05 100644
--- a/chrome/browser/ui/views/about_chrome_view.h
+++ b/chrome/browser/ui/views/about_chrome_view.h
@@ -95,7 +95,6 @@ class AboutChromeView : public views::View,
   views::ImageView* about_dlg_background_logo_;
   views::Label* about_title_label_;
   views::Textfield* version_label_;
-  views::Textfield* os_version_label_;

[tony, 2011/06/14 18:56:31]

I suspect this is here for a reason and we shouldn't remove it.  It probably
makes it easier for bug reporters to include the OS version.

[haraken1, 2011/06/15 04:20:41]

I think it is OK to remove |os_version_label_| since in this patch the OS name
is included in |version_label_|. Previously, Windows shows the OS name in
|os_version_label_|, which is displayed just under |version_label_|. Linux shows
the OS name in |version_label_|. Mac shows no OS name. Thus, my intent is to
unify their formats, specifically, all version information is included in
|version_label_| so that people can copy and paste it in one line. For this
reason, I guess that |os_version_label_| can be removed from here. 

   views::Label* copyright_label_;
   views::Label* main_text_label_;
   int main_text_label_height_;
@@ -135,12 +134,6 @@ class AboutChromeView : public views::View,
   scoped_refptr<GoogleUpdate> google_updater_;
 #endif
 
-  // Our current version.
-  std::string current_version_;
-
-  // Additional information about the version (channel and build number).
-  std::string version_details_;
-
   // The version Google Update reports is available to us.
   std::wstring new_version_available_;