Move NullWebViewClient implementation to chromium layer

android_webview/java/src/org/chromium/android_webview/AwContentViewClient.java

 // Copyright 2013 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.
 
 package org.chromium.android_webview;
 
+import android.app.Activity;
+import android.content.ActivityNotFoundException;
 import android.content.Context;
+import android.content.ContextWrapper;
+import android.content.Intent;
+import android.provider.Browser;
+import android.util.Log;
 import android.view.KeyEvent;
 import android.view.View;
 import android.webkit.URLUtil;
 import android.webkit.WebChromeClient;
 import android.widget.FrameLayout;
 
 import org.chromium.content.browser.ContentVideoViewClient;
 import org.chromium.content.browser.ContentViewClient;
 
+import java.net.URISyntaxException;
+
 /**
  * ContentViewClient implementation for WebView
  */
 public class AwContentViewClient extends ContentViewClient implements ContentVideoViewClient {
+    private static final String TAG = "AwContentViewClient";
+
     private final AwContentsClient mAwContentsClient;
     private final AwSettings mAwSettings;
     private final AwContents mAwContents;
     private final Context mContext;
     private FrameLayout mCustomView;
 
     public AwContentViewClient(AwContentsClient awContentsClient, AwSettings awSettings,
             AwContents awContents, Context context) {
         mAwContentsClient = awContentsClient;
         mAwSettings = awSettings;
         mAwContents = awContents;
         mContext = context;
     }
 
     @Override
     public void onBackgroundColorChanged(int color) {
         mAwContentsClient.onBackgroundColorChanged(color);
     }
 
+    private static boolean isActivityContext(Context context) {

[hush (inactive), 2015/03/06 23:43:55]

ContextWrapper is already exposed to Chromium.
you could make use of this static class in content layer:
https://code.google.com/p/chromium/codesearch#chromium/src/content/public/and...

[boliu, 2015/03/07 00:41:42]

Oh nice. I'm not the first person to run into this :p

+        if (context instanceof Activity) {
+            return true;
+        } else if (context instanceof ContextWrapper) {
+            return isActivityContext(((ContextWrapper) context).getBaseContext());
+        } else {
+            return false;
+        }
+    }
+
     @Override
     public void onStartContentIntent(Context context, String contentUrl) {
-        //  Callback when detecting a click on a content link.
-        mAwContentsClient.shouldOverrideUrlLoading(contentUrl);
+        if (mAwContentsClient.hasWebViewClient()) {
+            //  Callback when detecting a click on a content link.
+            mAwContentsClient.shouldOverrideUrlLoading(contentUrl);
+            return;
+        }
+
+        Intent intent;
+        // Perform generic parsing of the URI to turn it into an Intent.
+        try {
+            intent = Intent.parseUri(contentUrl, Intent.URI_INTENT_SCHEME);
+        } catch (URISyntaxException ex) {
+            Log.w(TAG, "Bad URI " + contentUrl + ": " + ex.getMessage());
+            return;
+        }
+        // Sanitize the Intent, ensuring web pages can not bypass browser
+        // security (only access to BROWSABLE activities).
+        intent.addCategory(Intent.CATEGORY_BROWSABLE);
+        intent.setComponent(null);
+        Intent selector = intent.getSelector();
+        if (selector != null) {
+            selector.addCategory(Intent.CATEGORY_BROWSABLE);
+            selector.setComponent(null);
+        }
+        // Pass the package name as application ID so that the intent from the
+        // same application can be opened in the same tab.
+        intent.putExtra(Browser.EXTRA_APPLICATION_ID, context.getPackageName());
+        if (!isActivityContext(context)) {
+            intent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
+        }
+
+        try {
+            context.startActivity(intent);
+        } catch (ActivityNotFoundException ex) {
+            Log.w(TAG, "No application can handle " + contentUrl);
+        }
     }
 
     @Override
     public void onUpdateTitle(String title) {
         mAwContentsClient.onReceivedTitle(title);
     }
 
     @Override
     public boolean shouldOverrideKeyEvent(KeyEvent event) {
-        return mAwContentsClient.shouldOverrideKeyEvent(event);
+        if (mAwContentsClient.hasWebViewClient()) {
+            // The check below is reflecting Clank's behavior and is a workaround for
+            // http://b/7697782.
+            if (!ContentViewClient.shouldPropagateKey(event.getKeyCode())) return true;
+            return mAwContentsClient.shouldOverrideKeyEvent(event);
+        }
+
+        return super.shouldOverrideKeyEvent(event);
     }
 
     @Override
     public final ContentVideoViewClient getContentVideoViewClient() {
         return this;
     }
 
     @Override
     public boolean shouldBlockMediaRequest(String url) {
         return mAwSettings != null
@@ skipping 68 matching lines @@
     @Override
     public boolean isJavascriptEnabled() {
         return mAwSettings != null && mAwSettings.getJavaScriptEnabled();
     }
 
     @Override
     public boolean isExternalFlingActive() {
         return mAwContents.isFlingActive();
     }
 }