From 11665019e5cc3c3522132f3bc23876799338b784 Mon Sep 17 00:00:00 2001 From: Joemer Ramos Date: Thu, 29 Aug 2024 21:30:02 +0000 Subject: [PATCH] [iOS] IsQueryStale Follow Up in SafeBrowsingTabHelper Addressing leftover comment from crrev.com/c/5824002 after CL was landed. Bug: 5824002 Change-Id: Ia01fe5fc2804991364714e19885dd531bc25a0e3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5826134 Auto-Submit: Joemer Ramos Commit-Queue: Ali Juma Commit-Queue: Joemer Ramos Reviewed-by: Ali Juma Cr-Commit-Position: refs/heads/main@{#1348888} --- .../safe_browsing/safe_browsing_tab_helper.mm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ios/components/security_interstitials/safe_browsing/safe_browsing_tab_helper.mm b/ios/components/security_interstitials/safe_browsing/safe_browsing_tab_helper.mm index e1b0583bd859e4..188114ea95c7ea 100644 --- a/ios/components/security_interstitials/safe_browsing/safe_browsing_tab_helper.mm +++ b/ios/components/security_interstitials/safe_browsing/safe_browsing_tab_helper.mm @@ -163,7 +163,8 @@ void RecordTotalDelayMetricForDelayedAllowedNavigation( return !GetOldestPendingMainFrameQuery(query_data); } - return !GetOldestPendingToBeCommittedQuery(query_data) && + return !GetOldestPendingMainFrameQuery(query_data) && + !GetOldestPendingToBeCommittedQuery(query_data) && !GetOldestPendingCommittedQuery(query_data); }