GTK: Cleanup of TabStripGtk::ResizeLayoutTabs() now that the layout is fixed.

chrome/browser/ui/gtk/tabs/tab_strip_gtk.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/gtk/tabs/tab_strip_gtk.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/i18n/rtl.h"
@@ skipping 1377 matching lines @@
   }
 }
 
 void TabStripGtk::HandleGlobalMouseMoveEvent() {
   if (!IsCursorInTabStripZone()) {
     // Mouse moved outside the tab slop zone, start a timer to do a resize
     // layout after a short while...
     if (!weak_factory_.HasWeakPtrs()) {
       MessageLoop::current()->PostDelayedTask(
           FROM_HERE,
-          base::Bind(&TabStripGtk::ResizeLayoutTabsWithoutResult,
+          base::Bind(&TabStripGtk::ResizeLayoutTabs,
                      weak_factory_.GetWeakPtr()),
           kResizeTabsTimeMs);
     }
   } else {
     // Mouse moved quickly out of the tab strip and then into it again, so
     // cancel the timer so that the strip doesn't move when the mouse moves
     // back over it.
     weak_factory_.InvalidateWeakPtrs();
   }
 }
@@ skipping 144 matching lines @@
       !GetTabAt(tab_index)->mini()) {
     return mini_to_non_mini_gap_ + kTabHOffset;
   }
   return kTabHOffset;
 }
 
 int TabStripGtk::tab_start_x() const {
   return 0;
 }
 
-bool TabStripGtk::ResizeLayoutTabs() {
+void TabStripGtk::ResizeLayoutTabs() {
   weak_factory_.InvalidateWeakPtrs();
   layout_factory_.InvalidateWeakPtrs();
 
   // It is critically important that this is unhooked here, otherwise we will
   // keep spying on messages forever.
   RemoveMessageLoopObserver();
 
   available_width_for_tabs_ = -1;
   int mini_tab_count = GetMiniTabCount();
   if (mini_tab_count == GetTabCount()) {
     // Only mini tabs, we know the tab widths won't have changed (all mini-tabs
     // have the same width), so there is nothing to do.
-    return false;
+    return;
   }
   TabGtk* first_tab = GetTabAt(mini_tab_count);
   double unselected, selected;
   GetDesiredTabWidths(GetTabCount(), mini_tab_count, &unselected, &selected);
   int w = Round(first_tab->IsActive() ? selected : unselected);
 
   // We only want to run the animation if we're not already at the desired
   // size.
-  if (abs(first_tab->width() - w) > 1) {
+  if (abs(first_tab->width() - w) > 1)
     StartResizeLayoutAnimation();
-    return true;
-  }
-
-  return false;
-}
-
-void TabStripGtk::ResizeLayoutTabsWithoutResult() {
-  ResizeLayoutTabs();
 }
 
 bool TabStripGtk::IsCursorInTabStripZone() const {
   gfx::Point tabstrip_topleft;
   gtk_util::ConvertWidgetPointToScreen(tabstrip_.get(), &tabstrip_topleft);
 
   gfx::Rect bds = bounds();
   bds.set_origin(tabstrip_topleft);
   bds.set_height(bds.height() + kTabStripAnimationVSlop);
 
@@ skipping 625 matching lines @@
 }
 
 void TabStripGtk::SetNewTabButtonBackground() {
   SkColor color = theme_service_->GetColor(
       ThemeService::COLOR_BUTTON_BACKGROUND);
   SkBitmap* background = theme_service_->GetBitmapNamed(
       IDR_THEME_WINDOW_CONTROL_BACKGROUND);
   SkBitmap* mask = theme_service_->GetBitmapNamed(IDR_NEWTAB_BUTTON_MASK);
   newtab_button_->SetBackground(color, background, mask);
 }