commit c6d218612513ca3bcc78aa1201f6e4ec576742bd
author yueg <yueg@google.com> Fri Mar 30 12:31:47 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Fri Mar 30 12:35:40 2018 -0700
tree 1ff4ab39241c91c6ae4a8abad8208ed9f2ce1ee8
parent df36d8f1c6ef91646fdbf169ec2c6cc9edcc8e3c

Fix bubble showing after dismiss bug.

Now we also show bubble in onCallListChange(), but we shouldn't show bubble after dismiss in this case.

Test: ReturnToCallControllerTest
PiperOrigin-RevId: 191101307
Change-Id: I13ff6bf4ae7ab574966c43c0e6fee13853899ba0

java/com/android/bubble/Bubble.java

diff --git a/java/com/android/bubble/Bubble.java b/java/com/android/bubble/Bubble.java
index e192e06..1b853cf 100644
--- a/java/com/android/bubble/Bubble.java
+++ b/java/com/android/bubble/Bubble.java
@@ -39,6 +39,9 @@
   /** Returns whether the bubble is currently visible */
   boolean isVisible();
 
+  /** Returns whether the bubble is currently dismissed */
+  boolean isDismissed();
+
   /**
    * Set the info for this Bubble to display
    *

java/com/android/bubble/stub/BubbleStub.java

diff --git a/java/com/android/bubble/stub/BubbleStub.java b/java/com/android/bubble/stub/BubbleStub.java
index 267f33f..2aa55a3 100644
--- a/java/com/android/bubble/stub/BubbleStub.java
+++ b/java/com/android/bubble/stub/BubbleStub.java
@@ -40,6 +40,11 @@
   }
 
   @Override
+  public boolean isDismissed() {
+    return false;
+  }
+
+  @Override
   public void setBubbleInfo(@NonNull BubbleInfo bubbleInfo) {}
 
   @Override