fix new tab regression

Change-Id: I788343c7df3c282d8f42aeac715ad54eacea77b9

4 files changed, 14 insertions, 6 deletions

diff --git a/src/com/android/browser/ActiveTabsPage.java b/src/com/android/browser/ActiveTabsPage.java
index 23b313a94..52d943fc4 100644
--- a/src/com/android/browser/ActiveTabsPage.java
+++ b/src/com/android/browser/ActiveTabsPage.java
@@ -75,7 +75,7 @@ public class ActiveTabsPage extends LinearLayout implements OnClickListener,
         if (v == mNewTab) {
             mController.openTabToHomePage();
         } else if (v == mNewIncognitoTab) {
-            mController.openTab(null, true, true, false);
+            mController.openIncognitoTab();
         }
         mController.removeActiveTabsPage(false);
     }
diff --git a/src/com/android/browser/Controller.java b/src/com/android/browser/Controller.java
index 0f967ceac..4b72ce36b 100644
--- a/src/com/android/browser/Controller.java
+++ b/src/com/android/browser/Controller.java
@@ -56,7 +56,6 @@ import android.provider.BrowserContract.Images;
 import android.provider.ContactsContract;
 import android.provider.ContactsContract.Intents.Insert;
 import android.speech.RecognizerIntent;
-import android.speech.RecognizerResultsIntent;
 import android.text.TextUtils;
 import android.util.Log;
 import android.util.Patterns;
@@ -1546,7 +1545,7 @@ public class Controller
                 break;
 
             case R.id.incognito_menu_id:
-                openTab(null, true, true, false);
+                openIncognitoTab();
                 break;
 
             case R.id.goto_menu_id:
@@ -2214,12 +2213,19 @@ public class Controller
     }
 
     @Override
+    public Tab openIncognitoTab() {
+        return openTab(INCOGNITO_URI, true, true, false);
+    }
+
+    @Override
     public Tab openTab(String url, boolean incognito, boolean setActive,
             boolean useCurrent) {
         Tab tab = createNewTab(incognito, setActive, useCurrent);
         if (tab != null) {
             WebView w = tab.getWebView();
-            loadUrl(w, ((incognito && url == null) ? INCOGNITO_URI : url));
+            if (url != null) {
+                loadUrl(w, url);
+            }
         }
         return tab;
     }
@@ -2551,7 +2557,7 @@ public class Controller
                 // exclusive use of a modifier
                 if (event.isCtrlPressed()) {
                     if (event.isShiftPressed()) {
-                        openTab(null, true, true, false);
+                        openIncognitoTab();
                     } else {
                         openTabToHomePage();
                     }
diff --git a/src/com/android/browser/NavScreen.java b/src/com/android/browser/NavScreen.java
index 4095a964c..9f2d4aeb9 100644
--- a/src/com/android/browser/NavScreen.java
+++ b/src/com/android/browser/NavScreen.java
@@ -239,7 +239,7 @@ public class NavScreen extends LinearLayout implements OnClickListener {
             showMenu();
         } else if (mNewIncognito == v) {
             mUi.hideNavScreen(true);
-            mUiController.openTab(null, true, true, false);
+            mUiController.openIncognitoTab();
         }
     }
 
diff --git a/src/com/android/browser/UiController.java b/src/com/android/browser/UiController.java
index 9f855476d..a2de1d794 100644
--- a/src/com/android/browser/UiController.java
+++ b/src/com/android/browser/UiController.java
@@ -42,6 +42,8 @@ public interface UiController extends BookmarksHistoryCallbacks {
 
     Tab openTabToHomePage();
 
+    Tab openIncognitoTab();
+
     Tab openTab(String url, boolean incognito, boolean setActive,
             boolean useCurrent);