Refactor the Android port to allow access to the chrome layer.

android_webview/native/webview_web_contents_delegate_android.h

Index: android_webview/native/webview_web_contents_delegate_android.h
diff --git a/android_webview/native/webview_web_contents_delegate_android.h b/android_webview/native/webview_web_contents_delegate_android.h
new file mode 100644
index 0000000000000000000000000000000000000000..aa9849d76a6b8b1d85bde617ca85c681266b10a4
--- /dev/null
+++ b/android_webview/native/webview_web_contents_delegate_android.h
@@ -0,0 +1,27 @@
+// 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.
+
+#ifndef ANDROID_WEBVIEW_NATIVE_WEBVIEW_WEB_CONTENTS_DELEGATE_ANDROID_H_
+#define ANDROID_WEBVIEW_NATIVE_WEBVIEW_WEB_CONTENTS_DELEGATE_ANDROID_H_
+
+#include <jni.h>
+
+#include "chrome/browser/component/web_contents_delegate_android/web_contents_delegate_android.h"
+
+namespace android_webview {
+
+// WebView specific WebContentsDelegate.
+// Should contain WebContentsDelegate code required by WebView that should not
+// be part of the Chromium Android port.
+class WebViewWebContentsDelegateAndroid

[Jói, 2012/08/20 14:02:39]

I think joth@ had proposed Aw as a prefix instead of WebView.

[Leandro Graciá Gil, 2012/08/20 20:26:17]

Confirmed the name to use with Joth. Renaming.

+    : public chrome::component::web_contents_delegate_android::

[Jói, 2012/08/20 14:02:39]

This namespace nesting is pretty deep.  Personally, I'd be fine with just a
single namespace per component, if any, so e.g. just
web_contents_delegate_android.

[Leandro Graciá Gil, 2012/08/20 20:26:17]

Done.

+          WebContentsDelegateAndroid {
+ public:
+  WebViewWebContentsDelegateAndroid(JNIEnv* env, jobject obj);
+  virtual ~WebViewWebContentsDelegateAndroid();
+};
+
+}  // namespace android_webview
+
+#endif  // ANDROID_WEBVIEW_NATIVE_WEBVIEW_WEB_CONTENTS_DELEGATE_ANDROID_H_