Adds helper methods for WebView.

content/browser/android/content_view_statics.cc

+// Copyright (c) 2012 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.
+
+#include <jni.h>
+
+#include "base/android/jni_android.h"
+#include "base/android/jni_string.h"
+#include "base/android/scoped_java_ref.h"
+#include "base/basictypes.h"
+#include "base/logging.h"
+#include "content/browser/android/content_view_statics.h"
+#include "content/common/android/address_parser.h"
+#include "jni/content_view_statics_jni.h"
+
+using base::android::ConvertJavaStringToUTF16;
+using base::android::ConvertUTF16ToJavaString;
+using base::android::ScopedJavaLocalRef;
+using namespace content::address_parser;

[John Grabowski, 2012/06/28 17:08:06]

chrome style doesn't like "using namespace".  List classes/methods explicitly.

[aurimas (slooooooooow), 2012/06/28 23:00:07]

Done.

+
+static jstring FindAddress(JNIEnv* env, jclass clazz, jstring addr) {

[John Grabowski, 2012/06/28 17:08:06]

chrome style prefers anon namespaces to static functions.
Also, add single-line comment for this function.

[aurimas (slooooooooow), 2012/06/28 23:00:07]

I tried putting this function in a anon namespace, however jni had issues
finding this method then.

+  string16 content_16 = ConvertJavaStringToUTF16(env, addr);
+  string16 result_16;
+  if (FindAddress(content_16, &result_16))
+    return ConvertUTF16ToJavaString(env, result_16).Release();
+  return NULL;
+}
+
+bool RegisterWebViewStatics(JNIEnv* env) {

[John Grabowski, 2012/06/28 17:08:06]

all in content napespace

[aurimas (slooooooooow), 2012/06/28 23:00:07]

Done.

+  return RegisterNativesImpl(env) >= 0;
+}