Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(347)

Side by Side Diff: content/renderer/gpu/renderer_compositor_frame_sink.cc

Issue 2789773003: Send FrameSwapMessageQueue's messages with a separate IPC (Closed)
Patch Set: Fix rebase mistake Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/gpu/renderer_compositor_frame_sink.h" 5 #include "content/renderer/gpu/renderer_compositor_frame_sink.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/location.h" 10 #include "base/location.h"
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 } 123 }
124 124
125 void RendererCompositorFrameSink::SubmitCompositorFrame( 125 void RendererCompositorFrameSink::SubmitCompositorFrame(
126 cc::CompositorFrame frame) { 126 cc::CompositorFrame frame) {
127 // We should only submit CompositorFrames with valid BeginFrameAcks. 127 // We should only submit CompositorFrames with valid BeginFrameAcks.
128 DCHECK_LE(cc::BeginFrameArgs::kStartingFrameNumber, 128 DCHECK_LE(cc::BeginFrameArgs::kStartingFrameNumber,
129 frame.metadata.begin_frame_ack.sequence_number); 129 frame.metadata.begin_frame_ack.sequence_number);
130 if (ShouldAllocateNewLocalSurfaceId(frame)) 130 if (ShouldAllocateNewLocalSurfaceId(frame))
131 local_surface_id_ = id_allocator_.GenerateId(); 131 local_surface_id_ = id_allocator_.GenerateId();
132 UpdateFrameData(frame); 132 UpdateFrameData(frame);
133
133 { 134 {
134 std::unique_ptr<FrameSwapMessageQueue::SendMessageScope> 135 std::unique_ptr<FrameSwapMessageQueue::SendMessageScope>
135 send_message_scope = 136 send_message_scope =
136 frame_swap_message_queue_->AcquireSendMessageScope(); 137 frame_swap_message_queue_->AcquireSendMessageScope();
137 std::vector<std::unique_ptr<IPC::Message>> messages; 138 std::vector<std::unique_ptr<IPC::Message>> messages;
138 std::vector<IPC::Message> messages_to_deliver_with_frame;
139 frame_swap_message_queue_->DrainMessages(&messages); 139 frame_swap_message_queue_->DrainMessages(&messages);
140 FrameSwapMessageQueue::TransferMessages(&messages, 140 std::vector<IPC::Message> messages_to_send;
141 &messages_to_deliver_with_frame); 141 FrameSwapMessageQueue::TransferMessages(&messages, &messages_to_send);
142 uint32_t frame_token = 0;
143 if (!messages_to_send.empty())
144 frame_token = frame_swap_message_queue_->AllocateFrameToken();
145 frame.metadata.frame_token = frame_token;
142 Send(new ViewHostMsg_SwapCompositorFrame( 146 Send(new ViewHostMsg_SwapCompositorFrame(
143 routing_id_, compositor_frame_sink_id_, local_surface_id_, frame, 147 routing_id_, compositor_frame_sink_id_, local_surface_id_, frame));
144 messages_to_deliver_with_frame)); 148 if (frame_token) {
149 Send(new ViewHostMsg_FrameSwapMessages(routing_id_, frame_token,
150 messages_to_send));
151 }
145 // ~send_message_scope. 152 // ~send_message_scope.
146 } 153 }
147 } 154 }
148 155
149 void RendererCompositorFrameSink::OnMessageReceived( 156 void RendererCompositorFrameSink::OnMessageReceived(
150 const IPC::Message& message) { 157 const IPC::Message& message) {
151 DCHECK(thread_checker_.CalledOnValidThread()); 158 DCHECK(thread_checker_.CalledOnValidThread());
152 IPC_BEGIN_MESSAGE_MAP(RendererCompositorFrameSink, message) 159 IPC_BEGIN_MESSAGE_MAP(RendererCompositorFrameSink, message)
153 IPC_MESSAGE_HANDLER(ViewMsg_ReclaimCompositorResources, 160 IPC_MESSAGE_HANDLER(ViewMsg_ReclaimCompositorResources,
154 OnReclaimCompositorResources) 161 OnReclaimCompositorResources)
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
238 245
239 void RendererCompositorFrameSink::OnDidFinishFrame( 246 void RendererCompositorFrameSink::OnDidFinishFrame(
240 const cc::BeginFrameAck& ack) { 247 const cc::BeginFrameAck& ack) {
241 DCHECK_LE(cc::BeginFrameArgs::kStartingFrameNumber, ack.sequence_number); 248 DCHECK_LE(cc::BeginFrameArgs::kStartingFrameNumber, ack.sequence_number);
242 // If there was damage, ViewHostMsg_SwapCompositorFrame includes the ack. 249 // If there was damage, ViewHostMsg_SwapCompositorFrame includes the ack.
243 if (!ack.has_damage) 250 if (!ack.has_damage)
244 Send(new ViewHostMsg_BeginFrameDidNotSwap(routing_id_, ack)); 251 Send(new ViewHostMsg_BeginFrameDidNotSwap(routing_id_, ack));
245 } 252 }
246 253
247 } // namespace content 254 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/gpu/queue_message_swap_promise.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698