Complete migration to InspectorFrontendHost.sendMessageToFrontendHost.

Source/core/inspector/InspectorFrontendHost.cpp

Index: Source/core/inspector/InspectorFrontendHost.cpp
diff --git a/Source/core/inspector/InspectorFrontendHost.cpp b/Source/core/inspector/InspectorFrontendHost.cpp
index 05509c14d4509801eb516d51646bd64102c407bd..8bb770ae3aeca653ab2975f64d0782163fe05fe7 100644
--- a/Source/core/inspector/InspectorFrontendHost.cpp
+++ b/Source/core/inspector/InspectorFrontendHost.cpp
@@ -136,32 +136,6 @@ void InspectorFrontendHost::loaded()
 {
 }
 
-void InspectorFrontendHost::requestSetDockSide(const String& side)
-{
-    if (!m_client)
-        return;
-    if (side == "undocked")
-        m_client->requestSetDockSide(InspectorFrontendClient::Undocked);

[pfeldman, 2013/08/23 11:34:15]

Remove these constants as well.

[Vladislav Kaznacheev, 2013/08/30 14:37:00]

Done.

-    else if (side == "right")
-        m_client->requestSetDockSide(InspectorFrontendClient::DockedToRight);
-    else if (side == "bottom")
-        m_client->requestSetDockSide(InspectorFrontendClient::DockedToBottom);
-}
-
-void InspectorFrontendHost::closeWindow()
-{
-    if (m_client) {
-        m_client->closeWindow();
-        disconnectClient(); // Disconnect from client.

[pfeldman, 2013/08/23 11:34:15]

This was also clearing m_client. Leave it as is and emit "closeWindow" from
here.

[Vladislav Kaznacheev, 2013/08/30 14:37:00]

Done in the previous patch, already committed
(https://chromiumcodereview.appspot.com/23289002/)

On 2013/08/23 11:34:15, pfeldman wrote:

-    }
-}
-
-void InspectorFrontendHost::bringToFront()
-{
-    if (m_client)
-        m_client->bringToFront();
-}
-
 void InspectorFrontendHost::setZoomFactor(float zoom)
 {
     m_frontendPage->mainFrame()->setPageAndTextZoomFactors(zoom, 1);
@@ -173,18 +147,6 @@ void InspectorFrontendHost::inspectedURLChanged(const String& newURL)
         m_client->inspectedURLChanged(newURL);
 }
 
-void InspectorFrontendHost::setAttachedWindowHeight(unsigned height)
-{
-    if (m_client)
-        m_client->changeAttachedWindowHeight(height);
-}
-
-void InspectorFrontendHost::moveWindowBy(float x, float y) const
-{
-    if (m_client)
-        m_client->moveWindowBy(x, y);
-}
-
 void InspectorFrontendHost::setInjectedScriptForOrigin(const String& origin, const String& script)
 {
     m_frontendPage->inspectorController().setInjectedScriptForOrigin(origin, script);
@@ -200,29 +162,11 @@ void InspectorFrontendHost::copyText(const String& text)
     Pasteboard::generalPasteboard()->writePlainText(text, Pasteboard::CannotSmartReplace);
 }
 
-void InspectorFrontendHost::openInNewTab(const String& url)
-{
-    if (m_client)
-        m_client->openInNewTab(url);
-}
-
 bool InspectorFrontendHost::canSave()
 {
     return true;
 }
 
-void InspectorFrontendHost::save(const String& url, const String& content, bool forceSaveAs)
-{
-    if (m_client)
-        m_client->save(url, content, forceSaveAs);
-}
-
-void InspectorFrontendHost::append(const String& url, const String& content)
-{
-    if (m_client)
-        m_client->append(url, content);
-}
-
 void InspectorFrontendHost::close(const String&)
 {
 }
@@ -292,48 +236,12 @@ bool InspectorFrontendHost::supportsFileSystems()
     return true;
 }
 
-void InspectorFrontendHost::requestFileSystems()
-{
-    if (m_client)
-        m_client->requestFileSystems();
-}
-
-void InspectorFrontendHost::addFileSystem()
-{
-    if (m_client)
-        m_client->addFileSystem();
-}
-
-void InspectorFrontendHost::removeFileSystem(const String& fileSystemPath)
-{
-    if (m_client)
-        m_client->removeFileSystem(fileSystemPath);
-}
-
 PassRefPtr<DOMFileSystem> InspectorFrontendHost::isolatedFileSystem(const String& fileSystemName, const String& rootURL)
 {
     ScriptExecutionContext* context = m_frontendPage->mainFrame()->document();
     return DOMFileSystem::create(context, fileSystemName, FileSystemTypeIsolated, KURL(ParsedURLString, rootURL), AsyncFileSystem::create());
 }
 
-void InspectorFrontendHost::indexPath(int requestId, const String& fileSystemPath)
-{
-    if (m_client)
-        m_client->indexPath(requestId, fileSystemPath);
-}
-
-void InspectorFrontendHost::stopIndexing(int requestId)
-{
-    if (m_client)
-        m_client->stopIndexing(requestId);
-}
-
-void InspectorFrontendHost::searchInPath(int requestId, const String& fileSystemPath, const String& query)
-{
-    if (m_client)
-        m_client->searchInPath(requestId, fileSystemPath, query);
-}
-
 bool InspectorFrontendHost::isUnderTest()
 {
     return m_client && m_client->isUnderTest();