Remove more #includes from Frame.h

Source/WebCore/page/Frame.h

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999-2001 Lars Knoll <knoll@kde.org>
  *                     1999-2001 Antti Koivisto <koivisto@kde.org>
  *                     2000-2001 Simon Hausmann <hausmann@kde.org>
  *                     2000-2001 Dirk Mueller <mueller@kde.org>
  *                     2000 Stefan Schimanski <1Stein@gmx.de>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
@@ skipping 11 matching lines @@
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #ifndef Frame_h
 #define Frame_h
 
 #include "AdjustViewSizeOrNot.h"
-#include "AnimationController.h"
 #include "DragImage.h"
-#include "Editor.h"
-#include "EventHandler.h"
 #include "FrameLoader.h"
-#include "FrameSelection.h"
 #include "FrameTree.h"
+#include "IntRect.h"
 #include "NavigationScheduler.h"
+#include "ScrollTypes.h"
+#include <wtf/RefCounted.h>
 
 namespace WebCore {
 
+    class AnimationController;
+    class Color;
     class Document;
+    class Editor;
+    class Element;
+    class EventHandler;
     class FrameDestructionObserver;
+    class FrameSelection;
     class FrameView;
     class HTMLTableCellElement;
+    class IntRect;
+    class Node;
     class RegularExpression;
     class RenderPart;
+    class RenderView;
     class TreeScope;
     class ScriptController;
+    class Settings;
+    class TreeScope;
+    class VisiblePosition;
 
     enum {
         LayerTreeFlagsIncludeDebugInfo = 1 << 0,
         LayerTreeFlagsIncludeVisibleRects = 1 << 1,
         LayerTreeFlagsIncludeTileCaches = 1 << 2,
         LayerTreeFlagsIncludeRepaintRects = 1 << 3,
         LayerTreeFlagsIncludePaintingPhases = 1 << 4
     };
     typedef unsigned LayerTreeFlags;
 
@@ skipping 110 matching lines @@
         Page* m_page;
         mutable FrameTree m_treeNode;
         mutable FrameLoader m_loader;
         mutable NavigationScheduler m_navigationScheduler;
 
         HTMLFrameOwnerElement* m_ownerElement;
         RefPtr<FrameView> m_view;
         RefPtr<Document> m_doc;
 
         OwnPtr<ScriptController> m_script;
-
-        mutable Editor m_editor;
-        mutable FrameSelection m_selection;
-        mutable EventHandler m_eventHandler;
-        mutable AnimationController m_animationController;
+        OwnPtr<Editor> m_editor;
+        OwnPtr<FrameSelection> m_selection;
+        OwnPtr<EventHandler> m_eventHandler;
+        OwnPtr<AnimationController> m_animationController;
 
         float m_pageZoomFactor;
         float m_textZoomFactor;
 
 #if ENABLE(ORIENTATION_EVENTS)
         int m_orientation;
 #endif
 
         bool m_inViewSourceMode;
     };
@@ skipping 23 matching lines @@
         return m_script.get();
     }
 
     inline Document* Frame::document() const
     {
         return m_doc.get();
     }
 
     inline FrameSelection* Frame::selection() const
     {
-        return &m_selection;
+        return m_selection.get();
     }
 
     inline Editor* Frame::editor() const
     {
-        return &m_editor;
+        return m_editor.get();
     }
 
     inline AnimationController* Frame::animation() const
     {
-        return &m_animationController;
+        return m_animationController.get();

[eseidel, 2013/04/17 06:18:47]

This is the only one I could imaging being at all hot.  But I suspect any
animation which is hitting WebCore has much larger bottlenecks than this. :)

     }
 
     inline HTMLFrameOwnerElement* Frame::ownerElement() const
     {
         return m_ownerElement;
     }
 
     inline bool Frame::inViewSourceMode() const
     {
         return m_inViewSourceMode;
@@ skipping 14 matching lines @@
         return m_page;
     }
 
     inline void Frame::detachFromPage()
     {
         m_page = 0;
     }
 
     inline EventHandler* Frame::eventHandler() const
     {
-        return &m_eventHandler;
+        return m_eventHandler.get();
     }
 
 } // namespace WebCore
 
 #endif // Frame_h