Main-thread scroll if we have fixed-pos children and overflow:hidden.

Source/core/page/scrolling/ScrollingCoordinator.cpp

 /*
  * Copyright (C) 2011 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 928 matching lines @@
         reasons |= ThreadedScrollingDisabled;
 
     if (!m_page->mainFrame()->isLocalFrame())
         return reasons;
     FrameView* frameView = m_page->deprecatedLocalMainFrame()->view();
     if (!frameView)
         return reasons;
 
     if (frameView->hasSlowRepaintObjects())
         reasons |= HasSlowRepaintObjects;
-    if (frameView->isScrollable() && hasVisibleSlowRepaintViewportConstrainedObjects(frameView))
+    FrameView::ScrollingReasons scrollingReasons = frameView->scrollingReasons();
+    if ((scrollingReasons == FrameView::NonScrollableFromLayout
+        || scrollingReasons == FrameView::Scrollable)
+        && hasVisibleSlowRepaintViewportConstrainedObjects(frameView)) {

[Ian Vollick, 2014/11/05 16:40:06]

I think this could be made clearer with some well named temporaries.

const bool mayBeScrolledByInput = scrollingReasons == FrameView::Scrollable;
const bool mayBeScrolledByScript = scrollingReasons ==
FrameView::NotScrollableExplicitlyDisabled;
const bool canScroll = mayBeScrolledByInput || mayBeScrolledByScript;

if (canScroll && hasBlahBlah(...) { ... }

[awoloszyn, 2014/11/05 19:25:11]

Done.

         reasons |= HasNonLayerViewportConstrainedObjects;
+    }
 
     return reasons;
 }
 
 String ScrollingCoordinator::mainThreadScrollingReasonsAsText(MainThreadScrollingReasons reasons)
 {
     StringBuilder stringBuilder;
 
     if (reasons & ScrollingCoordinator::HasSlowRepaintObjects)
         stringBuilder.appendLiteral("Has slow repaint objects, ");
@@ skipping 21 matching lines @@
     bool frameIsScrollable = frameView && frameView->isScrollable();
     if (frameIsScrollable != m_wasFrameScrollable)
         return true;
 
     if (WebLayer* scrollLayer = frameView ? toWebLayer(frameView->layerForScrolling()) : nullptr)
         return blink::WebSize(frameView->contentsSize()) != scrollLayer->bounds();
     return false;
 }
 
 } // namespace blink