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

Issue 723853003: Rewrite MachMessageServer::Run() (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@mach_message_deadline
Project:
crashpad
Visibility:
Public.

Description

Rewrite MachMessageServer::Run(). This method is now much more straightforward, easy to understand, and maintainable. There are no externally-visible changes. TEST=util_test MachMessageServer.* R=rsesek@chromium.org Committed: https://chromium.googlesource.com/crashpad/crashpad/+/ef0b7cf6d59c5708b6167a42c2b2b024f6b5205a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+159 lines, -187 lines) Patch
M util/mach/mach_message_server.cc View 4 chunks +159 lines, -187 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
Mark Mentovai
This is the final change in a series that improves the design of MachMessageServer. The ...
6 years ago (2014-12-02 22:00:39 UTC) #2
Robert Sesek
LGTM
6 years ago (2014-12-03 20:42:35 UTC) #3
Mark Mentovai
6 years ago (2014-12-03 21:45:52 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
ef0b7cf6d59c5708b6167a42c2b2b024f6b5205a (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698