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

Issue 1859763002: Fix SyntheticPointerActionParams compile error for ipc fuzzer (Closed)

Created:
4 years, 8 months ago by lanwei
Modified:
4 years, 8 months ago
CC:
chromium-reviews, darin-cc_chromium.org, jam
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Fix SyntheticPointerActionParams compile error for ipc fuzzer IPC Fuzzer redefines IPC_STRUCT_TRAITS_BEGIN, add another friend struct for SyntheticPointerActionParams. BUG=600129, 525187 TBR=tdresser@chromium.org Committed: https://crrev.com/76edb92f5b5568d9b26847c27ec68a95f38e01c8 Cr-Commit-Position: refs/heads/master@{#385072}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M content/common/input/synthetic_pointer_action_params.h View 2 chunks +6 lines, -0 lines 0 comments Download

Messages

Total messages: 20 (10 generated)
Oliver Chang
lgtm
4 years, 8 months ago (2016-04-04 20:58:05 UTC) #5
lanwei
4 years, 8 months ago (2016-04-04 20:59:07 UTC) #6
Tom Sepez
lgtm
4 years, 8 months ago (2016-04-04 22:21:50 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1859763002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1859763002/1
4 years, 8 months ago (2016-04-05 01:03:06 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/164474)
4 years, 8 months ago (2016-04-05 01:16:19 UTC) #11
Oliver Chang
On 2016/04/05 01:16:19, commit-bot: I haz the power wrote: > Try jobs failed on following ...
4 years, 8 months ago (2016-04-05 01:18:06 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1859763002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1859763002/1
4 years, 8 months ago (2016-04-05 01:18:57 UTC) #15
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 8 months ago (2016-04-05 01:24:50 UTC) #17
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/76edb92f5b5568d9b26847c27ec68a95f38e01c8 Cr-Commit-Position: refs/heads/master@{#385072}
4 years, 8 months ago (2016-04-05 01:26:27 UTC) #19
tdresser
4 years, 8 months ago (2016-04-05 11:58:11 UTC) #20
Message was sent while issue was closed.
LGTM!

Powered by Google App Engine
This is Rietveld 408576698