Create custom scrollbars when html element has overflow:scroll

Source/core/paint/DeprecatedPaintLayerScrollableArea.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  *
  * Portions are Copyright (C) 1998 Netscape Communications Corporation.
  *
  * Other contributors:
  *   Robert O'Callahan <roc+@cs.cmu.edu>
  *   David Baron <dbaron@fas.harvard.edu>
  *   Christian Biesinger <cbiesinger@web.de>
  *   Randall Jesup <rjesup@wgate.com>
@@ skipping 628 matching lines @@
     if (newScrollOffset != adjustedScrollOffset()) {
         DoublePoint origin(scrollOrigin());
         ScrollableArea::setScrollPosition(-origin + newScrollOffset, ProgrammaticScroll, scrollBehavior);
     }
 }
 
 void DeprecatedPaintLayerScrollableArea::updateAfterLayout()
 {
     ASSERT(box().hasOverflowClip());
 
+    if (needsScrollbarReconstruction()) {
+        if (m_hBar)
+            destroyScrollbar(HorizontalScrollbar);
+        if (m_vBar)
+            destroyScrollbar(VerticalScrollbar);
+    }
+
     DoubleSize originalScrollOffset = adjustedScrollOffset();
     computeScrollDimensions();
 
     // Layout may cause us to be at an invalid scroll position. In this case we need
     // to pull our scroll offsets back to the max (or push them up to the min).
     DoubleSize clampedScrollOffset = clampScrollOffset(adjustedScrollOffset());
     if (clampedScrollOffset != adjustedScrollOffset())
         scrollToOffset(clampedScrollOffset);
 
     if (originalScrollOffset != adjustedScrollOffset()) {
@@ skipping 13 matching lines @@
             horizontalScrollbar()->setEnabled(hasHorizontalOverflow);
         if (box().style()->overflowY() == OSCROLL && verticalScrollbar())
             verticalScrollbar()->setEnabled(hasVerticalOverflow);
     }
     if (hasOverlayScrollbars()) {
         if (!scrollSize(HorizontalScrollbar))
             setHasHorizontalScrollbar(false);
         if (!scrollSize(VerticalScrollbar))
             setHasVerticalScrollbar(false);
     }
-    // overflow:auto may need to lay out again if scrollbars got added/removed.
-    bool autoHorizontalScrollBarChanged = box().hasAutoHorizontalScrollbar() && (hasHorizontalScrollbar() != hasHorizontalOverflow);
-    bool autoVerticalScrollBarChanged = box().hasAutoVerticalScrollbar() && (hasVerticalScrollbar() != hasVerticalOverflow);
-
-    if (!visualViewportSuppliesScrollbars() && (autoHorizontalScrollBarChanged || autoVerticalScrollBarChanged)) {
-        if (box().hasAutoHorizontalScrollbar())
+    // overflow:auto may need to lay out again if scrollbars got added/removed and even on reconstruction of scrollbars
+    // from native to custom or vice versa also needs to layout again.
+    bool horizontalScrollBarChanged = (box().hasAutoHorizontalScrollbar() && (hasHorizontalScrollbar() != hasHorizontalOverflow)) || (box().style()->overflowX() == OSCROLL && !horizontalScrollbar());
+    bool verticalScrollBarChanged = (box().hasAutoVerticalScrollbar() && (hasVerticalScrollbar() != hasVerticalOverflow)) || (box().style()->overflowY() == OSCROLL && !verticalScrollbar());
+    if (!visualViewportSuppliesScrollbars() && (horizontalScrollBarChanged || verticalScrollBarChanged)) {
+        if (box().hasAutoHorizontalScrollbar() || box().style()->overflowX() == OSCROLL)

[MuVen, 2015/08/18 13:20:01]

in case of overflow-x:scroll and overflow-y:auto, for ex
LayoutTests/scrollbars/disabled-scrollbar.html
overflow logic during styling has calculated horizontal scrollbar, but as not
overflow of contents, thumb
is not shown, just painted the track. 

Think that verticalScrollbarChanged, 
horizontalScrollBarChanged = 0 verticalScrollBarChanged = 1

but the below condition will be executed, as overflow-x:scroll and doesnt has
horizontalOverflow. This disables
the scrollbar. I think its better we add && !horizontalScrollbar() check.
           
		   if (box().hasAutoHorizontalScrollbar() ||box().style()->overflowX() ==
OSCROLL)  {
            setHasHorizontalScrollbar(hasHorizontalOverflow);

             setHasHorizontalScrollbar(hasHorizontalOverflow);
-        if (box().hasAutoVerticalScrollbar())
+        if (box().hasAutoVerticalScrollbar() || box().style()->overflowY() == OSCROLL)
             setHasVerticalScrollbar(hasVerticalOverflow);
 
         if (hasVerticalOverflow || hasHorizontalOverflow)
             updateScrollCornerStyle();
 
         layer()->updateSelfPaintingLayer();
 
         // Force an update since we know the scrollbars have changed things.
         if (box().document().hasAnnotatedRegions())
             box().document().setAnnotatedRegionsDirty(true);
 
-        if (box().style()->overflowX() == OAUTO || box().style()->overflowY() == OAUTO) {
-            if (!m_inOverflowRelayout) {
-                // Our proprietary overflow: overlay value doesn't trigger a layout.
-                m_inOverflowRelayout = true;
-                SubtreeLayoutScope layoutScope(box());
-                layoutScope.setNeedsLayout(&box(), LayoutInvalidationReason::ScrollbarChanged);
-                if (box().isLayoutBlock()) {
-                    LayoutBlock& block = toLayoutBlock(box());
-                    block.scrollbarsChanged(autoHorizontalScrollBarChanged, autoVerticalScrollBarChanged);
-                    block.layoutBlock(true);
-                } else {
-                    box().layout();
-                }
-                m_inOverflowRelayout = false;
+        if (!m_inOverflowRelayout) {
+            // Our proprietary overflow: overlay value doesn't trigger a layout.
+            m_inOverflowRelayout = true;
+            SubtreeLayoutScope layoutScope(box());
+            layoutScope.setNeedsLayout(&box(), LayoutInvalidationReason::ScrollbarChanged);
+            if (box().isLayoutBlock()) {
+                LayoutBlock& block = toLayoutBlock(box());
+                block.scrollbarsChanged(horizontalScrollBarChanged, verticalScrollBarChanged);
+                block.layoutBlock(true);
+            } else {
+                box().layout();
             }
+            m_inOverflowRelayout = false;
         }
     }
 
     {
         // Hits in compositing/overflow/automatically-opt-into-composited-scrolling-after-style-change.html.
         DisableCompositingQueryAsserts disabler;
 
         // Set up the range (and page step/line step).
         if (Scrollbar* horizontalScrollbar = this->horizontalScrollbar()) {
             int clientWidth = box().pixelSnappedClientWidth();
@@ skipping 225 matching lines @@
         }
         if (ShadowRoot* shadowRoot = node->containingShadowRoot()) {
             if (shadowRoot->type() == ShadowRootType::UserAgent)
                 return shadowRoot->host()->layoutObject();
         }
     }
 
     return &layoutObject;
 }
 
+bool DeprecatedPaintLayerScrollableArea::needsScrollbarReconstruction() const
+{
+    LayoutObject* actualLayoutObject = layoutObjectForScrollbar(box());
+    bool shouldUseCustom = actualLayoutObject->isBox() && actualLayoutObject->style()->hasPseudoStyle(SCROLLBAR);
+    bool hasAnyScrollbar = hasScrollbar();
+    bool hasCustom = (m_hBar && m_hBar->isCustomScrollbar()) || (m_vBar && m_vBar->isCustomScrollbar());
+    return hasAnyScrollbar && (shouldUseCustom != hasCustom);
+}
+
 PassRefPtrWillBeRawPtr<Scrollbar> DeprecatedPaintLayerScrollableArea::createScrollbar(ScrollbarOrientation orientation)
 {
     RefPtrWillBeRawPtr<Scrollbar> widget = nullptr;
     LayoutObject* actualLayoutObject = layoutObjectForScrollbar(box());
     bool hasCustomScrollbarStyle = actualLayoutObject->isBox() && actualLayoutObject->style()->hasPseudoStyle(SCROLLBAR);
     if (hasCustomScrollbarStyle) {
         widget = LayoutScrollbar::createCustomScrollbar(this, orientation, actualLayoutObject->node());
     } else {
         ScrollbarControlSize scrollbarSize = RegularScrollbar;
         if (actualLayoutObject->style()->hasAppearance())
@@ skipping 480 matching lines @@
         return false;
 
     LocalFrame* frame = box().frame();
     if (!frame || !frame->isMainFrame() || !frame->settings())
         return false;
 
     return frame->settings()->viewportMetaEnabled();
 }
 
 } // namespace blink