Load web contents after tab is created.

chrome/browser/android/chrome_web_contents_delegate_android.cc

 // 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.
 
 #include "chrome/browser/android/chrome_web_contents_delegate_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/command_line.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ skipping 302 matching lines @@
     bool* was_blocked) {
   // No code for this yet.
   DCHECK_NE(disposition, SAVE_TO_DISK);
   // Can't create a new contents for the current tab - invalid case.
   DCHECK_NE(disposition, CURRENT_TAB);
 
   TabHelpers::AttachTabHelpers(new_contents);
 
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env);
-  bool handled = false;
+  AddWebContentsResult handled = ADD_WEB_CONTENTS_RESULT_STOP_LOAD_AND_DELETE;
   if (!obj.is_null()) {
     ScopedJavaLocalRef<jobject> jsource;
     if (source)
       jsource = source->GetJavaWebContents();
     ScopedJavaLocalRef<jobject> jnew_contents;
     if (new_contents)
       jnew_contents = new_contents->GetJavaWebContents();
 
-    handled = Java_ChromeWebContentsDelegateAndroid_addNewContents(
-        env,
-        obj.obj(),
-        jsource.obj(),
-        jnew_contents.obj(),
-        static_cast<jint>(disposition),
-        NULL,
-        user_gesture);
+    handled = static_cast<AddWebContentsResult>(
+        Java_ChromeWebContentsDelegateAndroid_addNewContents(
+            env,
+            obj.obj(),
+            jsource.obj(),
+            jnew_contents.obj(),
+            static_cast<jint>(disposition),
+            NULL,
+            user_gesture));
   }
 
-  if (was_blocked)
-    *was_blocked = !handled;
-  if (!handled)
-    delete new_contents;
+  switch (handled) {
+    case ADD_WEB_CONTENTS_RESULT_STOP_LOAD_AND_DELETE:
+      delete new_contents;
+      // fall through
+    case ADD_WEB_CONTENTS_RESULT_STOP_LOAD:
+      if (was_blocked)
+        *was_blocked = true;
+      break;
+    case ADD_WEB_CONTENTS_RESULT_PROCEED:
+      if (was_blocked)
+        *was_blocked = false;
+      break;
+  }

[gone, 2015/01/29 03:11:32]

Add a default: here.  Also, man this switch hurts my head to read.

 }
 
 }  // namespace android
 }  // namespace chrome