Remove wow_helper.exe from Win 7 and higher.

chrome/installer/setup/install_worker.cc

 // Copyright (c) 2011 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.
 //
 // This file contains the definitions of the installer functions that build
 // the WorkItemList used to install the application.
 
 #include "chrome/installer/setup/install_worker.h"
 
 #include <shlobj.h>
@@ skipping 735 matching lines @@
         src_path.Append(installer::kChromeExe).value(),
         target_path.Append(installer::kChromeExe).value(),
         temp_path.value(),
         WorkItem::NEW_NAME_IF_IN_USE,
         new_chrome_exe.value());
   }
 
   // Extra executable for 64 bit systems.
   // NOTE: We check for "not disabled" so that if the API call fails, we play it
   // safe and copy the executable anyway.
+  // NOTE: the file wow_helper.exe is only needed for Vista and below.
   if (base::win::OSInfo::GetInstance()->wow64_status() !=
-      base::win::OSInfo::WOW64_DISABLED) {
+      base::win::OSInfo::WOW64_DISABLED &&
+      base::win::GetVersion() <= base::win::VERSION_VISTA) {

[grt (UTC plus 2), 2012/01/31 21:27:56]

I think you're about to shave a yak.  Server 2008 is nearly Vista, whereas
Server 2008 R2 is nearly Win7.  windows_version.cc doesn't handle this
distinction, but I suspect the difference matters here: wow_helper is probably
needed for Server 2008 but not Server 2008 R2.  I think the right thing to do is
to give OSInfo::OSInfo in windows_version.cc the power to distinguish these. 
See
http://msdn.microsoft.com/en-us/library/windows/desktop/ms724833(v=vs.85).aspx
for details.

[arthurhsu, 2012/01/31 22:07:35]

Please refer to Ricardo's reply.  BTW, I copied the condition check that Ricardo
mentioned from chromium/src/sandbox/src/interception.cc:427.

[grt (UTC plus 2), 2012/02/01 02:12:17]

Cool, thanks for the confirmation, Ricardo.

     install_list->AddMoveTreeWorkItem(
         src_path.Append(installer::kWowHelperExe).value(),
         target_path.Append(installer::kWowHelperExe).value(),
         temp_path.value(),
         WorkItem::ALWAYS_MOVE);
   }
 
   // In the past, we copied rather than moved for system level installs so that
   // the permissions of %ProgramFiles% would be picked up.  Now that |temp_path|
   // is in %ProgramFiles% for system level installs (and in %LOCALAPPDATA%
@@ skipping 487 matching lines @@
     } else {
       DCHECK(operation == REMOVE_COMMAND);
       work_item_list->AddDeleteRegKeyWorkItem(installer_state.root_key(),
                                               cmd_key)->set_log_message(
           "removing quick-enable-cf command");
     }
   }
 }
 
 }  // namespace installer