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

Side by Side Diff: content/browser/frame_host/render_widget_host_view_child_frame.cc

Issue 2774373002: Use MojoCompositorFrameSink in RendererCompositorFrameSink (Closed)
Patch Set: addressed comments Created 3 years, 8 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/frame_host/render_widget_host_view_child_frame.h" 5 #include "content/browser/frame_host/render_widget_host_view_child_frame.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <utility> 8 #include <utility>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 329 matching lines...) Expand 10 before | Expand all | Expand 10 after
340 // RenderWidgetHostInputEventRouter. 340 // RenderWidgetHostInputEventRouter.
341 if (!frame_connector_) 341 if (!frame_connector_)
342 return; 342 return;
343 if ((event.GetType() == blink::WebInputEvent::kGestureScrollUpdate && 343 if ((event.GetType() == blink::WebInputEvent::kGestureScrollUpdate &&
344 not_consumed) || 344 not_consumed) ||
345 event.GetType() == blink::WebInputEvent::kGestureScrollEnd) 345 event.GetType() == blink::WebInputEvent::kGestureScrollEnd)
346 frame_connector_->BubbleScrollEvent(event); 346 frame_connector_->BubbleScrollEvent(event);
347 } 347 }
348 348
349 void RenderWidgetHostViewChildFrame::DidReceiveCompositorFrameAck() { 349 void RenderWidgetHostViewChildFrame::DidReceiveCompositorFrameAck() {
350 if (!host_) 350 renderer_compositor_frame_sink_->DidReceiveCompositorFrameAck();
351 return;
352 host_->SendReclaimCompositorResources(true /* is_swap_ack */,
353 cc::ReturnedResourceArray());
354 } 351 }
355 352
356 void RenderWidgetHostViewChildFrame::DidCreateNewRendererCompositorFrameSink() { 353 void RenderWidgetHostViewChildFrame::DidCreateNewRendererCompositorFrameSink(
354 cc::mojom::MojoCompositorFrameSinkClient* renderer_compositor_frame_sink) {
357 ResetCompositorFrameSinkSupport(); 355 ResetCompositorFrameSinkSupport();
356 renderer_compositor_frame_sink_ = renderer_compositor_frame_sink;
358 CreateCompositorFrameSinkSupport(); 357 CreateCompositorFrameSinkSupport();
359 has_frame_ = false; 358 has_frame_ = false;
360 } 359 }
361 360
362 void RenderWidgetHostViewChildFrame::ProcessCompositorFrame( 361 void RenderWidgetHostViewChildFrame::ProcessCompositorFrame(
363 const cc::LocalSurfaceId& local_surface_id, 362 const cc::LocalSurfaceId& local_surface_id,
364 cc::CompositorFrame frame) { 363 cc::CompositorFrame frame) {
365 current_surface_size_ = frame.render_pass_list.back()->output_rect.size(); 364 current_surface_size_ = frame.render_pass_list.back()->output_rect.size();
366 current_surface_scale_factor_ = frame.metadata.device_scale_factor; 365 current_surface_scale_factor_ = frame.metadata.device_scale_factor;
367 366
(...skipping 242 matching lines...) Expand 10 before | Expand all | Expand 10 after
610 support_->RequestCopyOfSurface(std::move(request)); 609 support_->RequestCopyOfSurface(std::move(request));
611 } 610 }
612 611
613 bool RenderWidgetHostViewChildFrame::HasAcceleratedSurface( 612 bool RenderWidgetHostViewChildFrame::HasAcceleratedSurface(
614 const gfx::Size& desired_size) { 613 const gfx::Size& desired_size) {
615 return false; 614 return false;
616 } 615 }
617 616
618 void RenderWidgetHostViewChildFrame::ReclaimResources( 617 void RenderWidgetHostViewChildFrame::ReclaimResources(
619 const cc::ReturnedResourceArray& resources) { 618 const cc::ReturnedResourceArray& resources) {
620 if (!host_) 619 renderer_compositor_frame_sink_->ReclaimResources(resources);
621 return;
622 host_->SendReclaimCompositorResources(false /* is_swap_ack */, resources);
623 } 620 }
624 621
625 void RenderWidgetHostViewChildFrame::OnBeginFrame( 622 void RenderWidgetHostViewChildFrame::OnBeginFrame(
626 const cc::BeginFrameArgs& args) { 623 const cc::BeginFrameArgs& args) {
627 host_->Send(new ViewMsg_BeginFrame(host_->GetRoutingID(), args)); 624 host_->Send(new ViewMsg_BeginFrame(host_->GetRoutingID(), args));
628 } 625 }
629 626
630 void RenderWidgetHostViewChildFrame::SetNeedsBeginFrames( 627 void RenderWidgetHostViewChildFrame::SetNeedsBeginFrames(
631 bool needs_begin_frames) { 628 bool needs_begin_frames) {
632 if (support_) 629 if (support_)
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
702 frame_sink_id_); 699 frame_sink_id_);
703 } 700 }
704 support_.reset(); 701 support_.reset();
705 } 702 }
706 703
707 bool RenderWidgetHostViewChildFrame::HasEmbedderChanged() { 704 bool RenderWidgetHostViewChildFrame::HasEmbedderChanged() {
708 return false; 705 return false;
709 } 706 }
710 707
711 } // namespace content 708 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698