Make the MediaQueryList listener an EventListener

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/MediaQueryListEvent.h"
 #include "core/css/MediaQueryListListener.h"
 #include "core/css/resolver/StyleResolver.h"
 #include "core/dom/Document.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
 
 namespace blink {
 
 PassRefPtrWillBeRawPtr<MediaQueryMatcher> MediaQueryMatcher::create(Document& document)
 {
@@ skipping 63 matching lines @@
         return;
     m_mediaLists.remove(query);
 }
 
 void MediaQueryMatcher::mediaFeaturesChanged()
 {
     if (!m_document)
         return;
 
     WillBeHeapVector<RefPtrWillBeMember<MediaQueryListListener> > listenersToNotify;
-    for (MediaQueryListSet::iterator it = m_mediaLists.begin(); it != m_mediaLists.end(); ++it)
-        (*it)->mediaFeaturesChanged(&listenersToNotify);
+    for (MediaQueryListSet::iterator it = m_mediaLists.begin(); it != m_mediaLists.end(); ++it) {
+        if ((*it)->mediaFeaturesChanged(&listenersToNotify)) {
+            RefPtrWillBeRawPtr<Event> event(MediaQueryListEvent::create((*it)->media(), (*it)->matches()));
+            event->setTarget(*it);
+            m_document->enqueueUniqueAnimationFrameEvent(event);

[haraken, 2014/07/20 05:14:49]

I'm not quite sure of this. I wonder why MediaqueryListEvent wants to use
per-animation-frame events whereas other events are using
not-per-animation-frame events. Elliott would be the best reviewer on this.

[cbiesinger, 2014/07/21 22:07:19]

Yeah, hmm, I'm not sure... come to think of it, with the way I do this, you only
get the information (.matches) of the first event per a frame, so maybe this
should be changed. Elliott, any thoughts?

+        }
+    }
 
     m_document->enqueueMediaQueryChangeListeners(listenersToNotify);
 }
 
 void MediaQueryMatcher::trace(Visitor* visitor)
 {
 #if ENABLE(OILPAN)
     visitor->trace(m_document);
     visitor->trace(m_mediaLists);
 #endif
 }
 
 }