Moves FrameHostMsg_SetCookie to a Mojo message

content/renderer/render_frame_impl.cc

 // Copyright 2013 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 "content/renderer/render_frame_impl.h"
 
 #include <map>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 1433 matching lines @@
 }
 #endif  // defined(ENABLE_PLUGINS)
 
 MediaStreamDispatcher* RenderFrameImpl::GetMediaStreamDispatcher() {
   if (!web_user_media_client_)
     InitializeUserMediaClient();
   return web_user_media_client_ ?
       web_user_media_client_->media_stream_dispatcher() : NULL;
 }
 
+mojom::RenderFrameMessageFilter*
+RenderFrameImpl::render_frame_message_filter() {
+  if (!render_frame_message_filter_) {
+    RenderThread::Get()->GetChannel()->GetRemoteAssociatedInterface(

[jam, 2016/07/26 19:13:32]

it's a bit odd that this mojo interface is registered on the process host object
on the browser side, but on the renderer it's fetched via the frame object. it
seems that there should be symmetry.

any reason why we can't make a getter for this on the render thread class? that
way we'd only create one MP for this, which I think is all we need.

[Ken Rockot(use gerrit already), 2016/07/26 21:00:34]

Done

+        &render_frame_message_filter_);
+  }
+  return render_frame_message_filter_.get();
+}
 bool RenderFrameImpl::Send(IPC::Message* message) {
   return RenderThread::Get()->Send(message);
 }
 
 #if defined(USE_EXTERNAL_POPUP_MENU)
 void RenderFrameImpl::DidHideExternalPopupMenu() {
   // We need to clear external_popup_menu_ as soon as ExternalPopupMenu::close
   // is called. Otherwise, createExternalPopupMenu() for new popup will fail.
   external_popup_menu_.reset();
 }
@@ skipping 4876 matching lines @@
   // event target. Potentially a Pepper plugin will receive the event.
   // In order to tell whether a plugin gets the last mouse event and which it
   // is, we set |pepper_last_mouse_event_target_| to null here. If a plugin gets
   // the event, it will notify us via DidReceiveMouseEvent() and set itself as
   // |pepper_last_mouse_event_target_|.
   pepper_last_mouse_event_target_ = nullptr;
 #endif
 }
 
 }  // namespace content