Only clear tooltip message on a user scroll or compositor scroll.

third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp

Index: third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
diff --git a/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp b/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
index 7a7d3894ed88097c3eff07c36ea5c819f6b11e88..a37cf224342f844afcb63a2dfb4d9c66539a0d8c 100644
--- a/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
+++ b/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
@@ -413,9 +413,10 @@ void PaintLayerScrollableArea::updateScrollOffset(const ScrollOffset& newOffset,
   frame->eventHandler().dispatchFakeMouseMoveEventSoonInQuad(
       quadForFakeMouseMoveEvent);
 
-  Page* page = frame->page();
-  if (page)
-    page->chromeClient().clearToolTip(*frame);
+  if (scrollType == UserScroll || scrollType == CompositorScroll) {
+    if (Page* page = frame->page())

[tdresser, 2017/01/03 13:56:32]

Why move this assignment into the conditional?

[szager1, 2017/01/03 20:24:22]

I have no opinion on this.  I restored the previous syntax.

[bokan, 2017/01/04 23:53:09]

No need to change anything again, just FYI, but I seem to recall Blink style at
one point encouraging this (though I can't find it now) and there's lots of
precedent in existing code. The advantage is that it limits the scope on a
variable to where its used.

+      page->chromeClient().clearToolTip(*frame);
+  }
 
   bool requiresPaintInvalidation = true;