Add on_startup_page/compiled_resources.gyp

chrome/browser/resources/settings/on_startup_page/compiled_resources.gyp

Index: chrome/browser/resources/settings/on_startup_page/compiled_resources.gyp
diff --git a/chrome/browser/resources/settings/languages_page/compiled_resources.gyp b/chrome/browser/resources/settings/on_startup_page/compiled_resources.gyp
similarity index 53%
copy from chrome/browser/resources/settings/languages_page/compiled_resources.gyp
copy to chrome/browser/resources/settings/on_startup_page/compiled_resources.gyp
index b1976cd503cdb98fc6f664258c771d652a2158a0..90467cd50ab1921c5d0dd2496ab48b2c98e460b9 100644
--- a/chrome/browser/resources/settings/languages_page/compiled_resources.gyp
+++ b/chrome/browser/resources/settings/on_startup_page/compiled_resources.gyp
@@ -4,15 +4,27 @@
 {
   'targets': [
     {
-      'target_name': 'languages_page',
+      'target_name': 'on_startup_page',
       'variables': {
         'depends': [
           '../../../../../ui/webui/resources/js/compiled_resources.gyp:assert',
-          '../../../../../ui/webui/resources/js/compiled_resources.gyp:cr',
           '../../../../../ui/webui/resources/js/compiled_resources.gyp:load_time_data',
+          '../settings_page/settings_animated_pages.js'
         ],
         'externs': [
-          '../../../../../third_party/closure_compiler/externs/language_settings_private.js',
+          '../../../../../third_party/closure_compiler/externs/settings_private.js'
+        ],
+      },
+      'includes': ['../../../../../third_party/closure_compiler/compile_js.gypi'],
+    },
+    {
+      'target_name': 'startup_urls_page',
+      'variables': {
+        'depends': [

[dschuyler, 2015/09/14 21:08:28]

I don't know closure well enough to review this imo.  I am getting the code
build it, as a kind of sanity check.

[dschuyler, 2015/09/14 21:46:52]

I've patched with this CL and tried out the on_startup page and ran the closure
compiler.  It looks good from that point of view.  I also read some about the
closure compiler to see what depends, externs, and includes were about.  It all
looks ok as far as I see.

[stevenjb, 2015/09/14 23:12:35]

Acknowledged.

[stevenjb, 2015/09/14 23:12:35]

Acknowledged.

+          '../../../../../ui/webui/resources/js/compiled_resources.gyp:load_time_data',
+        ],
+        'externs': [
+          '../../../../../third_party/closure_compiler/externs/settings_private.js'
         ],
       },
       'includes': ['../../../../../third_party/closure_compiler/compile_js.gypi'],