Changing animate to beginFrame and use struct rather than naked double to allow extension.

Source/web/WebPagePopupImpl.cpp

Index: Source/web/WebPagePopupImpl.cpp
diff --git a/Source/web/WebPagePopupImpl.cpp b/Source/web/WebPagePopupImpl.cpp
index e03b673c6b831fd25b73511e2f98aab0546c9905..f77a7f3808819714c6a2dc34746b02b01827c7b9 100644
--- a/Source/web/WebPagePopupImpl.cpp
+++ b/Source/web/WebPagePopupImpl.cpp
@@ -280,7 +280,7 @@ WebSize WebPagePopupImpl::size()
     return m_popupClient->contentSize();
 }
 
-void WebPagePopupImpl::animate(double)
+void WebPagePopupImpl::beginFrame(const WebBeginFrameArgs& frameTime)
 {
     PageWidgetDelegate::animate(m_page.get(), monotonicallyIncreasingTime());

[jamesr, 2014/07/17 19:58:16]

do you have a particular reason to not use the parameter for these types of
widgets?

[mithro-old, 2014/07/18 00:21:04]

This method *should* use the passed in parameter. 

Making it do so would make this patch have actually functional changes and sadly
it causes a bunch of tests to fail. (You can see it in previous patch sets.)

I've move that change into a separate patch which also fixes the tests.

[jamesr, 2014/07/18 00:30:15]

Then leave a FIXME, ideally referencing the bug you filed that described the
test failures so the next patch will have proper context

 }