commit 9b1458e14e5c72012fc8e0f5d60b0a8cbb36de9e
author Andrea Gomez <andreaxg@google.com> Thu May 30 16:39:46 2024
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu May 30 16:39:46 2024
tree 1d1cf657d2479af12dbadc6d6ff11922ca7dfe65
parent 25c1c98ecd190923bb1e014d5bff537a278effca

[Crash] Check tab is not destroyed in onTabSelected


Bug: 343516776
Change-Id: I5e3840e57c1dd328a1f8f4216d056bbefe0c1ac3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5581960
Code-Coverage: findit-for-me@appspot.gserviceaccount.com <findit-for-me@appspot.gserviceaccount.com>
Reviewed-by: Ian Wells <iwells@chromium.org>
Commit-Queue: Andrea Gomez <andreaxg@google.com>
Cr-Commit-Position: refs/heads/main@{#1308063}

chrome/browser/readaloud/android/java/src/org/chromium/chrome/browser/readaloud/ReadAloudController.java

diff --git a/chrome/browser/readaloud/android/java/src/org/chromium/chrome/browser/readaloud/ReadAloudController.java b/chrome/browser/readaloud/android/java/src/org/chromium/chrome/browser/readaloud/ReadAloudController.java
index 2bb4851..5ce42ef0 100644
--- a/chrome/browser/readaloud/android/java/src/org/chromium/chrome/browser/readaloud/ReadAloudController.java
+++ b/chrome/browser/readaloud/android/java/src/org/chromium/chrome/browser/readaloud/ReadAloudController.java
@@ -533,7 +533,9 @@
                             // url and then the destination url. Because of that we should not use
                             // this method to trigger readability.
                             super.onTabSelected(tab);
-                            if (tab != null && !GURL.isEmptyOrInvalid(tab.getUrl())) {
+                            if (tab != null
+                                    && !tab.isDestroyed()
+                                    && !GURL.isEmptyOrInvalid(tab.getUrl())) {
                                 if (mPausedForIncognito) {
                                     mPausedForIncognito = false;
                                     if (mPlayback != null) {