Call media query change listeners asynchronously.

Source/core/css/MediaQueryMatcher.cpp

 /*
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  *
  *  This library is free software; you can redistribute it and/or
  *  modify it under the terms of the GNU Library General Public
  *  License as published by the Free Software Foundation; either
  *  version 2 of the License, or (at your option) any later version.
  *
  *  This library is distributed in the hope that it will be useful,
  *  but WITHOUT ANY WARRANTY; without even the implied warranty of
  *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  *  Library General Public License for more details.
  *
  *  You should have received a copy of the GNU Library General Public License
  *  along with this library; see the file COPYING.LIB.  If not, write to
  *  the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  *  Boston, MA 02110-1301, USA.
  */
 
 #include "config.h"
 #include "core/css/MediaQueryMatcher.h"
 
 #include "core/css/MediaList.h"
 #include "core/css/MediaQueryEvaluator.h"
 #include "core/css/MediaQueryList.h"
 #include "core/css/MediaQueryListListener.h"
 #include "core/css/resolver/StyleResolver.h"
 #include "core/dom/Document.h"
+#include "core/dom/ScriptedAnimationController.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
 
 namespace WebCore {
 
 MediaQueryMatcher::MediaQueryMatcher(Document& document)
     : m_document(&document)
 {
     ASSERT(m_document);
 }
@@ skipping 59 matching lines @@
     if (!m_document)
         return;
     m_mediaLists.remove(query);
 }
 
 void MediaQueryMatcher::mediaFeaturesChanged()
 {
     if (!m_document)
         return;
 
-    WillBeHeapVector<RefPtrWillBeMember<MediaQueryListListener> > listenersToNotify;
+    WillBeHeapVector<RefPtrWillBeMember<MediaQueryList::Listener> > listenersToNotify;
     for (MediaQueryListSet::iterator it = m_mediaLists.begin(); it != m_mediaLists.end(); ++it)
         (*it)->mediaFeaturesChanged(&listenersToNotify);
 
-    // FIXME: This should be async! We're running script inside ::layout() or ::updateRenderTree().
     for (size_t i = 0; i < listenersToNotify.size(); ++i)
-        listenersToNotify[i]->call();
+        m_delayedListeners.append(listenersToNotify[i]);
+
+    m_document->ensureScriptedAnimationController().scheduleAnimationIfNeeded();

[esprehn, 2014/07/01 00:01:06]

scheduleAnimationIfNeeded() should be private, I think you want
scheduleVisualUpdate().

[cbiesinger, 2014/07/01 00:12:09]

On PageAnimator? Hm... I'm not positive but I don't believe that will call
ensureScriptedAnimationController(), but that does need to be called.

+}
+
+void MediaQueryMatcher::notifyDelayedListeners()
+{
+    WillBeHeapVector<RefPtrWillBeMember<MediaQueryList::Listener> > listeners;
+    listeners.swap(m_delayedListeners);
+
+    for (size_t i = 0; i < listeners.size(); ++i)
+        listeners[i]->mediaQueryChanged();
 }
 
 void MediaQueryMatcher::trace(Visitor* visitor)
 {
     visitor->trace(m_document);
+    visitor->trace(m_delayedListeners);
     visitor->trace(m_mediaLists);
 }
 
 }