Clean up the proxy code for handling frame navigation....

webkit/port/bindings/v8/ScriptController.cpp

 // Copyright (c) 2008, 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 305 matching lines @@
 void ScriptController::setEventHandlerLineno(int lineno)
 {
     m_proxy->setEventHandlerLineno(lineno);
 }
 
 void ScriptController::finishedWithEvent(Event* evt)
 {
     m_proxy->finishedWithEvent(evt);
 }
 
-void ScriptController::clearDocumentWrapper()
-{
-    m_proxy->clearDocumentWrapper();
-}
-
 // Create a V8 object with an interceptor of NPObjectPropertyGetter
 void ScriptController::BindToWindowObject(Frame* frame, const String& key, NPObject* object)
 {
     v8::HandleScope handle_scope;
 
     v8::Handle<v8::Context> context = V8Proxy::GetContext(frame);
     if (context.IsEmpty())
         return;
 
     v8::Context::Scope scope(context);
@@ skipping 164 matching lines @@
     v8::Handle<v8::Value> v8plugin = V8Proxy::ToV8Object(V8ClassIndex::HTMLEMBEDELEMENT, plugin);
     if (!v8plugin->IsObject())
         return createNoScriptObject();
 
     return NPN_CreateScriptObject(0, v8::Handle<v8::Object>::Cast(v8plugin), window);
 }
 
 
 void ScriptController::clearWindowShell()
 {
+    // V8 binding expects ScriptController::clearWindowShell only be called
+    // when a frame is loading a new page. V8Proxy::clearForNavigation
+    // creates a new context for the new page.
     m_proxy->clearForNavigation();
 }
 
 void ScriptController::attachDebugger(void*)
 {
     notImplemented();
 }
 
 void ScriptController::updateDocument()
 {
-    // TODO(eseidel): Should update document property on current window object
-    // and all previous window objects which may still be alive.
-    notImplemented();
+    m_proxy->updateDocument();
 }
 
 
 JSInstanceHolder::JSInstanceHolder()
 {
 }
 
 JSInstanceHolder::JSInstanceHolder(JSInstanceHandle instance)
 {
     *this = instance;
@@ skipping 41 matching lines @@
     v8::Persistent<v8::Object> handle =
         v8::Persistent<v8::Object>::New(instance);
     m_instance = handle;
 #ifndef NDEBUG
     V8Proxy::RegisterGlobalHandle(JSINSTANCE, this, handle);
 #endif
     return *this;
 }
 
 }  // namespace WebCpre