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

Issue 752243002: Add the child_port Mach subsystem (Closed)

Created:
6 years ago by Mark Mentovai
Modified:
6 years ago
Reviewers:
Robert Sesek
CC:
crashpad-dev_chromium.org
Base URL:
https://chromium.googlesource.com/crashpad/crashpad@master
Project:
crashpad
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+111 lines, -0 lines) Patch
A util/mach/child_port.defs View 1 chunk +64 lines, -0 lines 3 comments Download
A util/mach/child_port_types.h View 1 chunk +26 lines, -0 lines 0 comments Download
M util/util.gyp View 2 chunks +21 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (1 generated)
Mark Mentovai
6 years ago (2014-11-24 20:06:52 UTC) #2
Robert Sesek
https://codereview.chromium.org/752243002/diff/1/util/mach/child_port.defs File util/mach/child_port.defs (right): https://codereview.chromium.org/752243002/diff/1/util/mach/child_port.defs#newcode36 util/mach/child_port.defs:36: import "util/mach/child_port_types.h"; Do you need to define the Mach ...
6 years ago (2014-11-24 20:33:04 UTC) #3
Mark Mentovai
https://codereview.chromium.org/752243002/diff/1/util/mach/child_port.defs File util/mach/child_port.defs (right): https://codereview.chromium.org/752243002/diff/1/util/mach/child_port.defs#newcode36 util/mach/child_port.defs:36: import "util/mach/child_port_types.h"; Robert Sesek wrote: > Do you need ...
6 years ago (2014-11-24 20:42:28 UTC) #4
Robert Sesek
LGTM https://codereview.chromium.org/752243002/diff/1/util/mach/child_port.defs File util/mach/child_port.defs (right): https://codereview.chromium.org/752243002/diff/1/util/mach/child_port.defs#newcode36 util/mach/child_port.defs:36: import "util/mach/child_port_types.h"; On 2014/11/24 20:42:28, Mark Mentovai wrote: ...
6 years ago (2014-11-24 20:45:12 UTC) #5
Mark Mentovai
More MIG annoyances: in an earlier version of this change, the token was a uint64_t[4], ...
6 years ago (2014-11-24 20:47:55 UTC) #6
Mark Mentovai
6 years ago (2014-11-24 20:48:14 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
e9482a704d131827ce84f09f7df2f42978018160 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698