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

Issue 19525007: Update include paths in ipc for base/process changes. (Closed)

Created:
7 years, 5 months ago by Robert Sesek
Modified:
7 years, 5 months ago
Reviewers:
cpu_(ooo_6.6-7.5)
CC:
chromium-reviews, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Update include paths in ipc for base/process changes. BUG=242290 R=cpu@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=213349

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -12 lines) Patch
M ipc/ipc_channel.h View 1 chunk +1 line, -1 line 0 comments Download
M ipc/ipc_channel.cc View 1 chunk +0 lines, -1 line 0 comments Download
M ipc/ipc_channel_nacl.h View 1 chunk +1 line, -1 line 0 comments Download
M ipc/ipc_channel_nacl.cc View 1 chunk +0 lines, -1 line 0 comments Download
M ipc/ipc_channel_posix.h View 1 chunk +1 line, -1 line 0 comments Download
M ipc/ipc_channel_posix.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M ipc/ipc_channel_posix_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M ipc/ipc_channel_win.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M ipc/ipc_logging.cc View 1 chunk +0 lines, -1 line 0 comments Download
M ipc/ipc_platform_file.h View 1 chunk +1 line, -1 line 0 comments Download
M ipc/ipc_sync_channel_unittest.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M ipc/ipc_test_base.h View 1 chunk +1 line, -1 line 0 comments Download
M ipc/ipc_test_base.cc View 1 chunk +1 line, -0 lines 0 comments Download
M ipc/sync_socket_unittest.cc View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
Robert Sesek
7 years, 5 months ago (2013-07-22 21:39:11 UTC) #1
cpu_(ooo_6.6-7.5)
lgtm I was expecting to also see the changes to the existing headers like process_util.h, ...
7 years, 5 months ago (2013-07-24 02:09:41 UTC) #2
Robert Sesek
On 2013/07/24 02:09:41, cpu wrote: > lgtm > > I was expecting to also see ...
7 years, 5 months ago (2013-07-24 03:12:33 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rsesek@chromium.org/19525007/2001
7 years, 5 months ago (2013-07-24 03:13:58 UTC) #4
commit-bot: I haz the power
7 years, 5 months ago (2013-07-24 05:15:25 UTC) #5
Message was sent while issue was closed.
Change committed as 213349

Powered by Google App Engine
This is Rietveld 408576698