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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutGeometryMapStep.h

Issue 1870663002: Reland main thread position sticky implementation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Only descend into children which have an ancestor overflow layer. Created 4 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/layout/LayoutGeometryMapStep.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutGeometryMapStep.h b/third_party/WebKit/Source/core/layout/LayoutGeometryMapStep.h
index 85a7efb43e22d7f1ee60c61d86e7a808948f87a7..6b4350d37d1527ac6720f1f42a5b8a2c4b37cc19 100644
--- a/third_party/WebKit/Source/core/layout/LayoutGeometryMapStep.h
+++ b/third_party/WebKit/Source/core/layout/LayoutGeometryMapStep.h
@@ -51,6 +51,7 @@ struct LayoutGeometryMapStep {
: m_layoutObject(o.m_layoutObject)
, m_offset(o.m_offset)
, m_offsetForFixedPosition(o.m_offsetForFixedPosition)
+ , m_offsetForStickyPosition(o.m_offsetForStickyPosition)
, m_flags(o.m_flags)
{
ASSERT(!o.m_transform);
@@ -63,7 +64,10 @@ struct LayoutGeometryMapStep {
const LayoutObject* m_layoutObject;
LayoutSize m_offset;
OwnPtr<TransformationMatrix> m_transform; // Includes offset if non-null.
+ // If m_offsetForFixedPosition could only apply to the fixed position steps, we may be able to merge
+ // with m_offsetForStickyPosition and simplify mapping.
LayoutSize m_offsetForFixedPosition;
+ LayoutSize m_offsetForStickyPosition;
GeometryInfoFlags m_flags;
};
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp ('k') | third_party/WebKit/Source/core/layout/LayoutGrid.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698