[ios] Do not attempt to show recent tab if there is none [chromium/src : main]

0 views

Skip to first unread message

Chris Lu (Gerrit)

unread,

1:00 PM (5 hours ago) 1:00 PM

to Eric Ekey, Chromium LUCI CQ, chromium...@chromium.org, feature-me...@chromium.org, ios-revie...@chromium.org, ios-r...@chromium.org, marq+...@chromium.org

Chris Lu voted Commit-Queue+2

Commit-Queue

Open in Gerrit

Related details

Attention set is empty

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

open

diffy

Chromium LUCI CQ (Gerrit)

unread,

1:15 PM (5 hours ago) 1:15 PM

to Chris Lu, Eric Ekey, chromium...@chromium.org, feature-me...@chromium.org, ios-revie...@chromium.org, ios-r...@chromium.org, marq+...@chromium.org

Chromium LUCI CQ submitted the change

Unreviewed changes

2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Change information

Commit message:

[ios] Do not attempt to show recent tab if there is none

This is a fast fix for the crash. If there is no saved recent tab
WebState, then don't attempt to set up a recent tab magic stack card.

Bug: 487067852

Change-Id: I5cfbbb8154f2d9860bbfe81b1108e124ea66f0ef

Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7603184

Commit-Queue: Chris Lu <thegre...@chromium.org>

Reviewed-by: Eric Ekey <eric...@google.com>

Cr-Commit-Position: refs/heads/main@{#1589525}

Files:

M ios/chrome/browser/content_suggestions/tab_resumption/coordinator/tab_resumption_mediator.mm

Change size: XS

Delta: 1 file changed, 1 insertion(+), 1 deletion(-)

Branch: refs/heads/main

Submit Requirements:

Code-Review: +1 by Eric Ekey

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

open

diffy

satisfied_requirement

Reply all

Reply to author

Forward

0 new messages