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

Side by Side Diff: android_webview/browser/test/rendering_test.cc

Issue 2449853004: Getting rid of DelegatedFrameData (Closed)
Patch Set: nit Created 4 years, 1 month 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 "android_webview/browser/test/rendering_test.h" 5 #include "android_webview/browser/test/rendering_test.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "android_webview/browser/browser_view_renderer.h" 9 #include "android_webview/browser/browser_view_renderer.h"
10 #include "android_webview/browser/child_frame.h" 10 #include "android_webview/browser/child_frame.h"
(...skipping 99 matching lines...) Expand 10 before | Expand all | Expand 10 after
110 ui_task_runner_->PostTask(FROM_HERE, run_loop_.QuitWhenIdleClosure()); 110 ui_task_runner_->PostTask(FROM_HERE, run_loop_.QuitWhenIdleClosure());
111 } 111 }
112 112
113 content::SynchronousCompositor* RenderingTest::ActiveCompositor() const { 113 content::SynchronousCompositor* RenderingTest::ActiveCompositor() const {
114 return browser_view_renderer_->GetActiveCompositorForTesting(); 114 return browser_view_renderer_->GetActiveCompositorForTesting();
115 } 115 }
116 116
117 std::unique_ptr<cc::CompositorFrame> RenderingTest::ConstructEmptyFrame() { 117 std::unique_ptr<cc::CompositorFrame> RenderingTest::ConstructEmptyFrame() {
118 std::unique_ptr<cc::CompositorFrame> compositor_frame( 118 std::unique_ptr<cc::CompositorFrame> compositor_frame(
119 new cc::CompositorFrame); 119 new cc::CompositorFrame);
120 std::unique_ptr<cc::DelegatedFrameData> frame(new cc::DelegatedFrameData);
121 std::unique_ptr<cc::RenderPass> root_pass(cc::RenderPass::Create()); 120 std::unique_ptr<cc::RenderPass> root_pass(cc::RenderPass::Create());
122 gfx::Rect viewport(browser_view_renderer_->size()); 121 gfx::Rect viewport(browser_view_renderer_->size());
123 root_pass->SetNew(cc::RenderPassId(1, 1), viewport, viewport, 122 root_pass->SetNew(cc::RenderPassId(1, 1), viewport, viewport,
124 gfx::Transform()); 123 gfx::Transform());
125 frame->render_pass_list.push_back(std::move(root_pass)); 124 compositor_frame->render_pass_list.push_back(std::move(root_pass));
126 compositor_frame->delegated_frame_data = std::move(frame);
127 return compositor_frame; 125 return compositor_frame;
128 } 126 }
129 127
130 std::unique_ptr<cc::CompositorFrame> RenderingTest::ConstructFrame( 128 std::unique_ptr<cc::CompositorFrame> RenderingTest::ConstructFrame(
131 cc::ResourceId resource_id) { 129 cc::ResourceId resource_id) {
132 std::unique_ptr<cc::CompositorFrame> compositor_frame(ConstructEmptyFrame()); 130 std::unique_ptr<cc::CompositorFrame> compositor_frame(ConstructEmptyFrame());
133 cc::TransferableResource resource; 131 cc::TransferableResource resource;
134 resource.id = resource_id; 132 resource.id = resource_id;
135 compositor_frame->delegated_frame_data->resource_list.push_back(resource); 133 compositor_frame->resource_list.push_back(resource);
136 return compositor_frame; 134 return compositor_frame;
137 } 135 }
138 136
139 FakeFunctor* RenderingTest::GetFunctor() { 137 FakeFunctor* RenderingTest::GetFunctor() {
140 return functor_.get(); 138 return functor_.get();
141 } 139 }
142 140
143 void RenderingTest::WillOnDraw() { 141 void RenderingTest::WillOnDraw() {
144 DCHECK(compositor_); 142 DCHECK(compositor_);
145 compositor_->SetHardwareFrame(0u, ConstructEmptyFrame()); 143 compositor_->SetHardwareFrame(0u, ConstructEmptyFrame());
(...skipping 13 matching lines...) Expand all
159 void RenderingTest::PostInvalidate() { 157 void RenderingTest::PostInvalidate() {
160 if (window_) 158 if (window_)
161 window_->PostInvalidate(); 159 window_->PostInvalidate();
162 } 160 }
163 161
164 gfx::Point RenderingTest::GetLocationOnScreen() { 162 gfx::Point RenderingTest::GetLocationOnScreen() {
165 return gfx::Point(); 163 return gfx::Point();
166 } 164 }
167 165
168 } // namespace android_webview 166 } // namespace android_webview
OLDNEW
« no previous file with comments | « android_webview/browser/surfaces_instance.cc ('k') | blimp/client/core/compositor/blimp_compositor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698