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

Side by Side Diff: third_party/WebKit/Source/core/paint/PaintPropertyTreeUpdateTests.cpp

Issue 2804883006: Revert of Remove begin/end subseq. display items, and store on PaintController instead. (Closed)
Patch Set: 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 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 "core/html/HTMLIFrameElement.h" 5 #include "core/html/HTMLIFrameElement.h"
6 #include "core/paint/PaintPropertyTreeBuilderTest.h" 6 #include "core/paint/PaintPropertyTreeBuilderTest.h"
7 #include "core/paint/PaintPropertyTreePrinter.h" 7 #include "core/paint/PaintPropertyTreePrinter.h"
8 8
9 namespace blink { 9 namespace blink {
10 10
(...skipping 128 matching lines...) Expand 10 before | Expand all | Expand 10 after
139 " background-attachment: fixed;" 139 " background-attachment: fixed;"
140 " }" 140 " }"
141 "</style>" 141 "</style>"
142 "<iframe></iframe>" 142 "<iframe></iframe>"
143 "<div id='fixedBackground' class='fixedBackground'></div>" 143 "<div id='fixedBackground' class='fixedBackground'></div>"
144 "<div id='forceScroll' style='height: 8888px;'></div>"); 144 "<div id='forceScroll' style='height: 8888px;'></div>");
145 setChildFrameHTML( 145 setChildFrameHTML(
146 "<style>body { margin: 0; }</style>" 146 "<style>body { margin: 0; }</style>"
147 "<div id='forceScroll' style='height: 8888px;'></div>"); 147 "<div id='forceScroll' style='height: 8888px;'></div>");
148 document().view()->updateAllLifecyclePhases(); 148 document().view()->updateAllLifecyclePhases();
149
149 FrameView* parent = document().view(); 150 FrameView* parent = document().view();
150 EXPECT_TRUE(frameScroll(parent)->hasBackgroundAttachmentFixedDescendants()); 151 EXPECT_TRUE(frameScroll(parent)->hasBackgroundAttachmentFixedDescendants());
151 FrameView* child = childDocument().view(); 152 FrameView* child = childDocument().view();
152 EXPECT_TRUE(frameScroll(child)->hasBackgroundAttachmentFixedDescendants()); 153 EXPECT_TRUE(frameScroll(child)->hasBackgroundAttachmentFixedDescendants());
153 154
154 // Removing a main thread scrolling reason should update the entire tree. 155 // Removing a main thread scrolling reason should update the entire tree.
155 auto* fixedBackground = document().getElementById("fixedBackground"); 156 auto* fixedBackground = document().getElementById("fixedBackground");
156 fixedBackground->removeAttribute(HTMLNames::classAttr); 157 fixedBackground->removeAttribute(HTMLNames::classAttr);
157 document().view()->updateAllLifecyclePhases(); 158 document().view()->updateAllLifecyclePhases();
158 EXPECT_FALSE(frameScroll(parent)->hasBackgroundAttachmentFixedDescendants()); 159 EXPECT_FALSE(frameScroll(parent)->hasBackgroundAttachmentFixedDescendants());
(...skipping 573 matching lines...) Expand 10 before | Expand all | Expand 10 after
732 EXPECT_TRUE(transform->flattensInheritedTransform()); 733 EXPECT_TRUE(transform->flattensInheritedTransform());
733 734
734 document().getElementById("parent")->setAttribute( 735 document().getElementById("parent")->setAttribute(
735 HTMLNames::styleAttr, "transform-style: preserve-3d"); 736 HTMLNames::styleAttr, "transform-style: preserve-3d");
736 document().view()->updateAllLifecyclePhases(); 737 document().view()->updateAllLifecyclePhases();
737 EXPECT_EQ(transform, child->paintProperties()->transform()); 738 EXPECT_EQ(transform, child->paintProperties()->transform());
738 EXPECT_FALSE(transform->flattensInheritedTransform()); 739 EXPECT_FALSE(transform->flattensInheritedTransform());
739 } 740 }
740 741
741 } // namespace blink 742 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698