Fix a dependency issue w/ gn refs for android apks.

build/config/android/internal_rules.gni

Index: build/config/android/internal_rules.gni
diff --git a/build/config/android/internal_rules.gni b/build/config/android/internal_rules.gni
index 2a7a8921ed37fbd994a208877a58ec7ab52a0bca..547a8d0ebe5aad625b1aeb4056bda46b128091ec 100644
--- a/build/config/android/internal_rules.gni
+++ b/build/config/android/internal_rules.gni
@@ -616,6 +616,12 @@ template("create_apk") {
   group(target_name) {
     deps = [
       ":${target_name}__finalize",
+
+      # TODO: crbug.com/487897 - __finalize needs as input a file created

[brettw, 2015/05/14 16:42:51]

I don't understand this comment. As we talked about, "gn refs" isn't broken and
we're not working around it, and there's nothing to fix in the future. If you
have a dependency, you should declare it in the deps.

[Dirk Pranke, 2015/05/14 17:27:06]

Yeah, reading it in the morning, I don't think this comment captured the right
thing.

I was trying to capture two different thoughts, I think. 

The first was an explanation of why this change was needed, which probably
could've been a comment on the CL rather than in the code, or it could be a
comment in the code, but just not a TODO.

The second thing was to capture the idea that when 'gn check' will be able to
check that inputs are not declared as dependencies, we should make sure that
this particular one is caught correctly. That was the "TODO" part.

Does that make sense?

+      # by __package_resources. We explicitly declare this as a dependency
+      # so that 'gn refs' knows about it (and hence 'mb analyze' will work
+      # correctly).
+      ":${target_name}__package_resources",
     ]
   }
 }