Explicitly invalidate LayoutView when overflow is recalculated.

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

Index: third_party/WebKit/Source/core/frame/FrameView.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameView.cpp b/third_party/WebKit/Source/core/frame/FrameView.cpp
index 9aa2afd891f4c5db3153c129e59e86973d43ca32..94128d6e581268a02719b24756c79c75b8550d2e 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -815,6 +815,8 @@ void FrameView::recalcOverflowAfterStyleChange() {
   if (needsLayout())
     return;
 
+  layoutViewItem.setMayNeedPaintInvalidation();

[pdr., 2017/02/10 22:25:33]

Yeah I think this patch looks like the bug. We should only need to do this when
the visual viewport supplies the scrollbars though, so WDYT of something like:
if (visualViewportSuppliesScrollbars()) {
  layoutViewItem.setMayNeedPaintInvalidation();
} else {
  // do the regular computeScrollbarExistence code here.
}

If you can launch a debug build with "--enable-viewport
--enable-blink-features=PaintUnderInvalidationChecking" and run the demo, you
can confirm this is under-invalidation (missing call to
setMayNeedPaintInvalidation). This is significantly less useful _after_ you've
spent a day locating the bug but maybe it can be of use in the future :)

Idea for testing: you can add a test under paint/invalidation and ensure the
right invalidation rect is output.

[bokan, 2017/02/21 22:14:14]

Yah, I agree we can avoid the scrollbar updates below if
visualViewportSuppliesScrollbars. However, given that this needs to get merged
to M57, I'm not keen to change any more behavior than necessary.  How about I
land a version like this:

if (visualViewportSuppliesScrollbars())
  layoutViewItem.setMayNeedPaintInvalidation()

And put up a followup patch that conditionally does the scrollbar stuff below?
 
Neat, thanks for the tip!
 
Thanks, will do.

+
   // TODO(pdr): This should be refactored to just block scrollbar updates as
   // we are not in a scrollbar update here and m_inUpdateScrollbars has other
   // side effects. This scope is only for preventing a synchronous layout from