base: Do not allow MessagePumpObservers to consume events.

base/message_loop/message_pump_x11.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/message_loop/message_pump_x11.h"
 
 #include <glib.h>
 #include <X11/X.h>
 #include <X11/extensions/XInput2.h>
 #include <X11/XKBlib.h>
@@ skipping 48 matching lines @@
 #endif
 
 void MessagePumpX11::AddObserver(MessagePumpObserver* observer) {
   observers_.AddObserver(observer);
 }
 
 void MessagePumpX11::RemoveObserver(MessagePumpObserver* observer) {
   observers_.RemoveObserver(observer);
 }
 
-bool MessagePumpX11::WillProcessXEvent(XEvent* xevent) {
-  if (!observers_.might_have_observers())
-    return false;
-  ObserverListBase<MessagePumpObserver>::Iterator it(observers_);
-  MessagePumpObserver* obs;
-  while ((obs = it.GetNext()) != NULL) {
-    if (obs->WillProcessEvent(xevent))
-      return true;
-  }
-  return false;
+void MessagePumpX11::WillProcessXEvent(XEvent* xevent) {
+  FOR_EACH_OBSERVER(MessagePumpObserver, observers_, WillProcessEvent(xevent));
 }
 
 void MessagePumpX11::DidProcessXEvent(XEvent* xevent) {
   FOR_EACH_OBSERVER(MessagePumpObserver, observers_, DidProcessEvent(xevent));
 }
 
 }  // namespace base