Fix sky to pass 'gn check'

mojo/public/tools/bindings/mojom.gni

 # 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.
 
 # Generate C++ and JavaScript source files from mojom files. The output files
 # will go under the generated file directory tree with the same path as each
 # input file.
 #
 # Parameters:
 #
@@ skipping 117 matching lines @@
 
   source_set(target_name) {
     if (defined(invoker.visibility)) {
       visibility = invoker.visibility
     }
     if (defined(invoker.testonly)) {
       testonly = invoker.testonly
     }
     sources = process_file_template(invoker.sources, generator_cpp_outputs)
     data = process_file_template(invoker.sources, generator_js_outputs)
     deps = [

[jamesr, 2014/10/24 21:42:47]

can be one line now

deps = [ "//mojo/public/cpp/bindings" ]

-      ":$generator_target_name",
       "//mojo/public/cpp/bindings",
     ]
     if (defined(invoker.deps)) {
       deps += invoker.deps
     }
+    public_deps = [

[jamesr, 2014/10/24 21:42:47]

one line

+      ":$generator_target_name",
+    ]
     if (defined(invoker.public_deps)) {
-      public_deps = invoker.public_deps
+      public_deps += invoker.public_deps
     }
   }

[abarth-chromium, 2014/10/24 21:38:10]

We should have someone else look at this part of the change.

Otherwise, LGTM

 
   all_deps = []
   if (defined(invoker.deps)) {
     all_deps += invoker.deps
   }
   if (defined(invoker.public_deps)) {
     all_deps += invoker.public_deps
   }
 
   group("${target_name}__is_mojom") {
@@ skipping 25 matching lines @@
         # //mojo/something:something and we can append "_java" to get the java
         # dependency name.
         full_name = get_label_info(d, "label_no_toolchain")
         deps += [ "${full_name}_java" ]
       }
 
       srcjars = process_file_template(invoker.sources, generator_java_outputs)
     }
   }
 }