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

Issue 13333003: Create a new base/message_loop directory and move the message_loop_proxy. (Closed)

Created:
7 years, 8 months ago by brettw
Modified:
7 years, 8 months ago
CC:
chromium-reviews, erikwright+watch_chromium.org, sadrul
Visibility:
Public.

Description

Create a new base/message_loop directory and move the message_loop_proxy. Eventually we'll move all the message loop and message pump classes into this directory. This leaves a forwarding header to avoid having to update all includes in this patch. BUG= Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=191422

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -632 lines) Patch
M base/base.gyp View 1 chunk +2 lines, -2 lines 0 comments Download
M base/base.gypi View 1 chunk +4 lines, -4 lines 0 comments Download
M base/message_loop.cc View 1 chunk +1 line, -1 line 0 comments Download
A + base/message_loop/message_loop_proxy.h View 2 chunks +3 lines, -3 lines 0 comments Download
A + base/message_loop/message_loop_proxy.cc View 1 chunk +1 line, -1 line 0 comments Download
A + base/message_loop/message_loop_proxy_impl.h View 2 chunks +3 lines, -3 lines 0 comments Download
A + base/message_loop/message_loop_proxy_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
A + base/message_loop/message_loop_proxy_impl_unittest.cc View 4 chunks +21 lines, -18 lines 0 comments Download
A + base/message_loop/message_loop_proxy_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M base/message_loop_proxy.h View 1 chunk +4 lines, -35 lines 0 comments Download
D base/message_loop_proxy.cc View 1 chunk +0 lines, -17 lines 0 comments Download
D base/message_loop_proxy_impl.h View 1 chunk +0 lines, -62 lines 0 comments Download
D base/message_loop_proxy_impl.cc View 1 chunk +0 lines, -92 lines 0 comments Download
D base/message_loop_proxy_impl_unittest.cc View 1 chunk +0 lines, -126 lines 0 comments Download
D base/message_loop_proxy_unittest.cc View 1 chunk +0 lines, -266 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
brettw
7 years, 8 months ago (2013-03-29 18:05:43 UTC) #1
jar (doing other things)
lgtm
7 years, 8 months ago (2013-03-29 19:21:09 UTC) #2
brettw
7 years, 8 months ago (2013-03-29 19:22:54 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r191422 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698