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

Unified Diff: chrome/browser/views/tabs/dragged_tab_controller.cc

Issue 2821011: Removes phantom tabs. (Closed)
Patch Set: Created 10 years, 6 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
« no previous file with comments | « chrome/browser/views/tabs/browser_tab_strip_controller.cc ('k') | chrome/browser/views/tabs/side_tab.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/views/tabs/dragged_tab_controller.cc
diff --git a/chrome/browser/views/tabs/dragged_tab_controller.cc b/chrome/browser/views/tabs/dragged_tab_controller.cc
index bb97cd662a69176ede368fbd9d7363d53e9d99d8..8f4f19bf9ed304a182e3f18b18eff055de628a1f 100644
--- a/chrome/browser/views/tabs/dragged_tab_controller.cc
+++ b/chrome/browser/views/tabs/dragged_tab_controller.cc
@@ -883,7 +883,7 @@ void DraggedTabController::Detach() {
attached_model->DetachTabContentsAt(index);
// If we've removed the last Tab from the TabStrip, hide the frame now.
- if (!attached_model->HasNonPhantomTabs())
+ if (attached_model->empty())
HideFrame();
// Set up the photo booth to start capturing the contents of the dragged
@@ -1281,7 +1281,7 @@ void DraggedTabController::HideFrame() {
void DraggedTabController::CleanUpHiddenFrame() {
// If the model we started dragging from is now empty, we must ask the
// delegate to close the frame.
- if (!GetModel(source_tabstrip_)->HasNonPhantomTabs())
+ if (GetModel(source_tabstrip_)->empty())
GetModel(source_tabstrip_)->delegate()->CloseFrameAfterDragSession();
}
« no previous file with comments | « chrome/browser/views/tabs/browser_tab_strip_controller.cc ('k') | chrome/browser/views/tabs/side_tab.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698