Have Frame::loader() return a reference

Source/core/frame/Frame.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Simon Hausmann <hausmann@kde.org>
  *                     2000 Stefan Schimanski <1Stein@gmx.de>
  *                     2001 George Staikos <staikos@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2005 Alexey Proskuryakov <ap@nypop.com>
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
@@ skipping 126 matching lines @@
     RefPtr<Frame> frame = adoptRef(new Frame(page, ownerElement, client));
     if (!ownerElement)
         page->setMainFrame(frame);
     InspectorInstrumentation::frameAttachedToParent(frame.get());
     return frame.release();
 }
 
 Frame::~Frame()
 {
     setView(0);
-    loader()->clear(ClearScriptObjects | ClearWindowObject);
+    loader().clear(ClearScriptObjects | ClearWindowObject);
 
     // FIXME: We should not be doing all this work inside the destructor
 
 #ifndef NDEBUG
     frameCounter.decrement();
 #endif
 
     disconnectOwnerElement();
 
     HashSet<FrameDestructionObserver*>::iterator stop = m_destructionObservers.end();
@@ skipping 158 matching lines @@
     for (size_t i = 0; i < childFrames.size(); ++i)
         childFrames[i]->dispatchVisibilityStateChangeEvent();
 }
 
 void Frame::willDetachPage()
 {
     // We should never be detatching the page during a Layout.
     RELEASE_ASSERT(!m_view || !m_view->isInLayout());
 
     if (Frame* parent = tree().parent())
-        parent->loader()->checkLoadComplete();
+        parent->loader().checkLoadComplete();
 
     HashSet<FrameDestructionObserver*>::iterator stop = m_destructionObservers.end();
     for (HashSet<FrameDestructionObserver*>::iterator it = m_destructionObservers.begin(); it != stop; ++it)
         (*it)->willDetachPage();
 
     // FIXME: It's unclear as to why this is called more than once, but it is,
     // so page() could be NULL.
     if (page() && page()->focusController().focusedFrame() == this)
         page()->focusController().setFocusedFrame(0);
 
@@ skipping 340 matching lines @@
     if (!m_page)
         return 0;
 
     double ratio = m_page->deviceScaleFactor();
     if (RuntimeEnabledFeatures::devicePixelRatioIncludesZoomEnabled())
         ratio *= pageZoomFactor();
     return ratio;
 }
 
 } // namespace WebCore