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

Unified Diff: chrome/android/shell/java/src/org/chromium/chrome/shell/TabManager.java

Issue 738693002: Opening new "about:blank" tab if all tabs are closed in tab switcher mode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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/shell/java/src/org/chromium/chrome/shell/TabManager.java
diff --git a/chrome/android/shell/java/src/org/chromium/chrome/shell/TabManager.java b/chrome/android/shell/java/src/org/chromium/chrome/shell/TabManager.java
index fa25a1a4b96c3167d620a7973bdc631b6033c35a..611ff883363182f90af2e5ace2fba905cd020832 100644
--- a/chrome/android/shell/java/src/org/chromium/chrome/shell/TabManager.java
+++ b/chrome/android/shell/java/src/org/chromium/chrome/shell/TabManager.java
@@ -55,7 +55,10 @@ public class TabManager extends LinearLayout {
@Override
public void willCloseTab(Tab tab, boolean animate) {
- if (tab == mCurrentTab) setCurrentTab(null);
+ if (tab == mCurrentTab) {
+ hideTabSwitcher();
+ setCurrentTab(mCurrentTab);
Bernhard Bauer 2014/11/18 09:05:48 Wait, why are we doing this?
divya.bansal 2014/11/18 09:20:19 To get last tab so that its profile and url info w
Bernhard Bauer 2014/11/18 09:27:58 But this is called right before we close the tab.
divya.bansal 2014/11/19 09:51:30 Done.
+ }
}
};

Powered by Google App Engine
This is Rietveld 408576698