From ec0a12cf199659a749ff1fe7b788626eddc095eb Mon Sep 17 00:00:00 2001
From: Karl Rosaen <krosaen@google.com>
Date: Tue, 21 Jul 2009 16:21:32 -0700
Subject: [PATCH] Don't start search from a dialog if there is no activity
 associated with the search manager at that point.

Still works:
- hitting search when a managed dialog of an app is showing will dismiss
  the dialog and start in-app search (if it supports it), falling back
  on global search
- hitting search when a dialog of an app (not managed by the activity) will
  dismiss the dialog and start global search
- hitting search when a system dialog is showing will just dismiss the dialog.
---
 core/java/android/app/Dialog.java        | 5 +++++
 core/java/android/app/SearchManager.java | 4 ++++
 2 files changed, 9 insertions(+)

diff --git a/core/java/android/app/Dialog.java b/core/java/android/app/Dialog.java
index 444f222e5bde7..943275514caff 100644
--- a/core/java/android/app/Dialog.java
+++ b/core/java/android/app/Dialog.java
@@ -788,6 +788,11 @@ public class Dialog implements DialogInterface, Window.Callback,
         final SearchManager searchManager = (SearchManager) mContext
                 .getSystemService(Context.SEARCH_SERVICE);
 
+        // can't start search without an associated activity (e.g a system dialog)
+        if (!searchManager.hasIdent()) {
+            return false;
+        }
+
         // associate search with owner activity if possible (otherwise it will default to
         // global search).
         final ComponentName appName = mOwnerActivity == null ? null
diff --git a/core/java/android/app/SearchManager.java b/core/java/android/app/SearchManager.java
index c98d966ce7bdc..0631ad58929ba 100644
--- a/core/java/android/app/SearchManager.java
+++ b/core/java/android/app/SearchManager.java
@@ -1536,6 +1536,10 @@ public class SearchManager
         mService = ISearchManager.Stub.asInterface(
                 ServiceManager.getService(Context.SEARCH_SERVICE));
     }
+
+    /*package*/ boolean hasIdent() {
+        return mIdent != 0;
+    }
     
     /*package*/ void setIdent(int ident) {
         if (mIdent != 0) {