Refactor compile_js.gypi to support script_args and closure_args

third_party/closure_compiler/compiled_resources.gyp

Index: third_party/closure_compiler/compiled_resources.gyp
diff --git a/third_party/closure_compiler/compiled_resources.gyp b/third_party/closure_compiler/compiled_resources.gyp
index e9bbda7dbcc1539b2d700a6723c65b8401a1a7de..20427df79f8444a3f1fd03063cf758e51f27121e 100644
--- a/third_party/closure_compiler/compiled_resources.gyp
+++ b/third_party/closure_compiler/compiled_resources.gyp
@@ -25,6 +25,7 @@
         '../../chrome/browser/resources/options/compiled_resources.gyp:*',
         '../../chrome/browser/resources/ntp4/compiled_resources.gyp:*',
         '../../chrome/browser/resources/uber/compiled_resources.gyp:*',
+        '../../remoting/remoting_webapp_compile.gypi:*',

[Dan Beam, 2015/06/12 21:45:06]

hmmm, i didn't know .gypi:* worked. great.

[Theresa, 2015/06/12 22:12:15]

It works in other dependency lists in our code (and in my local tests building
the ar_sample_app target in remoting_webapp.gyp) so I'm 99% sure it works here
too. I haven't actually tried to build this target locally... that would be a
good test to run before landing.

         '../../ui/file_manager/audio_player/js/compiled_resources.gyp:*',
         '../../ui/file_manager/file_manager/background/js/compiled_resources.gyp:*',
         '../../ui/file_manager/file_manager/foreground/js/compiled_resources.gyp:*',
@@ -37,9 +38,6 @@
         '../../ui/webui/resources/js/compiled_resources.gyp:*',
         '../../ui/webui/resources/js/cr/ui/compiled_resources.gyp:*',
       ],
-      'includes': [
-        '../../remoting/remoting_webapp_compile.gypi',
-      ],
     },
   ]
 }