Log an error when no activity is found to handle a URL.

Rather than crashing due to ActivityNotFoundException.

Bug: 13930087
Change-Id: Id3af8301bb87298b885a24f70bf184abe4fc2a11

1 files changed, 13 insertions, 8 deletions

diff --git a/src/com/android/browser/UrlHandler.java b/src/com/android/browser/UrlHandler.java
index e21e3e87..edafebd1 100644
--- a/src/com/android/browser/UrlHandler.java
+++ b/src/com/android/browser/UrlHandler.java
@@ -144,14 +144,19 @@ public class UrlHandler {
               intent = new Intent(Intent.ACTION_VIEW, Uri
                       .parse("market://search?q=pname:" + packagename));
               intent.addCategory(Intent.CATEGORY_BROWSABLE);
-              mActivity.startActivity(intent);
-              // before leaving BrowserActivity, close the empty child tab.
-              // If a new tab is created through JavaScript open to load this
-              // url, we would like to close it as we will load this url in a
-              // different Activity.
-              mController.closeEmptyTab();
-              return true;
-          } else {
+              try {
+                  mActivity.startActivity(intent);
+                  // before leaving BrowserActivity, close the empty child tab.
+                  // If a new tab is created through JavaScript open to load this
+                  // url, we would like to close it as we will load this url in a
+                  // different Activity.
+                  mController.closeEmptyTab();
+                  return true;
+              } catch (ActivityNotFoundException e) {
+                  Log.w("Browser", "No activity found to handle " + url);
+                  return false;
+              }
+            } else {
               return false;
           }
       }