wayland: use message_pump_observer common code

wayland: use message_pump_observer common code

BUG=102903
TEST=compile

Change-Id: I96c63b55a6be4623f8edd202a1642ae6f7a6eb7a

R=mark@chromium.org,oshima@chromium.org,dnicoara@chromium.org


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110771

[Mark Mentovai, 2011-11-18 19:04:29]

LGTM

[oshima, 2011-11-18 19:21:27]

http://codereview.chromium.org/8561018/diff/1/base/message_pump_wayland.cc
File base/message_pump_wayland.cc (right):

http://codereview.chromium.org/8561018/diff/1/base/message_pump_wayland.cc#ne...
base/message_pump_wayland.cc:27: }
MessagePumpObserver::WillProcessEvent is pure virtual so you don't need this.

[Mandeep Singh Baines, 2011-11-18 20:05:22]

Fixed. PTAL.

http://codereview.chromium.org/8561018/diff/1/base/message_pump_wayland.cc
File base/message_pump_wayland.cc (right):

http://codereview.chromium.org/8561018/diff/1/base/message_pump_wayland.cc#ne...
base/message_pump_wayland.cc:27: }
On 2011/11/18 19:21:28, oshima wrote:
> MessagePumpObserver::WillProcessEvent is pure virtual so you don't need this.

Done.

[oshima, 2011-11-18 20:24:12]

LGTM

[commit-bot: I haz the power, 2011-11-18 20:28:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/msb@chromium.org/8561018/1003

[Mark Mentovai, 2011-11-18 21:35:59]

LGTM

(Test)

[commit-bot: I haz the power, 2011-11-18 22:39:30]

Change committed as 110771