Finish removing FrameLoader::Clear()

third_party/WebKit/Source/core/frame/LocalFrame.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>
@@ skipping 273 matching lines @@
   for (auto& initilization_callback : GetInitializationVector()) {
     initilization_callback(this);
   }
 
   loader_.Init();
 }
 
 void LocalFrame::SetView(FrameView* view) {
   DCHECK(!view_ || view_ != view);
   DCHECK(!GetDocument() || !GetDocument()->IsActive());
-
-  GetEventHandler().Clear();
-
   view_ = view;
 }
 
 void LocalFrame::CreateView(const IntSize& viewport_size,
                             const Color& background_color,
                             ScrollbarMode horizontal_scrollbar_mode,
                             bool horizontal_lock,
                             ScrollbarMode vertical_scrollbar_mode,
                             bool vertical_lock) {
   DCHECK(this);
@@ skipping 126 matching lines @@
   // detachChildren() will trigger the unload event handlers of any child
   // frames, and those event handlers might start a new subresource load in this
   // frame.
   loader_.StopAllLoaders();
   loader_.Detach();
   GetDocument()->Shutdown();
   // This is the earliest that scripting can be disabled:
   // - FrameLoader::detach() can fire XHR abort events
   // - Document::shutdown()'s deferred widget updates can run script.
   ScriptForbiddenScope forbid_script;
-  loader_.Clear();
   if (!Client())
     return;
 
   Client()->WillBeDetached();
   // Notify ScriptController that the frame is closing, since its cleanup ends
   // up calling back to LocalFrameClient via WindowProxy.
   GetScriptController().ClearForClose();
   SetView(nullptr);
 
   page_->GetEventHandlerRegistry().DidRemoveAllEventHandlers(*DomWindow());
@@ skipping 63 matching lines @@
 void LocalFrame::DetachChildren() {
   DCHECK(loader_.StateMachine()->CreatingInitialEmptyDocument() ||
          GetDocument());
 
   if (Document* document = this->GetDocument())
     ChildFrameDisconnector(*document).Disconnect();
 }
 
 void LocalFrame::DocumentAttached() {
   DCHECK(GetDocument());
+  GetEditor().Clear();
+  GetEventHandler().Clear();
   Selection().DocumentAttached(GetDocument());
   GetInputMethodController().DocumentAttached(GetDocument());
   GetSpellChecker().DocumentAttached(GetDocument());
   if (IsMainFrame())
     has_received_user_gesture_ = false;
 }
 
 LocalWindowProxy* LocalFrame::WindowProxy(DOMWrapperWorld& world) {
   return ToLocalWindowProxy(Frame::GetWindowProxy(world));
 }
@@ skipping 431 matching lines @@
         WebURLRequest::kClientLoFiOn);
     params.SetAllowImagePlaceholder();
   }
 }
 
 std::unique_ptr<WebURLLoader> LocalFrame::CreateURLLoader() {
   return Client()->CreateURLLoader();
 }
 
 }  // namespace blink