Have Frame::script() return a reference

Source/bindings/v8/ScriptEventListener.cpp

 /*
  * Copyright (C) 2009 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 45 matching lines @@
     if (value.isNull())
         return 0;
 
     // FIXME: Very strange: we initialize zero-based number with '1'.
     TextPosition position(OrdinalNumber::fromZeroBasedInt(1), OrdinalNumber::first());
     String sourceURL;
 
     v8::Isolate* isolate;
     if (Frame* frame = node->document().frame()) {
         isolate = toIsolate(frame);
-        ScriptController* scriptController = frame->script();
-        if (!scriptController->canExecuteScripts(AboutToExecuteScript))
+        ScriptController& scriptController = frame->script();
+        if (!scriptController.canExecuteScripts(AboutToExecuteScript))
             return 0;
-        position = scriptController->eventHandlerPosition();
+        position = scriptController.eventHandlerPosition();
         sourceURL = node->document().url().string();
     } else {
         isolate = v8::Isolate::GetCurrent();
     }
 
     return V8LazyEventListener::create(name.localName().string(), eventParameterName(node->isSVGElement()), value, sourceURL, position, node, isolate);
 }
 
 PassRefPtr<V8LazyEventListener> createAttributeEventListener(Frame* frame, const QualifiedName& name, const AtomicString& value)
 {
     if (!frame)
         return 0;
 
     if (value.isNull())
         return 0;
 
-    ScriptController* scriptController = frame->script();
-    if (!scriptController->canExecuteScripts(AboutToExecuteScript))
+    ScriptController& scriptController = frame->script();
+    if (!scriptController.canExecuteScripts(AboutToExecuteScript))
         return 0;
 
-    TextPosition position = scriptController->eventHandlerPosition();
+    TextPosition position = scriptController.eventHandlerPosition();
     String sourceURL = frame->document()->url().string();
 
     return V8LazyEventListener::create(name.localName().string(), eventParameterName(frame->document()->isSVGDocument()), value, sourceURL, position, 0, toIsolate(frame));
 }
 
 String eventListenerHandlerBody(Document* document, EventListener* listener)
 {
     if (listener->type() != EventListener::JSEventListenerType)
         return "";
 
@@ skipping 23 matching lines @@
         return ScriptValue();
     return ScriptValue(function, isolate);
 }
 
 ScriptState* eventListenerHandlerScriptState(Frame* frame, EventListener* listener)
 {
     if (listener->type() != EventListener::JSEventListenerType)
         return 0;
     V8AbstractEventListener* v8Listener = static_cast<V8AbstractEventListener*>(listener);
     v8::HandleScope scope(toIsolate(frame));
-    v8::Handle<v8::Context> v8Context = frame->script()->windowShell(v8Listener->world())->context();
+    v8::Handle<v8::Context> v8Context = frame->script().windowShell(v8Listener->world())->context();
     return ScriptState::forContext(v8Context);
 }
 
 bool eventListenerHandlerLocation(Document* document, EventListener* listener, String& sourceName, String& scriptId, int& lineNumber)
 {
     if (listener->type() != EventListener::JSEventListenerType)
         return false;
 
     v8::HandleScope scope(toIsolate(document));
     V8AbstractEventListener* v8Listener = static_cast<V8AbstractEventListener*>(listener);
     v8::Handle<v8::Context> context = toV8Context(document, v8Listener->world());
     v8::Context::Scope contextScope(context);
     v8::Handle<v8::Object> object = v8Listener->getListenerObject(document);
     if (object.IsEmpty() || !object->IsFunction())
         return false;
 
     v8::Handle<v8::Function> function = v8::Handle<v8::Function>::Cast(object);
     v8::Handle<v8::Value> scriptIdValue = function->GetScriptId();
     scriptId = toWebCoreStringWithUndefinedOrNullCheck(scriptIdValue);
     v8::ScriptOrigin origin = function->GetScriptOrigin();
     if (!origin.ResourceName().IsEmpty() && origin.ResourceName()->IsString())
         sourceName = toWebCoreString(origin.ResourceName().As<v8::String>());
     else
         sourceName = "";
     lineNumber = function->GetScriptLineNumber();
     return true;
 }
 
 } // namespace WebCore