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

Side by Side Diff: third_party/WebKit/Source/core/frame/FrameView.cpp

Issue 2657863004: Move scroll paint property nodes to be owned by the transform tree (Closed)
Patch Set: Created 3 years, 10 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 /* 1 /*
2 * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org> 2 * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
3 * 1999 Lars Knoll <knoll@kde.org> 3 * 1999 Lars Knoll <knoll@kde.org>
4 * 1999 Antti Koivisto <koivisto@kde.org> 4 * 1999 Antti Koivisto <koivisto@kde.org>
5 * 2000 Dirk Mueller <mueller@kde.org> 5 * 2000 Dirk Mueller <mueller@kde.org>
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved. 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
7 * (C) 2006 Graham Dennis (graham.dennis@gmail.com) 7 * (C) 2006 Graham Dennis (graham.dennis@gmail.com)
8 * (C) 2006 Alexey Proskuryakov (ap@nypop.com) 8 * (C) 2006 Alexey Proskuryakov (ap@nypop.com)
9 * Copyright (C) 2009 Google Inc. All rights reserved. 9 * Copyright (C) 2009 Google Inc. All rights reserved.
10 * 10 *
(...skipping 4943 matching lines...) Expand 10 before | Expand all | Expand 10 after
4954 return reasons; 4954 return reasons;
4955 } 4955 }
4956 4956
4957 String FrameView::mainThreadScrollingReasonsAsText() const { 4957 String FrameView::mainThreadScrollingReasonsAsText() const {
4958 if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) { 4958 if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
4959 DCHECK(lifecycle().state() >= DocumentLifecycle::PrePaintClean); 4959 DCHECK(lifecycle().state() >= DocumentLifecycle::PrePaintClean);
4960 4960
4961 // Slimming paint v2 stores main thread scrolling reasons on property 4961 // Slimming paint v2 stores main thread scrolling reasons on property
4962 // trees instead of in |m_mainThreadScrollingReasons|. 4962 // trees instead of in |m_mainThreadScrollingReasons|.
4963 MainThreadScrollingReasons reasons = 0; 4963 MainThreadScrollingReasons reasons = 0;
4964 if (const auto* scrollNode = scroll()) 4964 if (const auto* scrollTranslation = this->scrollTranslation())
4965 reasons |= scrollNode->mainThreadScrollingReasons(); 4965 reasons |= scrollTranslation->scrollNode()->mainThreadScrollingReasons();
4966 return String( 4966 return String(
4967 MainThreadScrollingReason::mainThreadScrollingReasonsAsText(reasons) 4967 MainThreadScrollingReason::mainThreadScrollingReasonsAsText(reasons)
4968 .c_str()); 4968 .c_str());
4969 } 4969 }
4970 4970
4971 DCHECK(lifecycle().state() >= DocumentLifecycle::CompositingClean); 4971 DCHECK(lifecycle().state() >= DocumentLifecycle::CompositingClean);
4972 if (layerForScrolling() && layerForScrolling()->platformLayer()) { 4972 if (layerForScrolling() && layerForScrolling()->platformLayer()) {
4973 String result( 4973 String result(
4974 MainThreadScrollingReason::mainThreadScrollingReasonsAsText( 4974 MainThreadScrollingReason::mainThreadScrollingReasonsAsText(
4975 layerForScrolling()->platformLayer()->mainThreadScrollingReasons()) 4975 layerForScrolling()->platformLayer()->mainThreadScrollingReasons())
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
5068 std::unique_ptr<CompositorAnimationTimeline> timeline) { 5068 std::unique_ptr<CompositorAnimationTimeline> timeline) {
5069 m_animationTimeline = std::move(timeline); 5069 m_animationTimeline = std::move(timeline);
5070 } 5070 }
5071 5071
5072 void FrameView::setAnimationHost( 5072 void FrameView::setAnimationHost(
5073 std::unique_ptr<CompositorAnimationHost> host) { 5073 std::unique_ptr<CompositorAnimationHost> host) {
5074 m_animationHost = std::move(host); 5074 m_animationHost = std::move(host);
5075 } 5075 }
5076 5076
5077 } // namespace blink 5077 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698