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

Side by Side Diff: services/ui/ws/frame_generator.cc

Issue 2548383002: Remove two NOTIMPLEMENTED() added in CL 2544103002 to reduce console spam (Closed)
Patch Set: Removed NOTIMPLEMENTED() in aura::WindowCompositorFrameSink::WillDrawSurface() Created 4 years 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "services/ui/ws/frame_generator.h" 5 #include "services/ui/ws/frame_generator.h"
6 6
7 #include "base/containers/adapters.h" 7 #include "base/containers/adapters.h"
8 #include "cc/output/compositor_frame.h" 8 #include "cc/output/compositor_frame.h"
9 #include "cc/quads/render_pass.h" 9 #include "cc/quads/render_pass.h"
10 #include "cc/quads/render_pass_draw_quad.h" 10 #include "cc/quads/render_pass_draw_quad.h"
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
120 } 120 }
121 121
122 void FrameGenerator::ReclaimResources( 122 void FrameGenerator::ReclaimResources(
123 const cc::ReturnedResourceArray& resources) { 123 const cc::ReturnedResourceArray& resources) {
124 // Nothing to do here because FrameGenerator CompositorFrames don't reference 124 // Nothing to do here because FrameGenerator CompositorFrames don't reference
125 // any resources. 125 // any resources.
126 } 126 }
127 127
128 void FrameGenerator::WillDrawSurface() { 128 void FrameGenerator::WillDrawSurface() {
129 // TODO(fsamuel, staraz): Implement this. 129 // TODO(fsamuel, staraz): Implement this.
130 NOTIMPLEMENTED();
131 } 130 }
132 131
133 cc::CompositorFrame FrameGenerator::GenerateCompositorFrame( 132 cc::CompositorFrame FrameGenerator::GenerateCompositorFrame(
134 const gfx::Rect& output_rect) { 133 const gfx::Rect& output_rect) {
135 const cc::RenderPassId render_pass_id(1, 1); 134 const cc::RenderPassId render_pass_id(1, 1);
136 std::unique_ptr<cc::RenderPass> render_pass = cc::RenderPass::Create(); 135 std::unique_ptr<cc::RenderPass> render_pass = cc::RenderPass::Create();
137 render_pass->SetNew(render_pass_id, output_rect, output_rect, 136 render_pass->SetNew(render_pass_id, output_rect, output_rect,
138 gfx::Transform()); 137 gfx::Transform());
139 138
140 DrawWindowTree(render_pass.get(), root_window_, gfx::Vector2d(), 1.0f); 139 DrawWindowTree(render_pass.get(), root_window_, gfx::Vector2d(), 1.0f);
(...skipping 217 matching lines...) Expand 10 before | Expand all | Expand 10 after
358 cc::SurfaceId underlay_surface_id = 357 cc::SurfaceId underlay_surface_id =
359 window->compositor_frame_sink_manager()->GetLatestSurfaceId( 358 window->compositor_frame_sink_manager()->GetLatestSurfaceId(
360 mojom::CompositorFrameSinkType::UNDERLAY); 359 mojom::CompositorFrameSinkType::UNDERLAY);
361 if (underlay_surface_id.is_valid()) 360 if (underlay_surface_id.is_valid())
362 RemoveFrameSinkReference(underlay_surface_id.frame_sink_id()); 361 RemoveFrameSinkReference(underlay_surface_id.frame_sink_id());
363 } 362 }
364 363
365 } // namespace ws 364 } // namespace ws
366 365
367 } // namespace ui 366 } // namespace ui
OLDNEW
« no previous file with comments | « services/ui/public/cpp/window_compositor_frame_sink.cc ('k') | ui/aura/mus/window_compositor_frame_sink.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698