Move FrameHostMsg_GetCookies to mojom

content/browser/frame_host/render_frame_message_filter_browsertest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/test/histogram_tester.h"
 #include "content/browser/bad_message.h"
@@ skipping 21 matching lines @@
 
 namespace {
 
 std::string GetCookieFromJS(RenderFrameHost* frame) {
   std::string cookie;
   EXPECT_TRUE(ExecuteScriptAndExtractString(
       frame, "window.domAutomationController.send(document.cookie);", &cookie));
   return cookie;
 }
 
+mojom::RenderFrameMessageFilter* GetFilterForProcess(
+    RenderProcessHost* process) {
+  return static_cast<RenderProcessHostImpl*>(process)
+      ->render_frame_message_filter_for_testing();
+}
+
 }  // namespace
 
 class RenderFrameMessageFilterBrowserTest : public ContentBrowserTest {
  protected:
   void SetUp() override {
     base::CommandLine::ForCurrentProcess()->AppendSwitch(
         switches::kEnableExperimentalWebPlatformFeatures);
     ContentBrowserTest::SetUp();
   }
 };
@@ skipping 131 matching lines @@
       "Where A = http://127.0.0.1/\n"
       "      B = http://baz.com/",
       v.DepictFrameTree(tab->GetFrameTree()->root()));
 
   RenderFrameHost* main_frame = tab->GetMainFrame();
   RenderFrameHost* iframe =
       tab->GetFrameTree()->root()->child_at(0)->current_frame_host();
 
   EXPECT_NE(iframe->GetProcess(), main_frame->GetProcess());
 
-  // Try to get cross-site cookies from the subframe's process and wait for it
-  // to be killed.
-  std::string response;
-  FrameHostMsg_GetCookies illegal_get_cookies(
-      iframe->GetRoutingID(), GURL("http://127.0.0.1/"),
-      GURL("http://127.0.0.1/"), &response);
-
   RenderProcessHostWatcher iframe_killed(
       iframe->GetProcess(), RenderProcessHostWatcher::WATCH_FOR_PROCESS_EXIT);
 
-  IPC::IpcSecurityTestUtil::PwnMessageReceived(
-      iframe->GetProcess()->GetChannel(), illegal_get_cookies);
+  // Try to get cross-site cookies from the subframe's process and wait for it
+  // to be killed.
+  BrowserThread::GetTaskRunnerForThread(BrowserThread::IO)->PostTask(
+      FROM_HERE,
+      base::Bind([] (RenderFrameHost* frame) {
+        GetFilterForProcess(frame->GetProcess())->GetCookies(
+            frame->GetRoutingID(), GURL("http://127.0.0.1/"),
+            GURL("http://127.0.0.1/"), base::Bind([] (mojo::String) {}));
+      }, iframe));
 
   iframe_killed.Wait();
 
   EXPECT_EQ(
       " Site A ------------ proxies for B\n"
       "   +--Site B ------- proxies for A\n"
       "Where A = http://127.0.0.1/\n"
       "      B = http://baz.com/ (no process)",
       v.DepictFrameTree(tab->GetFrameTree()->root()));
 
   // Now set a cross-site cookie from the main frame's process and wait for it
   // to be killed.
   RenderProcessHostWatcher main_frame_killed(
       tab->GetMainFrame()->GetProcess(),
       RenderProcessHostWatcher::WATCH_FOR_PROCESS_EXIT);
 
-  RenderProcessHostImpl* process =
-      static_cast<RenderProcessHostImpl*>(tab->GetMainFrame()->GetProcess());
   BrowserThread::GetTaskRunnerForThread(BrowserThread::IO)->PostTask(
       FROM_HERE,
-      base::Bind(
-          &mojom::RenderFrameMessageFilter::SetCookie,
-          base::Unretained(process->render_frame_message_filter_for_testing()),
-          tab->GetMainFrame()->GetRoutingID(), GURL("https://baz.com/"),
-          GURL("https://baz.com/"), "pwn=ed"));
+      base::Bind([] (RenderFrameHost* frame) {
+        GetFilterForProcess(frame->GetProcess())->SetCookie(
+            frame->GetRoutingID(), GURL("https://baz.com/"),
+            GURL("https://baz.com/"), "pwn=ed");
+      }, main_frame));
 
   main_frame_killed.Wait();
 
   EXPECT_EQ(
       " Site A\n"
       "Where A = http://127.0.0.1/ (no process)",
       v.DepictFrameTree(tab->GetFrameTree()->root()));
 }
 
 // FrameHostMsg_RenderProcessGone is a synthetic message that's really an
@@ skipping 22 matching lines @@
   // If the message had gone through, we'd have marked the RFH as dead but
   // left the RPH and its connection alive, and the Wait below would hang.
   web_process_killed.Wait();
 
   ASSERT_FALSE(web_rfh->GetProcess()->HasConnection());
   ASSERT_FALSE(web_rfh->IsRenderFrameLive());
   ASSERT_FALSE(web_process_killed.did_exit_normally());
 }
 
 }  // namespace content