[Android WebView] Provide more information about WebView for web debugging

android_webview/native/aw_contents.cc

Index: android_webview/native/aw_contents.cc
diff --git a/android_webview/native/aw_contents.cc b/android_webview/native/aw_contents.cc
index f740291f25fc9e02c4696e9d8e8d5d359bd86b99..a7b7c9681ae4de507b9a30d7e794194516e23de8 100644
--- a/android_webview/native/aw_contents.cc
+++ b/android_webview/native/aw_contents.cc
@@ -21,6 +21,7 @@
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
+#include "base/android/scoped_java_ref.h"
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/message_loop.h"
@@ -110,6 +111,13 @@ AwContents* AwContents::FromID(int render_process_id, int render_view_id) {
   return FromWebContents(web_contents);
 }
 
+// static
+BrowserViewRenderer* BrowserViewRenderer::FromWebContents(

[joth, 2013/03/08 23:23:46]

this is really weird here!

BrowserViewRenderer is already holding a lot of responsibility for the draw path
so maybe confusing it to pull it into the dev tools flow too... but I see
there's nothing more convenient in browser/ for you to use at this point. 

AwContentsClientBridgeBase almost works, but isn't ideal as you don't really
want to pop out via the AwContentsClient.

If we keep with BrowserViewRenderer, I think the 'correct' solution is to have
it register itself with the WebContents::SetUserData() and then it can implement
its own FromWebContents() method. (And that would also help tackle the lifetime
ambiguity that currently exists between the two)

[mnaganov (inactive), 2013/03/11 14:39:20]

I would be using AwContents directly (because I'm viewing it as an extension of
WebContents), but this is prohibited by our architecture.

BrowserViewRenderer was the second natural choice, because it was only missing
the screen location data.

I did this hack as it required the least amount of lines. Switched to the
userdata approach.

+    content::WebContents* web_contents) {
+  AwContents* aw_contents = AwContents::FromWebContents(web_contents);
+  return aw_contents ? aw_contents->browser_view_renderer() : NULL;
+}
+
 AwContents::AwContents(JNIEnv* env,
                        jobject obj,
                        jobject web_contents_delegate,
@@ -672,6 +680,26 @@ jint AwContents::ReleasePopupWebContents(JNIEnv* env, jobject obj) {
   return reinterpret_cast<jint>(pending_contents_.release());
 }
 
+bool AwContents::isAttachedToViewHierarchy() {
+  JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jobject> obj = java_ref_.get(env);
+  if (!obj.is_null())
+    return Java_AwContents_isAttachedToViewHierarchy(env, obj.obj());
+  return false;
+}
+
+gfx::Point AwContents::getLocationOnScreen() {
+  JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jobject> obj = java_ref_.get(env);
+  if (obj.is_null()) return gfx::Point();
+  std::vector<int> location;
+  base::android::JavaIntArrayToIntVector(
+      env,
+      Java_AwContents_getLocationOnScreen(env, obj.obj()).obj(),
+      &location);
+  return gfx::Point(location[0], location[1]);
+}
+
 ScopedJavaLocalRef<jobject> AwContents::CapturePicture(JNIEnv* env,
                                                        jobject obj) {
   return browser_view_renderer_->CapturePicture();