base: Do not allow MessagePumpObservers to consume events.

ui/base/clipboard/clipboard_aurax11.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 "ui/base/clipboard/clipboard.h"
 
 #include <X11/extensions/Xfixes.h>
 #include <X11/Xatom.h>
 #include <list>
 #include <set>
@@ skipping 55 matching lines @@
   }
   uint64 primary_sequence_number() const { return primary_sequence_number_; }
 
  private:
   friend struct DefaultSingletonTraits<SelectionChangeObserver>;
 
   SelectionChangeObserver();
   virtual ~SelectionChangeObserver();
 
   // Overridden from base::MessagePumpObserver:
-  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 {}
 
   int event_base_;
   Atom clipboard_atom_;
   uint64 clipboard_sequence_number_;
   uint64 primary_sequence_number_;
 
   DISALLOW_COPY_AND_ASSIGN(SelectionChangeObserver);
 };
@@ skipping 25 matching lines @@
 }
 
 SelectionChangeObserver::~SelectionChangeObserver() {
   // We are a singleton; we will outlive our message pump.
 }
 
 SelectionChangeObserver* SelectionChangeObserver::GetInstance() {
   return Singleton<SelectionChangeObserver>::get();
 }
 
-base::EventStatus SelectionChangeObserver::WillProcessEvent(
-    const base::NativeEvent& event) {
+void SelectionChangeObserver::WillProcessEvent(const base::NativeEvent& event) {
   if (event->type == event_base_ + XFixesSelectionNotify) {
     XFixesSelectionNotifyEvent* ev =
         reinterpret_cast<XFixesSelectionNotifyEvent*>(event);
     if (ev->selection == clipboard_atom_) {
       clipboard_sequence_number_++;
     } else if (ev->selection == XA_PRIMARY) {
       primary_sequence_number_++;
     } else {
       DLOG(ERROR) << "Unexpected selection atom: " << ev->selection;
     }
   }
-  return base::EVENT_CONTINUE;
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 
 // Represents a list of possible return types. Copy constructable.
 class TargetList {
  public:
   typedef std::vector< ::Atom> AtomVector;
 
   TargetList(const AtomVector& target_list, X11AtomCache* atom_cache);
@@ skipping 722 matching lines @@
   return type;
 }
 
 // static
 const Clipboard::FormatType& Clipboard::GetPepperCustomDataFormatType() {
   CR_DEFINE_STATIC_LOCAL(FormatType, type, (kMimeTypePepperCustomData));
   return type;
 }
 
 }  // namespace ui