Fix the bug that negative outline-offset is covered up by composited

third_party/WebKit/Source/core/layout/compositing/CompositedLayerMappingTest.cpp

 // Copyright 2014 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/layout/compositing/CompositedLayerMapping.h"
 
 #include "core/frame/FrameView.h"
 #include "core/layout/LayoutBoxModelObject.h"
 #include "core/layout/LayoutTestHelper.h"
 #include "core/layout/api/LayoutViewItem.h"
@@ skipping 819 matching lines @@
   ASSERT_TRUE(mapping->scrollingContentsLayer());
   EXPECT_EQ(
       static_cast<GraphicsLayerPaintingPhase>(
           GraphicsLayerPaintOverflowContents |
           GraphicsLayerPaintCompositedScroll | GraphicsLayerPaintForeground),
       mapping->scrollingContentsLayer()->paintingPhase());
   EXPECT_FALSE(mapping->foregroundLayer());
 }
 
 TEST_P(CompositedLayerMappingTest,
+       DecorationLayerCreatedAndDestroyedInCompositedScrolling) {

[flackr, 2016/11/16 21:14:20]

You should probably also test that we don't create a decoration layer if we're
not composited scrolling.

[yigu, 2016/11/17 15:53:15]

Test added.

+  setBodyInnerHTML(
+      "<style>"
+      "#scroller { overflow: scroll; height: 200px; width: 200px; background: "
+      "white local content-box; outline: 1px solid blue;}"
+      "#scrolled { height: 300px; }"
+      "</style>"
+      "<div id=\"parent\">"
+      "  <div id=\"scroller\"><div id=\"scrolled\"></div></div>"
+      "</div>");
+  document().view()->updateAllLifecyclePhases();
+
+  Element* scroller = document().getElementById("scroller");
+  PaintLayer* paintLayer =
+      toLayoutBoxModelObject(scroller->layoutObject())->layer();
+  ASSERT_TRUE(paintLayer);
+
+  CompositedLayerMapping* mapping = paintLayer->compositedLayerMapping();
+  EXPECT_FALSE(mapping->decorationLayer());
+
+  // The decoration layer is created when compositing.
+  scroller->setAttribute(HTMLNames::styleAttr, "outline-offset: -2px;");
+  document().view()->updateAllLifecyclePhases();
+  paintLayer = toLayoutBoxModelObject(scroller->layoutObject())->layer();
+  ASSERT_TRUE(paintLayer);
+
+  mapping = paintLayer->compositedLayerMapping();
+  EXPECT_TRUE(mapping->decorationLayer());
+
+  // The decoration layer is destroyed when not compositing.
+  scroller->removeAttribute(HTMLNames::styleAttr);
+  document().view()->updateAllLifecyclePhases();
+  paintLayer = toLayoutBoxModelObject(scroller->layoutObject())->layer();
+  ASSERT_TRUE(paintLayer);
+
+  mapping = paintLayer->compositedLayerMapping();
+  EXPECT_FALSE(mapping->decorationLayer());
+}

[flackr, 2016/11/16 21:14:20]

nit: Blank line between tests.

[yigu, 2016/11/17 15:53:15]

Done.

+TEST_P(CompositedLayerMappingTest,
        BackgroundPaintedIntoGraphicsLayerIfNotCompositedScrolling) {
   document().frame()->settings()->setPreferCompositingToLCDTextEnabled(true);
   setBodyInnerHTML(
       "<div id='container' style='overflow: scroll; width: 300px; height: "
       "300px; border-radius: 5px; background: white; will-change: transform;'>"
       "    <div style='background-color: blue; width: 2000px; height: "
       "2000px;'></div>"
       "</div>");
 
   PaintLayer* layer =
@@ skipping 151 matching lines @@
     document().view()->updateAllLifecyclePhases();
     ASSERT_EQ(document().documentElement(),
               rootScrollerController.globalRootScroller());
 
     EXPECT_TRUE(mapping3->clippingLayer());
     EXPECT_TRUE(mapping3->clippingLayer()->platformLayer()->masksToBounds());
   }
 }
 
 }  // namespace blink