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

Unified Diff: sky/engine/core/frame/FrameView.cpp

Issue 945003002: Remove LayoutState. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: improve test Created 5 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 side-by-side diff with in-line comments
Download patch
Index: sky/engine/core/frame/FrameView.cpp
diff --git a/sky/engine/core/frame/FrameView.cpp b/sky/engine/core/frame/FrameView.cpp
index 78824ceb4847cc4f5f8e74ab58f4207e776ce1d0..a6ffeccc30b2c381c4439c562db3fad6fabf7efb 100644
--- a/sky/engine/core/frame/FrameView.cpp
+++ b/sky/engine/core/frame/FrameView.cpp
@@ -231,10 +231,6 @@ void FrameView::performLayout(RenderObject* rootForThisLayout, bool inSubtreeLay
// performLayout is the actual guts of layout().
// FIXME: The 300 other lines in layout() probably belong in other helper functions
// so that a single human could understand what layout() is actually doing.
-
- LayoutState layoutState(*rootForThisLayout);
-
- // FIXME (crbug.com/256657): Do not do two layouts for text autosizing.
rootForThisLayout->layout();
lifecycle().advanceTo(DocumentLifecycle::AfterPerformLayout);

Powered by Google App Engine
This is Rietveld 408576698