Debounce popup block notification when blocking popups caused by back…

… button
MozillaReality · Nov 1, 2019 · d5a1f66 · d5a1f66
1 parent a0ec4bd
commit d5a1f66
Showing 1 changed file with 21 additions and 1 deletion.
diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/NavigationBarWidget.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/NavigationBarWidget.java
@@ -1190,11 +1190,31 @@ public void onPopUpAvailable() {
         @Override
         public void onPopUpsCleared() {
             mURLBar.setIsPopUpAvailable(false);
+            hidePopUpsBlockedNotification();
         }
     };
 
+    private int mBlockedCount;
+
     public void showPopUpsBlockedNotification() {
-        post(() -> showNotification(mURLBar.getPopUpButton(), R.string.popup_tooltip));
+        final int POP_UP_NOTIFICATION_DELAY = 800;
+        mBlockedCount++;
+        final int currentCount = mBlockedCount;
+        postDelayed(() -> {
+            if (currentCount == mBlockedCount) {
+                showNotification(mURLBar.getPopUpButton(), R.string.popup_tooltip);
+            }
+        }, POP_UP_NOTIFICATION_DELAY);
+    }
+
+    public void hidePopUpsBlockedNotification() {
+        mBlockedCount++;
+        final int currentCount = mBlockedCount;
+        post(() -> {
+            if (currentCount == mBlockedCount) {
+                hideNotification(mURLBar.getPopUpButton());
+            }
+        });
     }
 
     private void showNotification(UIButton button, int stringRes) {