[Chromoting] Disable building 64bit chromoting.msi file.

remoting/host/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//build/config/win/manifest.gni")
 import("//build/util/version.gni")
 import("//chrome/version.gni")
 import("//remoting/remoting_enable.gni")
@@ skipping 1129 matching lines @@
     }
   } else {
     group("remoting_me2me_host_archive") {
     }
   }
 
   if (is_win && is_chrome_branded) {
     if (target_cpu == "x86") {
       # The script uses "ia32" instead of "x86".
       msi_script_arch = "ia32"
+

[joedow, 2016/06/20 22:15:04]

I think adding a comment would be good on why we only build this target for x86.

[Hzj_jie, 2016/06/21 02:14:55]

Done.

+      # GYP version: remoting/remoting_host_win.gyp:remoting_host_installation
+      action("remoting_host_installation") {
+        deps = [
+          "//remoting/host:remoting_me2me_host_archive",
+        ]
+        script = "../tools/zip2msi.py"
+        outputs = [
+          "$root_out_dir/chromoting.msi",
+        ]
+        args = [
+          "--wix_path",
+          rebase_path("//third_party/wix"),
+          "--intermediate_dir",
+          rebase_path("$root_gen_dir/installation", root_build_dir),
+          "--target_arch",
+          msi_script_arch,
+          rebase_path("$root_out_dir/remoting-me2me-host-$current_os.zip",
+                      root_build_dir),
+          rebase_path(outputs[0], root_build_dir),
+        ]
+      }
     } else {
-      msi_script_arch = target_cpu
-    }
-
-    # GYP version: remoting/remoting_host_win.gyp:remoting_host_installation
-    action("remoting_host_installation") {
-      deps = [
-        "//remoting/host:remoting_me2me_host_archive",
-      ]
-      script = "../tools/zip2msi.py"
-      outputs = [
-        "$root_out_dir/chromoting.msi",
-      ]
-      args = [
-        "--wix_path",
-        rebase_path("//third_party/wix"),
-        "--intermediate_dir",
-        rebase_path("$root_gen_dir/installation", root_build_dir),
-        "--target_arch",
-        msi_script_arch,
-        rebase_path("$root_out_dir/remoting-me2me-host-$current_os.zip",
-                    root_build_dir),
-        rebase_path(outputs[0], root_build_dir),
-      ]
+      group("remoting_host_installation") {

[joedow, 2016/06/20 22:15:04]

Is this empty target needed?  The GYP change just removes the target for x64,
why is there an empty one here for GN?

[Hzj_jie, 2016/06/21 02:14:55]

In GYP, the default empty target has already been added.

+      }
     }
   }
 }