Send BeginFrames for the renderer using Mojo

content/renderer/gpu/renderer_compositor_frame_sink.cc

 // Copyright (c) 2012 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/gpu/renderer_compositor_frame_sink.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/location.h"
@@ skipping 150 matching lines @@
   }
 }
 
 void RendererCompositorFrameSink::DidNotProduceFrame(
     const cc::BeginFrameAck& ack) {
   DCHECK(!ack.has_damage);
   DCHECK_LE(cc::BeginFrameArgs::kStartingFrameNumber, ack.sequence_number);
   sink_->DidNotProduceFrame(ack);
 }
 
 void RendererCompositorFrameSink::OnMessageReceived(

[piman, 2017/05/26 16:23:00]

Well then, if this is empty, I think we can remove
RendererCompositorFrameSinkProxy and CompositorForwardingMessageFilter
altogether.

[Saman Sami, 2017/05/26 16:27:35]

I thought about that as well. Even though it's not used now, I think I'll need
them very soon. I'll get rid of them if that's not the case.

     const IPC::Message& message) {
   DCHECK(thread_checker_.CalledOnValidThread());
-  IPC_BEGIN_MESSAGE_MAP(RendererCompositorFrameSink, message)
-    IPC_MESSAGE_HANDLER(ViewMsg_BeginFrame, OnBeginFrameIPC)
-  IPC_END_MESSAGE_MAP()
-}
-
-void RendererCompositorFrameSink::OnBeginFrameIPC(
-    const cc::BeginFrameArgs& args) {
-  if (external_begin_frame_source_)
-    external_begin_frame_source_->OnBeginFrame(args);
 }
 
 void RendererCompositorFrameSink::DidReceiveCompositorFrameAck(
     const cc::ReturnedResourceArray& resources) {
   ReclaimResources(resources);
   client_->DidReceiveCompositorFrameAck();
 }
 
 void RendererCompositorFrameSink::OnBeginFrame(const cc::BeginFrameArgs& args) {
-  // See crbug.com/709689.
-  NOTREACHED() << "BeginFrames are delivered using Chrome IPC.";
+  if (external_begin_frame_source_)
+    external_begin_frame_source_->OnBeginFrame(args);
 }
 
 void RendererCompositorFrameSink::ReclaimResources(
     const cc::ReturnedResourceArray& resources) {
   client_->ReclaimResources(resources);
 }
 
 void RendererCompositorFrameSink::OnNeedsBeginFrames(bool needs_begin_frames) {
   sink_->SetNeedsBeginFrame(needs_begin_frames);
 }
 
 }  // namespace content