Merge pull request #9100 from /issues/16361-1.26.x

Fixes follow up of #13038 - swiping up ad notification doesn't trigger the dismiss event type - 1.26.x
brave · Jun 16, 2021 · 44e98bc · 44e98bc
2 parents 596c4d6 + 4ac045e
commit 44e98bc
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...rave_ads/android/java/org/chromium/chrome/browser/dialogs/BraveAdsNotificationDialog.java b/...rave_ads/android/java/org/chromium/chrome/browser/dialogs/BraveAdsNotificationDialog.java
@@ -116,7 +116,7 @@ public boolean onTouch(View v, MotionEvent event) {
                                     mAdsDialog = null;
                                     BraveAdsNativeHelper.nativeOnCloseAdNotification(
                                             Profile.getLastUsedRegularProfile(), mNotificationId,
-                                            false);
+                                            true);
                                     mNotificationId = null;
                                 } else if (deltaYDp <= MAX_DISTANCE_FOR_TAP
                                         && deltaYDp >= (-1 * MAX_DISTANCE_FOR_TAP)) {
@@ -170,9 +170,9 @@ private static void closeAdsNotification(final String notificationId) {
         try {
             if (mNotificationId != null && mNotificationId.equals(notificationId) && mAdsDialog != null) {
                 mAdsDialog.dismiss();
+                mAdsDialog = null;
                 BraveAdsNativeHelper.nativeOnCloseAdNotification(
                         Profile.getLastUsedRegularProfile(), mNotificationId, false);
-                mAdsDialog = null;
             }
         } catch (IllegalArgumentException e) {
             mAdsDialog = null;