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

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

Issue 2096493002: Make cc::CompositorFrames movable [Part 1 of 2] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed Dana's nits Created 4 years, 6 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
« no previous file with comments | « content/renderer/gpu/compositor_output_surface.h ('k') | content/test/mailbox_output_surface.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/compositor_output_surface.h" 5 #include "content/renderer/gpu/compositor_output_surface.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 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 if (!HasClient()) 96 if (!HasClient())
97 return; 97 return;
98 if (output_surface_proxy_) { 98 if (output_surface_proxy_) {
99 output_surface_proxy_->ClearOutputSurface(); 99 output_surface_proxy_->ClearOutputSurface();
100 output_surface_filter_->RemoveHandlerOnCompositorThread( 100 output_surface_filter_->RemoveHandlerOnCompositorThread(
101 routing_id_, output_surface_filter_handler_); 101 routing_id_, output_surface_filter_handler_);
102 } 102 }
103 cc::OutputSurface::DetachFromClient(); 103 cc::OutputSurface::DetachFromClient();
104 } 104 }
105 105
106 void CompositorOutputSurface::SwapBuffers(cc::CompositorFrame* frame) { 106 void CompositorOutputSurface::SwapBuffers(cc::CompositorFrame frame) {
107 { 107 {
108 std::unique_ptr<FrameSwapMessageQueue::SendMessageScope> 108 std::unique_ptr<FrameSwapMessageQueue::SendMessageScope>
109 send_message_scope = 109 send_message_scope =
110 frame_swap_message_queue_->AcquireSendMessageScope(); 110 frame_swap_message_queue_->AcquireSendMessageScope();
111 std::vector<std::unique_ptr<IPC::Message>> messages; 111 std::vector<std::unique_ptr<IPC::Message>> messages;
112 std::vector<IPC::Message> messages_to_deliver_with_frame; 112 std::vector<IPC::Message> messages_to_deliver_with_frame;
113 frame_swap_message_queue_->DrainMessages(&messages); 113 frame_swap_message_queue_->DrainMessages(&messages);
114 FrameSwapMessageQueue::TransferMessages(&messages, 114 FrameSwapMessageQueue::TransferMessages(&messages,
115 &messages_to_deliver_with_frame); 115 &messages_to_deliver_with_frame);
116 Send(new ViewHostMsg_SwapCompositorFrame(routing_id_, output_surface_id_, 116 Send(new ViewHostMsg_SwapCompositorFrame(routing_id_, output_surface_id_,
117 *frame, 117 frame,
118 messages_to_deliver_with_frame)); 118 messages_to_deliver_with_frame));
119 // ~send_message_scope. 119 // ~send_message_scope.
120 } 120 }
121 client_->DidSwapBuffers(); 121 client_->DidSwapBuffers();
122 } 122 }
123 123
124 void CompositorOutputSurface::BindFramebuffer() { 124 void CompositorOutputSurface::BindFramebuffer() {
125 // This is a delegating output surface, no framebuffer/direct drawing support. 125 // This is a delegating output surface, no framebuffer/direct drawing support.
126 NOTREACHED(); 126 NOTREACHED();
127 } 127 }
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
173 if (output_surface_id != output_surface_id_) 173 if (output_surface_id != output_surface_id_)
174 return; 174 return;
175 ReclaimResources(&ack); 175 ReclaimResources(&ack);
176 } 176 }
177 177
178 bool CompositorOutputSurface::Send(IPC::Message* message) { 178 bool CompositorOutputSurface::Send(IPC::Message* message) {
179 return message_sender_->Send(message); 179 return message_sender_->Send(message);
180 } 180 }
181 181
182 } // namespace content 182 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/gpu/compositor_output_surface.h ('k') | content/test/mailbox_output_surface.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698