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

Issue 1033693002: Fix building the IPC fuzzer in light of the new Protobuf<->IPC bridge. (Closed)

Created:
5 years, 9 months ago by grt (UTC plus 2)
Modified:
5 years, 9 months ago
CC:
chromium-reviews, Tom Sepez
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Fix building the IPC fuzzer in light of the new Protobuf<->IPC bridge. r321237 introduced a framework for sending protobuf messages over IPC in chrome/common/safe_browsing. r321803 starting putting this new framework to use. This change updates the ICP fuzzer's message_list so that it can handle types that include protobuf messages. BUG=469867 R=mbarbella@chromium.org Committed: https://crrev.com/1e3c4d2d8e72124b7ba48b7f0e7f4b0122e68f1b Cr-Commit-Position: refs/heads/master@{#322007}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M tools/ipc_fuzzer/message_lib/message_lib.gyp View 1 chunk +1 line, -0 lines 0 comments Download
M tools/ipc_fuzzer/message_tools/DEPS View 1 chunk +1 line, -0 lines 0 comments Download
M tools/ipc_fuzzer/message_tools/message_list.cc View 1 chunk +1 line, -0 lines 0 comments Download
M tools/ipc_fuzzer/message_tools/message_tools.gyp View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 8 (2 generated)
grt (UTC plus 2)
5 years, 9 months ago (2015-03-24 14:21:57 UTC) #1
grt (UTC plus 2)
+cc tsepez
5 years, 9 months ago (2015-03-24 14:22:18 UTC) #2
inferno
lgtm
5 years, 9 months ago (2015-03-24 15:51:21 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1033693002/1
5 years, 9 months ago (2015-03-24 15:52:05 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 9 months ago (2015-03-24 15:56:00 UTC) #7
commit-bot: I haz the power
5 years, 9 months ago (2015-03-24 15:57:31 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/1e3c4d2d8e72124b7ba48b7f0e7f4b0122e68f1b
Cr-Commit-Position: refs/heads/master@{#322007}

Powered by Google App Engine
This is Rietveld 408576698