Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2434)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/TabBase.java

Issue 11778005: JavascriptAppModalDialog tests and necessary infrastructure. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Upstreamed pliard@'s click-after-dismiss regression test. Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/TabBase.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/TabBase.java b/chrome/android/java/src/org/chromium/chrome/browser/TabBase.java
index f65d228b19537479dfcfb5d6f2ba0132a6aad275..bf83f23cea6a5490a5b5155e0bdcc6fde855a0d9 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/TabBase.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/TabBase.java
@@ -70,6 +70,9 @@ public class TabBase {
* be used.
*/
public void destroy() {
+ for (int i = 0; i < mObservers.size(); ++i) {
+ mObservers.get(i).onCloseTab(TabBase.this);
+ }
destroyContentView();
if (mNativeTabBaseAndroidImpl != 0) {
mCleanupReference.cleanupNow();

Powered by Google App Engine
This is Rietveld 408576698