Comment #4 on issue 6585 by [email protected]: REGRESSION Open in new tab
causes all tabs to go to minimum width
http://code.google.com/p/chromium/issues/detail?id=6585
I think r8109 might be the cause:
Index: chrome/browser/views/tabs/tab_strip.cc
===================================================================
--- chrome/browser/views/tabs/tab_strip.cc (revision 6886)
+++ chrome/browser/views/tabs/tab_strip.cc (working copy)
@@ -1470,8 +1470,17 @@
}
void TabStrip::StartInsertTabAnimation(int index) {
- // The TabStrip can now use its entire width to lay out Tabs.
- available_width_for_tabs_ = -1;
+ // Don't shock users by letting all tabs move when they are focused
+ // on the tab-strip. Wait for later, when they aren't looking.
+ int last_tab_index = GetTabCount() - 2;
+ if (last_tab_index > 0) {
+ Tab* last_tab = GetTabAt(last_tab_index);
+ available_width_for_tabs_ = std::min(
+ GetAvailableWidthForTabs(last_tab) + last_tab->width(),
+ width() - (kNewTabButtonHOffset +
newtab_button_size_.width()));
+ } else {
+ available_width_for_tabs_ = -1;
+ }
if (active_animation_.get())
active_animation_->Stop();
active_animation_.reset(new InsertTabAnimation(this, index));
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
