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

chrome/browser/ui/views/tabs/base_tab.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_H_
 #define CHROME_BROWSER_UI_VIEWS_TABS_BASE_TAB_H_
 #pragma once
 
 #include "base/ref_counted.h"
 #include "base/scoped_ptr.h"
@@ skipping 118 matching lines @@
 
   // views::ContextMenuController overrides:
   virtual void ShowContextMenu(views::View* source,
                                const gfx::Point& p,
                                bool is_mouse_gesture);
 
   // Returns the bounds of the title and icon.
   virtual const gfx::Rect& GetTitleBounds() const = 0;
   virtual const gfx::Rect& GetIconBounds() const = 0;
 
-  virtual int loading_animation_frame() const {
-    return loading_animation_frame_;
-  }
-  virtual bool should_display_crashed_favicon() const {
-    return should_display_crashed_favicon_;
-  }
-  virtual int fav_icon_hiding_offset() const { return fav_icon_hiding_offset_; }
+  virtual int loading_animation_frame() const;
+  virtual bool should_display_crashed_favicon() const;
+  virtual int fav_icon_hiding_offset() const;
 
   static gfx::Font* font() { return font_; }
   static int font_height() { return font_height_; }
 
  private:
   // The animation object used to swap the favicon with the sad tab icon.
   class FavIconCrashAnimation;
 
   // Set the temporary offset for the favicon. This is used during the crash
   // animation.
@@ skipping 55 matching lines @@
 
   ui::ThemeProvider* theme_provider_;
 
   static gfx::Font* font_;
   static int font_height_;
 
   DISALLOW_COPY_AND_ASSIGN(BaseTab);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TABS_BASE_TAB_H_