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

Issue 23439010: Remove now defunct MessageBuilder class. (Closed)

Created:
7 years, 3 months ago by darin (slow to review)
Modified:
7 years, 3 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Remove now defunct MessageBuilder class. R=ben@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=224893

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -485 lines) Patch
M mojo/mojo.gyp View 1 1 chunk +0 lines, -31 lines 0 comments Download
D mojo/public/libs/message/README View 1 chunk +0 lines, -40 lines 0 comments Download
D mojo/public/libs/message/message.h View 1 chunk +0 lines, -57 lines 0 comments Download
D mojo/public/libs/message/message.cc View 1 chunk +0 lines, -15 lines 0 comments Download
D mojo/public/libs/message/message_builder.h View 1 chunk +0 lines, -70 lines 0 comments Download
D mojo/public/libs/message/message_builder.cc View 1 chunk +0 lines, -189 lines 0 comments Download
D mojo/public/libs/message/message_unittest.cc View 1 chunk +0 lines, -83 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
darin (slow to review)
7 years, 3 months ago (2013-09-24 04:05:40 UTC) #1
Ben Goodger (Google)
lgtm
7 years, 3 months ago (2013-09-24 04:09:06 UTC) #2
darin (slow to review)
7 years, 3 months ago (2013-09-24 04:36:51 UTC) #3
Message was sent while issue was closed.
Committed patchset #2 manually as r224893 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698