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

Issue 140503005: Mojo: foo_[0-9] -> foo[0-9]. (Closed)

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

Description

Mojo: foo_[0-9] -> foo[0-9]. For foolish consistency. I'll never make this mistake again. R=sky@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245573

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+471 lines, -474 lines) Patch
M mojo/public/system/core_cpp.h View 1 chunk +9 lines, -9 lines 0 comments Download
M mojo/public/tests/system/core_cpp_unittest.cc View 9 chunks +134 lines, -135 lines 0 comments Download
M mojo/public/tests/system/core_perftest.cc View 5 chunks +14 lines, -14 lines 0 comments Download
M mojo/public/tests/system/core_unittest.cc View 2 chunks +23 lines, -24 lines 0 comments Download
M mojo/system/core_impl.cc View 2 chunks +6 lines, -8 lines 0 comments Download
M mojo/system/core_impl_unittest.cc View 2 chunks +6 lines, -6 lines 0 comments Download
M mojo/system/message_pipe.h View 1 chunk +2 lines, -2 lines 0 comments Download
M mojo/system/message_pipe.cc View 1 chunk +4 lines, -4 lines 0 comments Download
M mojo/system/message_pipe_dispatcher_unittest.cc View 9 chunks +119 lines, -119 lines 0 comments Download
M mojo/system/remote_message_pipe_posix_unittest.cc View 10 chunks +105 lines, -104 lines 0 comments Download
M mojo/system/waiter_list_unittest.cc View 3 chunks +49 lines, -49 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
viettrungluu
6 years, 11 months ago (2014-01-17 18:25:55 UTC) #1
sky
LGTM (your description is wrong).
6 years, 11 months ago (2014-01-17 18:28:44 UTC) #2
viettrungluu
6 years, 11 months ago (2014-01-17 18:57:31 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r245573 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698