Make omaha, gcapi and uninstall registry accesses use Wow6432Node on 64-bit

chrome/installer/util/compat_checks.cc

 // Copyright (c) 2010 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/installer/util/compat_checks.h"
 
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "base/win/registry.h"
 
 namespace {
 
 // SEP stands for Symantec End Point Protection.
 std::wstring GetSEPVersion() {
   const wchar_t kProductKey[] =
       L"SOFTWARE\\Symantec\\Symantec Endpoint Protection\\SMC";
-  base::win::RegKey key(HKEY_LOCAL_MACHINE, kProductKey, KEY_READ);
+  base::win::RegKey key(HKEY_LOCAL_MACHINE, kProductKey,
+                        KEY_READ | KEY_WOW64_32KEY);

[grt (UTC plus 2), 2014/05/27 16:42:08]

won't this miss if Symantec ever ships a 64-bit SEP? do they ship one already
that we're missing?

[Will Harris, 2014/05/27 19:25:10]

We're only checking for anything before 11MR3, which was 32-bit only and from
2008.

[grt (UTC plus 2), 2014/05/27 22:06:19]

groovy. sounds like good info for a comment in the code. ;-)

[Will Harris, 2014/05/27 22:29:28]

Done.

   std::wstring version_str;
   key.ReadValue(L"ProductVersion", &version_str);
   return version_str;
 }
 
 // The product version should be a string like "11.0.3001.2224". This function
 // returns as params the first 3 values. Return value is false if anything
 // does not fit the format.
 bool ParseSEPVersion(const std::wstring& version, int* v0, int* v1, int* v2) {
   std::vector<std::wstring> v;
@@ skipping 22 matching lines @@
 
 }  // namespace
 
 bool HasIncompatibleSymantecEndpointVersion(const wchar_t* version) {
   int v0, v1, v2;
   std::wstring ver_str(version ? version : GetSEPVersion());
   if (!ParseSEPVersion(ver_str, &v0, &v1, &v2))
     return false;
   return IsBadSEPVersion(v0, v1, v2);
 }