Enable gn release builds of remoting_client_plugin_newlib

remoting/webapp/build_template.gni

 # Copyright 2015 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.
 
 # Keep in sync with 'remoting_webapp' target in remoting/remoting_client.gypi.
 
 import("//build/config/chrome_build.gni")
 import("//build/config/features.gni")
 import("//build/util/version.gni")
 import("//remoting/remoting_locales.gni")
@@ skipping 207 matching lines @@
 
     dr_generated_html_files = [
       background_html_output,
       message_window_html_output,
       wcs_sandbox_html_output,
       main_html_output,
       public_session_html_output,
       "$target_gen_dir/credits.html",
     ]
 
-    # TODO(crbug.com/584516) - fix OOM issues and re-enable on win.
-    if (enable_nacl && !is_win) {
+    # Windows debug builds of remoting_client_plugin_newlib.pexe trigger OOM in
+    # arm-nacl-ld.gold.exe - enable for release builds only on Windows.
+    if (enable_nacl && (!is_win || !is_debug)) {
       pnacl_tc = "//build/toolchain/nacl:newlib_pnacl"
       pexe_label = "//remoting/client/plugin:remoting_client_plugin_newlib"
       pexe_dir = get_label_info("${pexe_label}($pnacl_tc)", "root_out_dir")
 
       extra_files += [ "$pexe_dir/remoting_client_plugin_newlib.pexe" ]
       if (is_debug) {
         extra_files += [ "$pexe_dir/remoting_client_plugin_newlib.pexe.debug" ]
       }
     }
 
     inputs = [ rebase_path("crd/manifest.json.jinja2") ] +
              remoting_version_files + rebase_path(remoting_webapp_crd_files) +
              extra_files + dr_generated_html_files
 
     outputs = [
       zip_path,
     ]
 
     deps = [
       ":$background_html",
       ":$locales_listfile",
       ":$main_html",
       ":$message_window_html",
       ":$public_session_html",
       ":$wcs_sandbox_html",
       "//remoting/resources",
       "//remoting/webapp:credits",
     ]
 
-    # TODO(crbug.com/584516) - fix OOM issues and re-enable on win.
-    if (enable_nacl && !is_win) {
+    # Windows debug builds of remoting_client_plugin_newlib.pexe trigger OOM in
+    # arm-nacl-ld.gold.exe - enable for release builds only on Windows.
+    if (enable_nacl && (!is_win || !is_debug)) {
       deps += [ "${pexe_label}($pnacl_tc)" ]
       if (is_debug) {
         deps += [ "${pexe_label}_debug($pnacl_tc)" ]
       }
     }
 
     # Create a file that contains a list of all the resource files needed
     # to build the webapp. This is needed to avoid problems on platforms that
     # limit the size of a command line.
     file_list = "$target_gen_dir/${target_name}_files.txt"
@@ skipping 17 matching lines @@
     args += [
       "--locales_listfile",
       rebase_path(locales_listfile_output, root_build_dir),
     ]
     args += [
       "--use_gcd",
       "$remoting_use_gcd",
     ]
   }
 }