Simplify VersionInfo code, avoid hitting sandbox IPC constantly on Windows

chrome/browser/extensions/updater/extension_downloader.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/updater/extension_downloader.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 654 matching lines @@
           continue;
         }
       }
     }
 
     // If the update specifies a browser minimum version, do we qualify?
     if (update->browser_min_version.length() > 0) {
       // First determine the browser version if we haven't already.
       if (!browser_version.IsValid()) {
         chrome::VersionInfo version_info;
-        if (version_info.is_valid())
-          browser_version = Version(version_info.Version());
+        browser_version = Version(version_info.Version());
       }
       Version browser_min_version(update->browser_min_version);
       if (browser_version.IsValid() && browser_min_version.IsValid() &&
           browser_min_version.CompareTo(browser_version) > 0) {
         // TODO(asargent) - We may want this to show up in the extensions UI
         // eventually. (http://crbug.com/12547).
         LOG(WARNING) << "Updated version of extension " << id
                      << " available, but requires chrome version "
                      << update->browser_min_version;
         continue;
@@ skipping 269 matching lines @@
       ping_mode = ManifestFetchData::PING_WITH_METRICS;
     } else {
       ping_mode = ManifestFetchData::PING;
     }
   }
   return new ManifestFetchData(
       update_url, request_id, brand_code_, manifest_query_params_, ping_mode);
 }
 
 }  // namespace extensions