Fix an incorrect comment in the Java Bridge when calling array methods

content/browser/renderer_host/java/java_bound_object.cc

Index: content/browser/renderer_host/java/java_bound_object.cc
diff --git a/content/browser/renderer_host/java/java_bound_object.cc b/content/browser/renderer_host/java/java_bound_object.cc
index ff113d43513ed3f92143973a43e6a5d3d27c6cd1..b87917ba48cefdfc38b36f4a14c2ba8247a08d15 100644
--- a/content/browser/renderer_host/java/java_bound_object.cc
+++ b/content/browser/renderer_host/java/java_bound_object.cc
@@ -153,7 +153,9 @@ NPVariant CallJNIMethod(jobject object, JavaType::Type return_type,
       VOID_TO_NPVARIANT(result);
       break;
     case JavaType::TypeArray:
-      // TODO(steveblock): Handle arrays
+      // LIVECONNECT_COMPLIANCE: Don't call methods that return arrays, to
+      // maintain existing behavior. We should call the method and convert the

[bulach, 2011/11/29 11:11:14]

is the second sentence a TODO? also, I got comments that we should avoid we :)
so this either would be TODO(steveblock) call the method and.. or something like
"The new behavior should call.."

[Steve Block, 2011/11/29 11:18:00]

It't not really a TODO. This file has many comments of this style. They're used
to document areas where we don't follow the LiveConnect spec. The first sentence
is what we do. The second is what we should do to match the spec. We have no
immediate plans to match the spec, but may revisit this in the future.

If using 'we' is discouraged, I could update all the comments to use something
like 'The spec requires calling the method ...'. Can you point me to the
guidance on 'we'?

[bulach, 2011/11/29 11:46:19]

got it, thanks for the clarification!
here's the comment I had:
http://codereview.chromium.org/8586054/diff/2001/base/base.gypi#newcode606

yes, I think the "The spec requires" would make it clearer the distinction
between LIVECONNECT_COMPLIANCE and the spec..

Maybe something like:

LIVECONNECT_COMPLIANCE: Existing behavior, %s.
The spec requires %s.

+      // result to a JavaScript array.
       VOID_TO_NPVARIANT(result);
       break;
     case JavaType::TypeString: {