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

Side by Side Diff: content/browser/compositor/gpu_browser_compositor_output_surface.cc

Issue 628703005: Remove GpuHostMsg_FrameDrawn and replace with client channel swap ack (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments Created 6 years, 2 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/browser/compositor/gpu_browser_compositor_output_surface.h" 5 #include "content/browser/compositor/gpu_browser_compositor_output_surface.h"
6 6
7 #include "cc/output/compositor_frame.h" 7 #include "cc/output/compositor_frame.h"
8 #include "cc/output/output_surface_client.h"
8 #include "content/browser/compositor/reflector_impl.h" 9 #include "content/browser/compositor/reflector_impl.h"
10 #include "content/browser/renderer_host/render_widget_host_impl.h"
9 #include "content/common/gpu/client/context_provider_command_buffer.h" 11 #include "content/common/gpu/client/context_provider_command_buffer.h"
12 #include "gpu/command_buffer/client/context_support.h"
10 #include "gpu/command_buffer/client/gles2_interface.h" 13 #include "gpu/command_buffer/client/gles2_interface.h"
11 14
12 namespace content { 15 namespace content {
13 16
14 GpuBrowserCompositorOutputSurface::GpuBrowserCompositorOutputSurface( 17 GpuBrowserCompositorOutputSurface::GpuBrowserCompositorOutputSurface(
15 const scoped_refptr<ContextProviderCommandBuffer>& context, 18 const scoped_refptr<ContextProviderCommandBuffer>& context,
16 int surface_id, 19 int surface_id,
17 IDMap<BrowserCompositorOutputSurface>* output_surface_map, 20 IDMap<BrowserCompositorOutputSurface>* output_surface_map,
18 const scoped_refptr<ui::CompositorVSyncManager>& vsync_manager, 21 const scoped_refptr<ui::CompositorVSyncManager>& vsync_manager,
19 scoped_ptr<cc::OverlayCandidateValidator> overlay_candidate_validator) 22 scoped_ptr<cc::OverlayCandidateValidator> overlay_candidate_validator)
20 : BrowserCompositorOutputSurface(context, 23 : BrowserCompositorOutputSurface(context,
21 surface_id, 24 surface_id,
22 output_surface_map, 25 output_surface_map,
23 vsync_manager) { 26 vsync_manager),
27 swap_buffers_completion_callback_(
28 base::Bind(&GpuBrowserCompositorOutputSurface::OnSwapBuffersCompleted,
29 base::Unretained(this))) {
24 overlay_candidate_validator_ = overlay_candidate_validator.Pass(); 30 overlay_candidate_validator_ = overlay_candidate_validator.Pass();
25 } 31 }
26 32
27 GpuBrowserCompositorOutputSurface::~GpuBrowserCompositorOutputSurface() {} 33 GpuBrowserCompositorOutputSurface::~GpuBrowserCompositorOutputSurface() {}
28 34
35 CommandBufferProxyImpl*
36 GpuBrowserCompositorOutputSurface::GetCommandBufferProxy() {
37 ContextProviderCommandBuffer* provider_command_buffer =
38 static_cast<content::ContextProviderCommandBuffer*>(
39 context_provider_.get());
40 CommandBufferProxyImpl* command_buffer_proxy =
41 provider_command_buffer->GetCommandBufferProxy();
42 DCHECK(command_buffer_proxy);
43 return command_buffer_proxy;
44 }
45
46 bool GpuBrowserCompositorOutputSurface::BindToClient(
47 cc::OutputSurfaceClient* client) {
48 if (!BrowserCompositorOutputSurface::BindToClient(client))
49 return false;
50
51 GetCommandBufferProxy()->SetSwapBuffersCompletionCallback(
52 swap_buffers_completion_callback_.callback());
53 return true;
54 }
55
29 void GpuBrowserCompositorOutputSurface::SwapBuffers( 56 void GpuBrowserCompositorOutputSurface::SwapBuffers(
30 cc::CompositorFrame* frame) { 57 cc::CompositorFrame* frame) {
31 DCHECK(frame->gl_frame_data); 58 DCHECK(frame->gl_frame_data);
32 59
33 ContextProviderCommandBuffer* provider_command_buffer = 60 GetCommandBufferProxy()->SetLatencyInfo(frame->metadata.latency_info);
34 static_cast<ContextProviderCommandBuffer*>(context_provider_.get());
35 CommandBufferProxyImpl* command_buffer_proxy =
36 provider_command_buffer->GetCommandBufferProxy();
37 DCHECK(command_buffer_proxy);
38 command_buffer_proxy->SetLatencyInfo(frame->metadata.latency_info);
39 61
40 if (reflector_.get()) { 62 if (reflector_.get()) {
41 if (frame->gl_frame_data->sub_buffer_rect == 63 if (frame->gl_frame_data->sub_buffer_rect ==
42 gfx::Rect(frame->gl_frame_data->size)) 64 gfx::Rect(frame->gl_frame_data->size))
43 reflector_->OnSwapBuffers(); 65 reflector_->OnSwapBuffers();
44 else 66 else
45 reflector_->OnPostSubBuffer(frame->gl_frame_data->sub_buffer_rect); 67 reflector_->OnPostSubBuffer(frame->gl_frame_data->sub_buffer_rect);
46 } 68 }
47 69
48 OutputSurface::SwapBuffers(frame); 70 if (frame->gl_frame_data->sub_buffer_rect ==
71 gfx::Rect(frame->gl_frame_data->size)) {
72 context_provider_->ContextSupport()->Swap();
73 } else {
74 context_provider_->ContextSupport()->PartialSwapBuffers(
75 frame->gl_frame_data->sub_buffer_rect);
76 }
77
78 client_->DidSwapBuffers();
79 }
80
81 void GpuBrowserCompositorOutputSurface::OnSwapBuffersCompleted(
82 const std::vector<ui::LatencyInfo>& latency_info) {
83 RenderWidgetHostImpl::CompositorFrameDrawn(latency_info);
84 BrowserCompositorOutputSurface::OnSwapBuffersComplete();
49 } 85 }
50 86
51 } // namespace content 87 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698