summaryrefslogtreecommitdiff
path: root/java/com/android/dialer/main
diff options
context:
space:
mode:
authorcalderwoodra <calderwoodra@google.com>2018-04-04 16:26:10 -0700
committerCopybara-Service <copybara-piper@google.com>2018-04-04 18:10:19 -0700
commit93ab1bc3e4d7a37c859fb24dc35b18fa637df0f9 (patch)
treee7fa0a7230e221a823fbe868c5bdbb41b1d05e5f /java/com/android/dialer/main
parent05945e8a79ddc7ee14bb1347276193e796571713 (diff)
Added some logging for mysterious bugs.
Bug: 74196353,77485680,77486095,77540395 Test: tap PiperOrigin-RevId: 191666270 Change-Id: I0423735dde42299bf33ea0b8482c03fd3c2d4675
Diffstat (limited to 'java/com/android/dialer/main')
-rw-r--r--java/com/android/dialer/main/impl/MainSearchController.java19
1 files changed, 17 insertions, 2 deletions
diff --git a/java/com/android/dialer/main/impl/MainSearchController.java b/java/com/android/dialer/main/impl/MainSearchController.java
index 7b4bc3569..2279d56a7 100644
--- a/java/com/android/dialer/main/impl/MainSearchController.java
+++ b/java/com/android/dialer/main/impl/MainSearchController.java
@@ -184,7 +184,7 @@ public class MainSearchController implements SearchBarListener {
*/
private void hideDialpad(boolean animate) {
LogUtil.enterBlock("MainSearchController.hideDialpad");
- Assert.checkArgument(isDialpadVisible());
+ assertDialpadVisible();
fab.show();
toolbar.slideDown(animate, fragmentContainer);
@@ -291,7 +291,7 @@ public class MainSearchController implements SearchBarListener {
/** Calls {@link #hideDialpad(boolean)}, removes the search fragment and clears the dialpad. */
private void closeSearch(boolean animate) {
LogUtil.enterBlock("MainSearchController.closeSearch");
- Assert.checkArgument(isSearchVisible());
+ assertSearchIsVisible();
if (isDialpadVisible()) {
hideDialpad(animate);
} else if (!fab.isShown()) {
@@ -337,11 +337,26 @@ public class MainSearchController implements SearchBarListener {
&& fragment.isDialpadSlideUp();
}
+ private void assertDialpadVisible() {
+ DialpadFragment fragment = getDialpadFragment();
+ Assert.checkArgument(fragment != null, "Dialpad Fragment is null");
+ Assert.checkArgument(fragment.isAdded(), "Dialpad Fragment is no added");
+ Assert.checkArgument(!fragment.isHidden(), "Dialpad Fragment is hidden");
+ Assert.checkArgument(fragment.isDialpadSlideUp(), "Dialpad Fragment is slide down");
+ }
+
private boolean isSearchVisible() {
NewSearchFragment fragment = getSearchFragment();
return fragment != null && fragment.isAdded() && !fragment.isHidden();
}
+ private void assertSearchIsVisible() {
+ NewSearchFragment fragment = getSearchFragment();
+ Assert.checkArgument(fragment != null, "Search Fragment is null");
+ Assert.checkArgument(fragment.isAdded(), "Search Fragment is not added");
+ Assert.checkArgument(!fragment.isHidden(), "Search Fragment is hidden.");
+ }
+
/** Returns true if the search UI is visible. */
public boolean isInSearch() {
return isSearchVisible();