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

Issue 8931009: Revert "Refactor MessagePumpX to dispatch events inside of the source Dispatch function." (Closed)

Created:
9 years ago by oshima
Modified:
9 years ago
Reviewers:
piman
CC:
chromium-reviews, sadrul, brettw-cc_chromium.org
Visibility:
Public.

Description

Revert r114168 "Refactor MessagePumpX to dispatch events inside of the source Dispatch function." This reverts commit bae6675d851184f2be0ac117c1a847289c1c7bd9. TBR=piman@chromium.org BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114245

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+85 lines, -15 lines) Patch
M base/base.gypi View 1 chunk +1 line, -0 lines 0 comments Download
M base/message_loop.cc View 1 chunk +10 lines, -1 line 0 comments Download
M base/message_pump_glib.h View 1 chunk +5 lines, -0 lines 0 comments Download
M base/message_pump_glib.cc View 1 chunk +1 line, -1 line 0 comments Download
M base/message_pump_gtk.h View 1 chunk +3 lines, -0 lines 0 comments Download
M base/message_pump_gtk.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M base/message_pump_wayland.h View 2 chunks +21 lines, -1 line 0 comments Download
A base/message_pump_wayland.cc View 1 chunk +24 lines, -0 lines 0 comments Download
M base/message_pump_x.h View 2 chunks +3 lines, -4 lines 0 comments Download
M base/message_pump_x.cc View 3 chunks +12 lines, -8 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
oshima
menu stopped working after this change. I think nested loop is broken.
9 years ago (2011-12-13 19:24:44 UTC) #1
piman
9 years ago (2011-12-13 19:25:04 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698