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

Unified Diff: cc/output/gl_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 side-by-side diff with in-line comments
Download patch
Index: cc/output/gl_renderer.cc
diff --git a/cc/output/gl_renderer.cc b/cc/output/gl_renderer.cc
index b882c276b594f23dbbd097ac8f57f3cf7bcd3b29..2977cb5d600bd84ef09ad6cc4e301a748cc967ee 100644
--- a/cc/output/gl_renderer.cc
+++ b/cc/output/gl_renderer.cc
@@ -31,9 +31,9 @@
#include "cc/output/context_provider.h"
#include "cc/output/copy_output_request.h"
#include "cc/output/dynamic_geometry_binding.h"
-#include "cc/output/gl_frame_data.h"
#include "cc/output/layer_quad.h"
#include "cc/output/output_surface.h"
+#include "cc/output/output_surface_frame.h"
#include "cc/output/render_surface_filters.h"
#include "cc/output/renderer_settings.h"
#include "cc/output/static_geometry_binding.h"
@@ -2892,10 +2892,9 @@ void GLRenderer::SwapBuffers(CompositorFrameMetadata metadata) {
gfx::Size surface_size = output_surface_->SurfaceSize();
- CompositorFrame compositor_frame;
- compositor_frame.metadata = std::move(metadata);
- compositor_frame.gl_frame_data = base::WrapUnique(new GLFrameData);
- compositor_frame.gl_frame_data->size = surface_size;
+ OutputSurfaceFrame frame;
danakj 2016/10/04 21:58:40 nit: name the variable output_frame or something a
kylechar 2016/10/05 13:45:53 Done.
+ frame.latency_info = std::move(metadata.latency_info);
+ frame.size = surface_size;
if (use_partial_swap_) {
// If supported, we can save significant bandwidth by only swapping the
// damaged/scissored region (clamped to the viewport).
@@ -2903,7 +2902,7 @@ void GLRenderer::SwapBuffers(CompositorFrameMetadata metadata) {
int flipped_y_pos_of_rect_bottom = surface_size.height() -
swap_buffer_rect_.y() -
swap_buffer_rect_.height();
- compositor_frame.gl_frame_data->sub_buffer_rect =
+ frame.sub_buffer_rect =
gfx::Rect(swap_buffer_rect_.x(),
FlippedRootFramebuffer() ? flipped_y_pos_of_rect_bottom
: swap_buffer_rect_.y(),
@@ -2914,13 +2913,13 @@ void GLRenderer::SwapBuffers(CompositorFrameMetadata metadata) {
if (!swap_buffer_rect_.IsEmpty() || !allow_empty_swap_) {
swap_buffer_rect_ = gfx::Rect(surface_size);
}
- compositor_frame.gl_frame_data->sub_buffer_rect = swap_buffer_rect_;
+ frame.sub_buffer_rect = swap_buffer_rect_;
}
swapping_overlay_resources_.push_back(std::move(pending_overlay_resources_));
pending_overlay_resources_.clear();
- output_surface_->SwapBuffers(std::move(compositor_frame));
+ output_surface_->SwapBuffers(std::move(frame));
swap_buffer_rect_ = gfx::Rect();
}

Powered by Google App Engine
This is Rietveld 408576698