extensions: Use Version ctor instead of calling GetVersionFromString().

chrome/browser/extensions/extension_updater.cc

Index: chrome/browser/extensions/extension_updater.cc
diff --git a/chrome/browser/extensions/extension_updater.cc b/chrome/browser/extensions/extension_updater.cc
index 2bd4e6b22fbd3e3928e1fd7eded5256230d817fe..2959077b480019e3b5738c2524d684111a38d806 100644
--- a/chrome/browser/extensions/extension_updater.cc
+++ b/chrome/browser/extensions/extension_updater.cc
@@ -313,12 +313,11 @@ void ManifestFetchesBuilder::AddPendingExtension(
   // Use a zero version to ensure that a pending extension will always
   // be updated, and thus installed (assuming all extensions have
   // non-zero versions).
-  scoped_ptr<Version> version(
-      Version::GetVersionFromString("0.0.0.0"));
+  Version version("0.0.0.0");
+  DCHECK(version.IsValid());

[cpu_(ooo_6.6-7.5), 2011/09/01 02:11:32]

that dcheck is checking a constant from the line before. Looks something that
goes into a unit test instead.

 
-  AddExtensionData(
-      info.install_source(), id, *version,
-      Extension::TYPE_UNKNOWN, info.update_url(), "");
+  AddExtensionData(info.install_source(), id, version,
+                   Extension::TYPE_UNKNOWN, info.update_url(), "");
 }
 
 void ManifestFetchesBuilder::ReportStats() const {