Fire overflowchanged events at raf timing

Source/core/dom/ScriptedAnimationController.cpp

 /*
  * Copyright (C) 2011 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 18 matching lines @@
 #include "core/dom/Document.h"
 #include "core/dom/RequestAnimationFrameCallback.h"
 #include "core/events/Event.h"
 #include "core/frame/DOMWindow.h"
 #include "core/frame/FrameView.h"
 #include "core/inspector/InspectorInstrumentation.h"
 #include "core/loader/DocumentLoader.h"
 
 namespace WebCore {
 
-std::pair<EventTarget*, StringImpl*> scheduledEventTargetKey(const Event* event)
+std::pair<EventTarget*, StringImpl*> eventTargetKey(const Event* event)
 {
     return std::make_pair(event->target(), event->type().impl());
 }
 
 ScriptedAnimationController::ScriptedAnimationController(Document* document)
     : m_document(document)
     , m_nextCallbackId(0)
     , m_suspendCount(0)
 {
 }
@@ skipping 38 matching lines @@
             m_callbacks.remove(i);
             return;
         }
     }
 }
 
 void ScriptedAnimationController::dispatchEvents()
 {
     Vector<RefPtr<Event> > events;
     events.swap(m_eventQueue);
-    m_scheduledEventTargets.clear();
+    m_perFrameEvents.clear();
 
     for (size_t i = 0; i < events.size(); ++i) {
         EventTarget* eventTarget = events[i]->target();
-        // FIXME: we should figure out how to make dispatchEvent properly virtual to avoid this.
+        // FIXME: we should figure out how to make dispatchEvent properly virtual to avoid
+        // special casting window.
+        // FIXME: We should not fire events for nodes that are no longer in the tree.
         if (DOMWindow* window = eventTarget->toDOMWindow())
             window->dispatchEvent(events[i], 0);
         else
             eventTarget->dispatchEvent(events[i]);
     }
 }
 
 void ScriptedAnimationController::executeCallbacks(double monotonicTimeNow)
 {
     // dispatchEvents() runs script which can cause the document to be destroyed.
@@ skipping 38 matching lines @@
         return;
 
     RefPtr<ScriptedAnimationController> protect(this);
 
     dispatchEvents();
     executeCallbacks(monotonicTimeNow);
 
     scheduleAnimationIfNeeded();
 }
 
-void ScriptedAnimationController::scheduleEvent(PassRefPtr<Event> event)
+void ScriptedAnimationController::enqueueEvent(PassRefPtr<Event> event)
 {
-    if (!m_scheduledEventTargets.add(scheduledEventTargetKey(event.get())).isNewEntry)
-        return;
     m_eventQueue.append(event);
     scheduleAnimationIfNeeded();
 }
 
+void ScriptedAnimationController::enqueuePerFrameEvent(PassRefPtr<Event> event)
+{
+    if (!m_perFrameEvents.add(eventTargetKey(event.get())).isNewEntry)
+        return;
+    enqueueEvent(event);
+}
+
 void ScriptedAnimationController::scheduleAnimationIfNeeded()
 {
     if (!m_document)
         return;
 
     if (m_suspendCount)
         return;
 
     if (!m_callbacks.size() && !m_eventQueue.size())
         return;
 
     if (FrameView* frameView = m_document->view())
         frameView->scheduleAnimation();
 }
 
 }