element.scrollIntoView only scrolls the layout viewport

third_party/WebKit/Source/core/layout/LayoutBox.h

Index: third_party/WebKit/Source/core/layout/LayoutBox.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutBox.h b/third_party/WebKit/Source/core/layout/LayoutBox.h
index 61509f857388094fc55b179128c6d19bd2db9971..4790b81f77d36d188fc8356d53aa83af33446f2a 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBox.h
+++ b/third_party/WebKit/Source/core/layout/LayoutBox.h
@@ -425,7 +425,7 @@ public:
 
     void scrollToOffset(const DoubleSize&, ScrollBehavior = ScrollBehaviorInstant);
     void scrollByRecursively(const DoubleSize& delta, ScrollOffsetClamping = ScrollOffsetUnclamped);
-    void scrollRectToVisible(const LayoutRect&, const ScrollAlignment& alignX, const ScrollAlignment& alignY, ScrollType = ProgrammaticScroll);
+    void scrollRectToVisible(const LayoutRect&, const ScrollAlignment& alignX, const ScrollAlignment& alignY, ScrollType = ProgrammaticScroll, bool scrollVisualViewport = true);

[bokan, 2015/10/30 16:08:55]

I think this param should be called something like makeVisibleInVisualViewport
since scrollVisualViewport could plausibly be interpreted that it'll still
scroll the layout viewport such that the rect is visible in the visual viewport.
Please also add a comment describing what this param will do.

[ymalik, 2015/11/01 00:02:24]

Done.

 
     LayoutRectOutsets marginBoxOutsets() const override { return m_marginBoxOutsets; }
     LayoutUnit marginTop() const override { return m_marginBoxOutsets.top(); }