Do all OOLing in the views code. linux_views now builds clean with the clang plugin.

chrome/browser/ui/views/tabs/base_tab_strip.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_TABS_BASE_TAB_STRIP_H_
 #define CHROME_BROWSER_UI_VIEWS_TABS_BASE_TAB_STRIP_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 155 matching lines @@
   // move.
   virtual void StartMoveTabAnimation();
 
   // Starts the remove tab animation.
   virtual void StartRemoveTabAnimation(int model_index);
 
   // Starts the mini-tab animation.
   virtual void StartMiniTabAnimation();
 
   // Returns whether the highlight button should be highlighted after a remove.
-  virtual bool ShouldHighlightCloseButtonAfterRemove() { return true; }
+  virtual bool ShouldHighlightCloseButtonAfterRemove();
 
   // Animates all the views to their ideal bounds.
   // NOTE: this does *not* invoke GenerateIdealBounds, it uses the bounds
   // currently set in ideal_bounds.
   virtual void AnimateToIdealBounds() = 0;
 
   // Cleans up the Tab from the TabStrip. This is called from the tab animation
   // code and is not a general-purpose method.
   void RemoveAndDeleteTab(BaseTab* tab);
 
@@ skipping 56 matching lines @@
   // model.
   bool attaching_dragged_tab_;
 
   views::BoundsAnimator bounds_animator_;
 
   // Size we last layed out at.
   gfx::Size last_layout_size_;
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TABS_BASE_TAB_STRIP_H_