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

Unified Diff: Source/core/layout/LayoutThemeTest.cpp

Issue 1126883002: Change all one-off lifecycle callers to FrameView::updateLayoutAndStyleForPainting (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/layout/LayoutTestHelper.cpp ('k') | Source/core/layout/LayoutView.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/layout/LayoutThemeTest.cpp
diff --git a/Source/core/layout/LayoutThemeTest.cpp b/Source/core/layout/LayoutThemeTest.cpp
index 112bd255e488f6219538923e6757047105a06204..826b449da02f3afdd9ee9e1fdfe72ab3e9240b90 100644
--- a/Source/core/layout/LayoutThemeTest.cpp
+++ b/Source/core/layout/LayoutThemeTest.cpp
@@ -42,7 +42,7 @@ void LayoutThemeTest::SetUp()
void LayoutThemeTest::setHtmlInnerHTML(const char* htmlContent)
{
document().documentElement()->setInnerHTML(String::fromUTF8(htmlContent), ASSERT_NO_EXCEPTION);
- document().view()->updateLayoutAndStyleIfNeededRecursive();
+ document().view()->updateLayoutAndStyleForPainting();
}
inline Color outlineColor(Element* element)
@@ -73,7 +73,7 @@ TEST_F(LayoutThemeTest, ChangeFocusRingColor)
document().page()->focusController().setActive(true);
document().page()->focusController().setFocused(true);
span->focus();
- document().view()->updateLayoutAndStyleIfNeededRecursive();
+ document().view()->updateLayoutAndStyleForPainting();
// Checking focused style.
EXPECT_NE(BNONE, outlineStyle(span));
@@ -82,7 +82,7 @@ TEST_F(LayoutThemeTest, ChangeFocusRingColor)
// Change focus ring color.
LayoutTheme::theme().setCustomFocusRingColor(customColor);
Page::platformColorsChanged();
- document().view()->updateLayoutAndStyleIfNeededRecursive();
+ document().view()->updateLayoutAndStyleForPainting();
// Check that the focus ring color is updated.
EXPECT_NE(BNONE, outlineStyle(span));
« no previous file with comments | « Source/core/layout/LayoutTestHelper.cpp ('k') | Source/core/layout/LayoutView.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698