Only create sticky position constraints for constrained sticky position.

third_party/WebKit/Source/core/frame/FrameViewTest.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/frame/FrameView.h"
 
 #include <memory>
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "core/frame/Settings.h"
@@ skipping 154 matching lines @@
 }
 
 TEST_P(FrameViewTest, StyleChangeUpdatesViewportConstrainedObjects) {
   // When using root layer scrolling there is no concept of viewport constrained
   // objects, so skip this test.
   if (RuntimeEnabledFeatures::rootLayerScrollingEnabled())
     return;
 
   document().body()->setInnerHTML(
       "<style>.container { height: 200%; }"
-      "#sticky { position: sticky; top: 0; height: 50px; }</style>"
-      "<div class='container'><div id='sticky'></div></div>");
+      "#sticky1 { position: sticky; top: 0; height: 50px; }"
+      "#sticky2 { position: sticky; height: 50px; }</style>"
+      "<div class='container'>"
+      "  <div id='sticky1'></div>"
+      "  <div id='sticky2'></div>"
+      "</div>");
   document().view()->updateAllLifecyclePhases();
 
-  LayoutBoxModelObject* sticky = toLayoutBoxModelObject(
-      document().getElementById("sticky")->layoutObject());
+  LayoutBoxModelObject* sticky1 = toLayoutBoxModelObject(
+      document().getElementById("sticky1")->layoutObject());
+  LayoutBoxModelObject* sticky2 = toLayoutBoxModelObject(
+      document().getElementById("sticky2")->layoutObject());
 
   EXPECT_TRUE(
-      document().view()->viewportConstrainedObjects()->contains(sticky));
+      document().view()->viewportConstrainedObjects()->contains(sticky1));
+  // #sticky2 is not viewport constrained because it has no anchor edges.
+  EXPECT_FALSE(
+      document().view()->viewportConstrainedObjects()->contains(sticky2));
 
   // Making the element non-sticky should remove it from the set of
   // viewport-constrained objects.
-  document().getElementById("sticky")->setAttribute(HTMLNames::styleAttr,
-                                                    "position: relative");
+  document().getElementById("sticky1")->setAttribute(HTMLNames::styleAttr,
+                                                     "position: relative");
   document().view()->updateAllLifecyclePhases();
 
   EXPECT_FALSE(
-      document().view()->viewportConstrainedObjects()->contains(sticky));
+      document().view()->viewportConstrainedObjects()->contains(sticky1));
 
   // And making it sticky again should put it back in that list.
-  document().getElementById("sticky")->setAttribute(HTMLNames::styleAttr, "");
+  document().getElementById("sticky1")->setAttribute(HTMLNames::styleAttr, "");
   document().view()->updateAllLifecyclePhases();
 
   EXPECT_TRUE(
-      document().view()->viewportConstrainedObjects()->contains(sticky));
+      document().view()->viewportConstrainedObjects()->contains(sticky1));
+
+  // Adding an anchor edge on the non-anchored sticky should add it to the
+  // viewport-constrained objects.
+  document().getElementById("sticky2")->setAttribute(HTMLNames::styleAttr,
+                                                     "top: 0");
+  document().view()->updateAllLifecyclePhases();
+
+  EXPECT_TRUE(
+      document().view()->viewportConstrainedObjects()->contains(sticky2));
+
+  // Removing the anchor edge on a sticky position element should remove it from
+  // the viewport-constrained objects.
+  document().getElementById("sticky2")->setAttribute(HTMLNames::styleAttr, "");
+  document().view()->updateAllLifecyclePhases();
+
+  EXPECT_FALSE(
+      document().view()->viewportConstrainedObjects()->contains(sticky2));
 }
 
 }  // namespace
 }  // namespace blink