Rename JavaObjectWeakGlobalRef::is_empty to avoid misuse.

android_webview/native/aw_contents_io_thread_client_impl.cc

Index: android_webview/native/aw_contents_io_thread_client_impl.cc
diff --git a/android_webview/native/aw_contents_io_thread_client_impl.cc b/android_webview/native/aw_contents_io_thread_client_impl.cc
index 1d9a655beb4028ba36f4d66d629ae6a7a069540b..63965db2074f4dfdf8844bde3d4ea9f9f660d1a1 100644
--- a/android_webview/native/aw_contents_io_thread_client_impl.cc
+++ b/android_webview/native/aw_contents_io_thread_client_impl.cc
@@ -273,13 +273,12 @@ void AwContentsIoThreadClientImpl::SetServiceWorkerIoThreadClient(
 // static
 std::unique_ptr<AwContentsIoThreadClient>
 AwContentsIoThreadClient::GetServiceWorkerIoThreadClient() {
-  if (g_sw_instance_.Get().is_empty())
-    return std::unique_ptr<AwContentsIoThreadClient>();
-
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> java_delegate = g_sw_instance_.Get().get(env);
 
-  DCHECK(!java_delegate.is_null());
+  if (java_delegate.is_null())

[timvolodine, 2016/11/07 17:13:01]

to be more precise, I was looking here and comparing
g_sw_instance_.Get().is_empty() with g_sw_instance_.Get().get(env).is_null() it
seemed they were equivalent modulo the comment about JNIEnv::NewLocalRef(obj)

[boliu, 2016/11/07 17:23:33]

They are not equivalent.

is_empty (now is_initialized) will return true even if the *referent* object has
been garbage collected; it's only checking whether the *reference* is
initialized with something non-null

so it's possible for is_empty/is_initialized to be true and get(env).is_null to
be false

[boliu, 2016/11/07 17:25:29]

darn it, got that backwards

it's possible for get(env).is_null to be true, and is_empty to be false

[timvolodine, 2016/11/07 20:16:15]

Just to follow-up: we chatted here a bit about this, ServiceWorkerClient
currently has singleton semantics so not much gc happening unless the user
clears it explicitly. In general it seems fine (and correct) to return null in
this case, we have code to handle null cases further down the road.

+    return std::unique_ptr<AwContentsIoThreadClient>();
+
   return std::unique_ptr<AwContentsIoThreadClient>(
       new AwContentsIoThreadClientImpl(false, java_delegate));
 }