commit 991412749b1e0837ab602d77a2264a2186741a05
author John Reck <jreck@google.com> Tue Dec 21 11:34:12 2010 -0800
committer John Reck <jreck@google.com> Tue Dec 21 11:34:12 2010 -0800
tree 6246e0f61c76da0e1f97ff05e460304217961658
parent db49cab0d3e12ec1e99306ef78c7b8d8a2785724

Fix empty url check

 Bug: 3213668
 Moved empty check after url fix and trim

Change-Id: I79422cae12eefa33db7a970c780db6c0cea16a28

src/com/android/browser/IntentHandler.java

diff --git a/src/com/android/browser/IntentHandler.java b/src/com/android/browser/IntentHandler.java
index e5abb11..ff838f4 100644
--- a/src/com/android/browser/IntentHandler.java
+++ b/src/com/android/browser/IntentHandler.java
@@ -283,11 +283,12 @@
     private static boolean handleWebSearchRequest(Activity activity,
             Controller controller, String inUrl, Bundle appData,
             String extraData) {
-        if (TextUtils.isEmpty(inUrl)) return false;
+        if (inUrl == null) return false;
 
         // In general, we shouldn't modify URL from Intent.
         // But currently, we get the user-typed URL from search box as well.
         String url = UrlUtils.fixUrl(inUrl).trim();
+        if (TextUtils.isEmpty(url)) return false;
 
         // URLs are handled by the regular flow of control, so
         // return early.