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

Issue 454603002: Mojo: Plumb new Wait/WaitMany API out to Core. (Closed)

Created:
6 years, 4 months ago by viettrungluu
Modified:
6 years, 4 months ago
CC:
chromium-reviews, qsr+mojo_chromium.org, viettrungluu+watch_chromium.org, yzshen+watch_chromium.org, abarth-chromium, Aaron Boodman, darin (slow to review), ben+mojo_chromium.org
Project:
chromium
Visibility:
Public.

Description

Mojo: Plumb new Wait/WaitMany API out to Core. We still need to expose these to apps, and transition things appropriately. R=darin@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288453

Patch Set 1 #

Patch Set 2 : rebased #

Unified diffs Side-by-side diffs Delta from patch set Stats (+493 lines, -99 lines) Patch
M mojo/system/core.h View 3 chunks +8 lines, -3 lines 0 comments Download
M mojo/system/core.cc View 6 chunks +55 lines, -15 lines 0 comments Download
M mojo/system/core_unittest.cc View 30 chunks +415 lines, -76 lines 0 comments Download
M mojo/system/entrypoints.cc View 2 chunks +12 lines, -5 lines 0 comments Download
M mojo/system/memory.cc View 2 chunks +3 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
viettrungluu
6 years, 4 months ago (2014-08-07 22:50:50 UTC) #1
viettrungluu
ping
6 years, 4 months ago (2014-08-08 19:52:47 UTC) #2
darin (slow to review)
LGTM (sorry for the delay!)
6 years, 4 months ago (2014-08-08 20:02:30 UTC) #3
viettrungluu
6 years, 4 months ago (2014-08-08 22:46:59 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 manually as 288453 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698