Add on_startup_page/compiled_resources.gyp

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

 # 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.
 {
   '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/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/language_settings_private.js',
+          '../../../../../third_party/closure_compiler/externs/settings_private.js'
         ],
       },
       'includes': ['../../../../../third_party/closure_compiler/compile_js.gypi'],
     },
   ],
 }