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

Side by Side Diff: content/renderer/render_widget.cc

Issue 2448543002: Remove dead paint flush tracking code. (Closed)
Patch Set: rebase 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
« no previous file with comments | « content/renderer/render_widget.h ('k') | content/renderer/render_widget_fullscreen_pepper.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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 "content/renderer/render_widget.h" 5 #include "content/renderer/render_widget.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/auto_reset.h" 10 #include "base/auto_reset.h"
(...skipping 757 matching lines...) Expand 10 before | Expand all | Expand 10 after
768 observer.DidCommitCompositorFrame(); 768 observer.DidCommitCompositorFrame();
769 for (auto& observer : render_frame_proxies_) 769 for (auto& observer : render_frame_proxies_)
770 observer.DidCommitCompositorFrame(); 770 observer.DidCommitCompositorFrame();
771 } 771 }
772 772
773 void RenderWidget::DidCompletePageScaleAnimation() {} 773 void RenderWidget::DidCompletePageScaleAnimation() {}
774 774
775 void RenderWidget::DidReceiveCompositorFrameAck() { 775 void RenderWidget::DidReceiveCompositorFrameAck() {
776 TRACE_EVENT0("renderer", "RenderWidget::DidReceiveCompositorFrameAck"); 776 TRACE_EVENT0("renderer", "RenderWidget::DidReceiveCompositorFrameAck");
777 777
778 // Notify subclasses threaded composited rendering was flushed to the screen.
779 DidFlushPaint();
780
781 if (!next_paint_flags_ && !need_update_rect_for_auto_resize_) { 778 if (!next_paint_flags_ && !need_update_rect_for_auto_resize_) {
782 return; 779 return;
783 } 780 }
784 781
785 ViewHostMsg_UpdateRect_Params params; 782 ViewHostMsg_UpdateRect_Params params;
786 params.view_size = size_; 783 params.view_size = size_;
787 params.flags = next_paint_flags_; 784 params.flags = next_paint_flags_;
788 785
789 Send(new ViewHostMsg_UpdateRect(routing_id_, params)); 786 Send(new ViewHostMsg_UpdateRect(routing_id_, params));
790 next_paint_flags_ = 0; 787 next_paint_flags_ = 0;
(...skipping 893 matching lines...) Expand 10 before | Expand all | Expand 10 after
1684 1681
1685 if (owner_delegate_) 1682 if (owner_delegate_)
1686 owner_delegate_->RenderWidgetDidSetColorProfile(color_profile); 1683 owner_delegate_->RenderWidgetDidSetColorProfile(color_profile);
1687 1684
1688 return true; 1685 return true;
1689 } 1686 }
1690 1687
1691 void RenderWidget::OnOrientationChange() { 1688 void RenderWidget::OnOrientationChange() {
1692 } 1689 }
1693 1690
1694 void RenderWidget::DidFlushPaint() {
1695 if (owner_delegate_)
1696 owner_delegate_->RenderWidgetDidFlushPaint();
1697 }
1698
1699 void RenderWidget::SetHidden(bool hidden) { 1691 void RenderWidget::SetHidden(bool hidden) {
1700 if (is_hidden_ == hidden) 1692 if (is_hidden_ == hidden)
1701 return; 1693 return;
1702 1694
1703 // The status has changed. Tell the RenderThread about it and ensure 1695 // The status has changed. Tell the RenderThread about it and ensure
1704 // throttled acks are released in case frame production ceases. 1696 // throttled acks are released in case frame production ceases.
1705 is_hidden_ = hidden; 1697 is_hidden_ = hidden;
1706 1698
1707 if (is_hidden_) 1699 if (is_hidden_)
1708 RenderThreadImpl::current()->WidgetHidden(); 1700 RenderThreadImpl::current()->WidgetHidden();
(...skipping 388 matching lines...) Expand 10 before | Expand all | Expand 10 after
2097 bool RenderWidget::isPointerLocked() { 2089 bool RenderWidget::isPointerLocked() {
2098 return mouse_lock_dispatcher_->IsMouseLockedTo( 2090 return mouse_lock_dispatcher_->IsMouseLockedTo(
2099 webwidget_mouse_lock_target_.get()); 2091 webwidget_mouse_lock_target_.get());
2100 } 2092 }
2101 2093
2102 blink::WebWidget* RenderWidget::GetWebWidget() const { 2094 blink::WebWidget* RenderWidget::GetWebWidget() const {
2103 return webwidget_internal_; 2095 return webwidget_internal_;
2104 } 2096 }
2105 2097
2106 } // namespace content 2098 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/render_widget.h ('k') | content/renderer/render_widget_fullscreen_pepper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698