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

Side by Side Diff: cc/output/software_renderer.cc

Issue 2374183006: Remove GLFrameData from CompositorFrame. (Closed)
Patch Set: Rebase + fix includes. Created 4 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 2012 The Chromium Authors. All rights reserved. 1 // Copyright 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 "cc/output/software_renderer.h" 5 #include "cc/output/software_renderer.h"
6 6
7 #include "base/memory/ptr_util.h" 7 #include "base/memory/ptr_util.h"
8 #include "base/trace_event/trace_event.h" 8 #include "base/trace_event/trace_event.h"
9 #include "cc/base/math_util.h" 9 #include "cc/base/math_util.h"
10 #include "cc/output/compositor_frame.h"
11 #include "cc/output/compositor_frame_metadata.h" 10 #include "cc/output/compositor_frame_metadata.h"
12 #include "cc/output/copy_output_request.h" 11 #include "cc/output/copy_output_request.h"
13 #include "cc/output/output_surface.h" 12 #include "cc/output/output_surface.h"
13 #include "cc/output/output_surface_frame.h"
14 #include "cc/output/render_surface_filters.h" 14 #include "cc/output/render_surface_filters.h"
15 #include "cc/output/renderer_settings.h" 15 #include "cc/output/renderer_settings.h"
16 #include "cc/output/software_output_device.h" 16 #include "cc/output/software_output_device.h"
17 #include "cc/quads/debug_border_draw_quad.h" 17 #include "cc/quads/debug_border_draw_quad.h"
18 #include "cc/quads/picture_draw_quad.h" 18 #include "cc/quads/picture_draw_quad.h"
19 #include "cc/quads/render_pass_draw_quad.h" 19 #include "cc/quads/render_pass_draw_quad.h"
20 #include "cc/quads/solid_color_draw_quad.h" 20 #include "cc/quads/solid_color_draw_quad.h"
21 #include "cc/quads/texture_draw_quad.h" 21 #include "cc/quads/texture_draw_quad.h"
22 #include "cc/quads/tile_draw_quad.h" 22 #include "cc/quads/tile_draw_quad.h"
23 #include "cc/resources/scoped_resource.h" 23 #include "cc/resources/scoped_resource.h"
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 void SoftwareRenderer::FinishDrawingFrame(DrawingFrame* frame) { 74 void SoftwareRenderer::FinishDrawingFrame(DrawingFrame* frame) {
75 TRACE_EVENT0("cc", "SoftwareRenderer::FinishDrawingFrame"); 75 TRACE_EVENT0("cc", "SoftwareRenderer::FinishDrawingFrame");
76 current_framebuffer_lock_ = nullptr; 76 current_framebuffer_lock_ = nullptr;
77 current_framebuffer_canvas_.reset(); 77 current_framebuffer_canvas_.reset();
78 current_canvas_ = nullptr; 78 current_canvas_ = nullptr;
79 root_canvas_ = nullptr; 79 root_canvas_ = nullptr;
80 80
81 output_device_->EndPaint(); 81 output_device_->EndPaint();
82 } 82 }
83 83
84 void SoftwareRenderer::SwapBuffers(CompositorFrameMetadata metadata) { 84 void SoftwareRenderer::SwapBuffers(CompositorFrameMetadata metadata) {
danakj 2016/10/04 21:58:40 same here, can it take vector instead of full meta
kylechar 2016/10/05 13:45:54 Done.
85 DCHECK(visible_); 85 DCHECK(visible_);
86 TRACE_EVENT0("cc,benchmark", "SoftwareRenderer::SwapBuffers"); 86 TRACE_EVENT0("cc,benchmark", "SoftwareRenderer::SwapBuffers");
87 CompositorFrame compositor_frame; 87 OutputSurfaceFrame frame;
danakj 2016/10/04 21:58:40 nit: output_frame or something?
kylechar 2016/10/05 13:45:54 Done.
88 compositor_frame.metadata = std::move(metadata); 88 frame.latency_info = std::move(metadata.latency_info);
89 output_surface_->SwapBuffers(std::move(compositor_frame)); 89 output_surface_->SwapBuffers(std::move(frame));
90 } 90 }
91 91
92 bool SoftwareRenderer::FlippedFramebuffer(const DrawingFrame* frame) const { 92 bool SoftwareRenderer::FlippedFramebuffer(const DrawingFrame* frame) const {
93 return false; 93 return false;
94 } 94 }
95 95
96 void SoftwareRenderer::EnsureScissorTestEnabled() { 96 void SoftwareRenderer::EnsureScissorTestEnabled() {
97 is_scissor_enabled_ = true; 97 is_scissor_enabled_ = true;
98 SetClipRect(scissor_rect_); 98 SetClipRect(scissor_rect_);
99 } 99 }
(...skipping 610 matching lines...) Expand 10 before | Expand all | Expand 10 after
710 ApplyImageFilter(filter.get(), quad, backdrop_bitmap, nullptr); 710 ApplyImageFilter(filter.get(), quad, backdrop_bitmap, nullptr);
711 711
712 if (!filter_backdrop_image) 712 if (!filter_backdrop_image)
713 return nullptr; 713 return nullptr;
714 714
715 return filter_backdrop_image->makeShader(content_tile_mode, content_tile_mode, 715 return filter_backdrop_image->makeShader(content_tile_mode, content_tile_mode,
716 &filter_backdrop_transform); 716 &filter_backdrop_transform);
717 } 717 }
718 718
719 } // namespace cc 719 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698