base: Do not allow MessagePumpObservers to consume events.

ui/display/chromeos/x11/native_display_delegate_x11.cc

 // Copyright 2014 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 "ui/display/chromeos/x11/native_display_delegate_x11.h"
 
 #include <X11/Xatom.h>
 #include <X11/Xlib.h>
 #include <X11/extensions/dpms.h>
 #include <X11/extensions/Xrandr.h>
@@ skipping 93 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 // NativeDisplayDelegateX11::MessagePumpObserverX11
 
 class NativeDisplayDelegateX11::MessagePumpObserverX11
     : public base::MessagePumpObserver {
  public:
   MessagePumpObserverX11(HelperDelegate* delegate);
   virtual ~MessagePumpObserverX11();
 
   // base::MessagePumpObserver overrides:
-  virtual base::EventStatus WillProcessEvent(const base::NativeEvent& event)
-      OVERRIDE;
+  virtual void WillProcessEvent(const base::NativeEvent& event) OVERRIDE;
   virtual void DidProcessEvent(const base::NativeEvent& event) OVERRIDE;
 
  private:
   HelperDelegate* delegate_;  // Not owned.
 
   DISALLOW_COPY_AND_ASSIGN(MessagePumpObserverX11);
 };
 
 NativeDisplayDelegateX11::MessagePumpObserverX11::MessagePumpObserverX11(
     HelperDelegate* delegate)
     : delegate_(delegate) {}
 
 NativeDisplayDelegateX11::MessagePumpObserverX11::~MessagePumpObserverX11() {}
 
-base::EventStatus
-NativeDisplayDelegateX11::MessagePumpObserverX11::WillProcessEvent(
+void NativeDisplayDelegateX11::MessagePumpObserverX11::WillProcessEvent(
     const base::NativeEvent& event) {
   // XI_HierarchyChanged events are special. There is no window associated with
   // these events. So process them directly from here.
   if (event->type == GenericEvent &&
       event->xgeneric.evtype == XI_HierarchyChanged) {
     VLOG(1) << "Received XI_HierarchyChanged event";
     // Defer configuring outputs to not stall event processing.
     // This also takes care of same event being received twice.
     delegate_->NotifyDisplayObservers();
   }
-
-  return base::EVENT_CONTINUE;
 }
 
 void NativeDisplayDelegateX11::MessagePumpObserverX11::DidProcessEvent(
     const base::NativeEvent& event) {}
 
 ////////////////////////////////////////////////////////////////////////////////
 // NativeDisplayDelegateX11 implementation:
 
 NativeDisplayDelegateX11::NativeDisplayDelegateX11()
     : display_(base::MessagePumpX11::GetDefaultXDisplay()),
@@ skipping 499 matching lines @@
 
 void NativeDisplayDelegateX11::AddObserver(NativeDisplayObserver* observer) {
   observers_.AddObserver(observer);
 }
 
 void NativeDisplayDelegateX11::RemoveObserver(NativeDisplayObserver* observer) {
   observers_.RemoveObserver(observer);
 }
 
 }  // namespace ui