Prevent refreshing app search results with empty query.

- Any change to the TextView text will cause mQuery to be set,
  which will cause a new search next time refreshSearchResult
  is called.  We should also be checking there if it is a valid
  search query before starting a new search.

Bug: 30606307
Change-Id: I08640c56199211f2aeea2386fcf699810853ab58
This commit is contained in:
Winson
2016-08-03 17:08:10 -07:00
parent 5a5eb846c4
commit 45a11dd9ee

View File

@@ -20,6 +20,7 @@ import android.content.Intent;
import android.graphics.Rect;
import android.net.Uri;
import android.text.Editable;
import android.text.TextUtils;
import android.text.TextWatcher;
import android.view.KeyEvent;
import android.view.View;
@@ -102,7 +103,7 @@ public abstract class AllAppsSearchBarController
}
protected void refreshSearchResult() {
if (mQuery == null) {
if (TextUtils.isEmpty(mQuery)) {
return;
}
// If play store continues auto updating an app, we want to show partial result.
@@ -143,7 +144,6 @@ public abstract class AllAppsSearchBarController
unfocusSearchField();
mCb.clearSearchResult();
mInput.setText("");
// We need to reset this after we clear the input text
mQuery = null;
hideKeyboard();
}