Finish removing FrameLoader::Clear()

third_party/WebKit/Source/core/editing/Editor.cpp

Index: third_party/WebKit/Source/core/editing/Editor.cpp
diff --git a/third_party/WebKit/Source/core/editing/Editor.cpp b/third_party/WebKit/Source/core/editing/Editor.cpp
index 2fc9b42fc338aee591b90e97681a6c6115edeaa3..dbcf86c7bdb4551bac1ca9552a362b224fb8c939 100644
--- a/third_party/WebKit/Source/core/editing/Editor.cpp
+++ b/third_party/WebKit/Source/core/editing/Editor.cpp
@@ -1040,7 +1040,6 @@ Editor::Editor(LocalFrame& frame)
 Editor::~Editor() {}
 
 void Editor::Clear() {
-  GetFrame().GetInputMethodController().Clear();

[Nate Chapin, 2017/05/17 18:18:59]

This is also called during Document::Shutdown(), so it's redundant here.

[dcheng, 2017/05/22 21:23:26]

Due to InputMethodController::ContextDestroyed()?

I'd say that it's a bit unclear how this whole class works together, but I think
this doesn't make it any worse.

   should_style_with_css_ = false;
   default_paragraph_separator_ = kEditorParagraphSeparatorIsDiv;
   last_edit_command_ = nullptr;