Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1536)

Unified Diff: base/message_loop/message_pump_x11.h

Issue 223483002: base: Do not allow MessagePumpObservers to consume events. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: tot-merge-r262009 Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/message_loop/message_pump_observer.h ('k') | base/message_loop/message_pump_x11.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/message_loop/message_pump_x11.h
diff --git a/base/message_loop/message_pump_x11.h b/base/message_loop/message_pump_x11.h
index 9a61a22559899a7d1fed1e069b6f8ba16b9f2e97..ee20672e1eb3c646c25e6bcc4079b74e4da058fd 100644
--- a/base/message_loop/message_pump_x11.h
+++ b/base/message_loop/message_pump_x11.h
@@ -49,10 +49,8 @@ class BASE_EXPORT MessagePumpX11 : public MessagePumpGlib {
// receiving a notification callback.
void RemoveObserver(MessagePumpObserver* observer);
- // Sends the event to the observers. If an observer returns true, then it does
- // not send the event to any other observers and returns true. Returns false
- // if no observer returns true.
- bool WillProcessXEvent(XEvent* xevent);
+ // Sends the event to the observers.
+ void WillProcessXEvent(XEvent* xevent);
void DidProcessXEvent(XEvent* xevent);
private:
« no previous file with comments | « base/message_loop/message_pump_observer.h ('k') | base/message_loop/message_pump_x11.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698