Have Frame::loader() return a reference

Source/bindings/v8/custom/V8WindowCustom.cpp

 /*
  * Copyright (C) 2009, 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 199 matching lines @@
         return;
     }
 
     // Opener can be shadowed if it is in the same domain.
     // Have a special handling of null value to behave
     // like Firefox. See bug http://b/1224887 & http://b/791706.
     if (value->IsNull()) {
         // imp->frame() cannot be null,
         // otherwise, SameOrigin check would have failed.
         ASSERT(imp->frame());
-        imp->frame()->loader()->setOpener(0);
+        imp->frame()->loader().setOpener(0);
     }
 
     // Delete the accessor from this object.
     info.Holder()->Delete(name);
 
     // Put property on the front (this) object.
     info.This()->Set(name, value);
 }
 
 static bool isLegacyTargetOriginDesignation(v8::Handle<v8::Value> value)
@@ skipping 215 matching lines @@
 
     DOMWindow* targetWindow = V8Window::toNative(window);
 
     ASSERT(targetWindow);
 
     Frame* target = targetWindow->frame();
     if (!target)
         return false;
 
     // Notify the loader's client if the initial document has been accessed.
-    if (target->loader()->stateMachine()->isDisplayingInitialEmptyDocument())
-        target->loader()->didAccessInitialDocument();
+    if (target->loader().stateMachine()->isDisplayingInitialEmptyDocument())
+        target->loader().didAccessInitialDocument();
 
     if (key->IsString()) {
         DEFINE_STATIC_LOCAL(AtomicString, nameOfProtoProperty, ("__proto__", AtomicString::ConstructFromLiteral));
 
         String name = toWebCoreString(key.As<v8::String>());
         Frame* childFrame = target->tree().scopedChild(name);
         // Notice that we can't call HasRealNamedProperty for ACCESS_HAS
         // because that would generate infinite recursion.
         if (type == v8::ACCESS_HAS && childFrame)
             return true;
@@ skipping 21 matching lines @@
 
     DOMWindow* targetWindow = V8Window::toNative(window);
 
     ASSERT(targetWindow);
 
     Frame* target = targetWindow->frame();
     if (!target)
         return false;
 
     // Notify the loader's client if the initial document has been accessed.
-    if (target->loader()->stateMachine()->isDisplayingInitialEmptyDocument())
-        target->loader()->didAccessInitialDocument();
+    if (target->loader().stateMachine()->isDisplayingInitialEmptyDocument())
+        target->loader().didAccessInitialDocument();
 
     Frame* childFrame =  target->tree().scopedChild(index);
 
     // Notice that we can't call HasRealNamedProperty for ACCESS_HAS
     // because that would generate infinite recursion.
     if (type == v8::ACCESS_HAS && childFrame)
         return true;
     if (type == v8::ACCESS_GET
         && childFrame
         && !host->HasRealIndexedProperty(index)
@@ skipping 32 matching lines @@
     v8::Handle<v8::Context> context = frame->script().currentWorldContext();
     if (context.IsEmpty())
         return v8Undefined();
 
     v8::Handle<v8::Object> global = context->Global();
     ASSERT(!global.IsEmpty());
     return global;
 }
 
 } // namespace WebCore