Add support for closing Android Shell instances from Java.

content/shell/browser/shell_android.cc

Index: content/shell/browser/shell_android.cc
diff --git a/content/shell/browser/shell_android.cc b/content/shell/browser/shell_android.cc
index b11ba37abaa1d3c7eba35fbc76ddf17159853ba2..9c8647760f9ef453657f9169e5c5c082bb36337e 100644
--- a/content/shell/browser/shell_android.cc
+++ b/content/shell/browser/shell_android.cc
@@ -30,6 +30,10 @@ void Shell::PlatformExit() {
 }
 
 void Shell::PlatformCleanUp() {
+  JNIEnv* env = AttachCurrentThread();
+  if (java_object_.is_null())
+    return;
+  Java_Shell_onNativeDestroyed(env, java_object_.obj());
 }
 
 void Shell::PlatformEnableUIControl(UIControl control, bool is_enabled) {
@@ -83,8 +87,7 @@ bool Shell::PlatformIsFullscreenForTabOrPending(
 }
 
 void Shell::Close() {
-  CloseShellView(java_object_.obj());
-  java_object_.Reset();
+  RemoveShellView(java_object_.obj());
   delete this;
 }
 
@@ -93,4 +96,10 @@ bool Shell::Register(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
+// static
+void CloseShell(JNIEnv* env, jclass clazz, jlong nativeShell) {

[cjhopman, 2014/01/09 02:12:44]

This function should be generated by the jni generator. Looks like content_shell
jni doesn't yet use longs for pointers so that's why this isn't autogenerating.

[Ted C, 2014/01/09 02:46:32]

Oh Chris, how you got my hopes and dashed them so well.

Sadly, the generator will convert the Java side of:
static native void nativeClose(long nativeShell)

to:
static void Close(JNIEnv* env, jobject jcaller,
    jlong nativeShell) {
  Shell* native = reinterpret_cast<Shell*>(nativeShell);
  CHECK_NATIVE_PTR(env, jcaller, native, "Close");
  return native->Close(env, jcaller);
}

It's so close, but would need it to drop the env and jcaller params and allow
it to directly call non-java aware native functions (which would be amazing
BTW).  Since I don't want to pollute the cross platform shell.h with a bunch
of JNI bridging cruft, I would like to keep it isolated to this file and keep
it much like it was.

+  Shell* shell = reinterpret_cast<Shell*>(nativeShell);
+  shell->Close();
+}
+
 }  // namespace content