Notify Blink to suppress frame requests during BeginMainFrame

third_party/WebKit/Source/web/WebPagePopupImpl.cpp

 /*
  * Copyright (C) 2012 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 377 matching lines @@
   if (m_layerTreeView) {
     m_layerTreeView->setVisible(true);
     m_animationHost = WTF::makeUnique<CompositorAnimationHost>(
         m_layerTreeView->compositorAnimationHost());
     m_page->layerTreeViewInitialized(*m_layerTreeView, nullptr);
   } else {
     m_animationHost = nullptr;
   }
 }
 
+void WebPagePopupImpl::setSuppressFrameRequestsWorkaroundFor704763Only(
+    bool suppressFrameRequests) {

[esprehn, 2017/04/04 06:13:55]

I think you need to null check the page just like below?

[alancutter (OOO until 2018), 2017/04/04 09:54:31]

Ty, done.

+  m_page->animator().setSuppressFrameRequestsWorkaroundFor704763Only(
+      suppressFrameRequests);
+}
 void WebPagePopupImpl::beginFrame(double lastFrameTimeMonotonic) {
   if (!m_page)
     return;
   // FIXME: This should use lastFrameTimeMonotonic but doing so
   // breaks tests.
   PageWidgetDelegate::animate(*m_page, monotonicallyIncreasingTime());
 }
 
 void WebPagePopupImpl::willCloseLayerTreeView() {
   if (m_page && m_layerTreeView)
@@ skipping 182 matching lines @@
   // A WebPagePopupImpl instance usually has two references.
   //  - One owned by the instance itself. It represents the visible widget.
   //  - One owned by a WebViewImpl. It's released when the WebViewImpl ask the
   //    WebPagePopupImpl to close.
   // We need them because the closing operation is asynchronous and the widget
   // can be closed while the WebViewImpl is unaware of it.
   return adoptRef(new WebPagePopupImpl(client)).leakRef();
 }
 
 }  // namespace blink