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

Unified Diff: chrome/browser/ui/views/tabs/stacked_tab_strip_layout.h

Issue 2963023002: [stacked tabs] Fix stacking logic with 'open link in new tab' (Closed)
Patch Set: Created 3 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
Index: chrome/browser/ui/views/tabs/stacked_tab_strip_layout.h
diff --git a/chrome/browser/ui/views/tabs/stacked_tab_strip_layout.h b/chrome/browser/ui/views/tabs/stacked_tab_strip_layout.h
index ac671afb88affc13436f3e6f707073f501928128..bbfd1e73e6428abffb6d91a2e6002ccafb0e60ca 100644
--- a/chrome/browser/ui/views/tabs/stacked_tab_strip_layout.h
+++ b/chrome/browser/ui/views/tabs/stacked_tab_strip_layout.h
@@ -103,8 +103,10 @@ class StackedTabStripLayout {
// Resets to an ideal layout state.
void ResetToIdealState();
- // Makes |index| visible. This is used when a new tab is added that isn't
- // active.
+ // Makes all tabs between the active tab and |index| (inclusive) visible. If
+ // this is not possible, then |index| is made visible and a stack forms
+ // immediately to the right of the active tab. This is used when a new tab is
+ // added that isn't active.
void MakeVisible(int index);
// Returns the x-coordinate for the active tab constrained by the current tab

Powered by Google App Engine
This is Rietveld 408576698