Fire visualViewport resize event when browser zoomed

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 15cff79b46d15ca13cc20c86a0a8353235f3a9d2..e055f72e2c64f16dd5f29322a5b8ee310f6f8e98 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -2153,6 +2153,9 @@ void FrameView::sendResizeEventIfNeeded()
     m_lastViewportSize = layoutSize(IncludeScrollbars);
     m_lastZoomFactor = layoutViewItem.style()->zoom();
 
+    if (RuntimeEnabledFeatures::visualViewportAPIEnabled())
+        m_frame->document()->enqueueVisualViewportResizeEvent();
+
     m_frame->document()->enqueueResizeEvent();

[ymalik, 2016/08/15 19:12:48]

@dtapuska, this is where we send window resize event for the browser zoom. This
is probably where we should send one for the visualViewport as well. We don't
really resize the visual viewport when we browser zoom. If you look at the API,
we always adjust the values before returning (meaning the viewport hasn't really
changed).
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/frame/Loc...

[bokan, 2016/08/18 21:18:11]

Yah, I'm not sure about this. Perhaps it'd be better to add a
didChangePageZoom(float oldZoomFactor) method on VisualViewport and call that
from LocalFrame::setPageAndTextZoomFactors. That way it can calculate if the
width/height actually changed. That or just directly in
setPageAndTextZoomFactors so you can save and compare the clientWidth/Height
before and after.

 
     if (m_frame->isMainFrame())