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

Issue 1755703004: Revert of IPC::ParamTraits for ui::Event (towards ui::Events over mojo IPC) (Closed)

Created:
4 years, 9 months ago by Dmitry Titov
Modified:
4 years, 9 months ago
CC:
chromium-reviews, tdresser+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of IPC::ParamTraits for ui::Event (towards ui::Events over mojo IPC) (patchset #17 id:320001 of https://codereview.chromium.org/1695783002/ ) Reason for revert: It broke compile on: https://build.chromium.org/p/chromium.win/builders/Win8%20GN%20%28dbg%29 BUG=591488 Original issue's description: > Implements and tests IPC::ParamTraits for ui::Event (towards ui::Events over mojo IPC). Support for (scoped) pointers for abstract ui::Event objects implemented using ui::ScopedEvent, and trusting ui::EventType information to instantiate the correct concrete type. > > BUG=584687 > > Committed: https://crrev.com/482ddfefe2602f440146eebf4b56e8e040fb81fa > Cr-Commit-Position: refs/heads/master@{#378821} TBR=rockot@chromium.org,sadrul@chromium.org,kenrb@chromium.org,tsepez@chromium.org,jam@chromium.org,sky@chromium.org,markdittmer@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=584687 Committed: https://crrev.com/f0444bfd6b3c6b43f34da7709debdbc248395ef5 Cr-Commit-Position: refs/heads/master@{#378838}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -1110 lines) Patch
M components/mus/DEPS View 1 chunk +1 line, -1 line 0 comments Download
M components/mus/common/BUILD.gn View 2 chunks +0 lines, -27 lines 0 comments Download
D components/mus/common/event_param_traits.h View 1 chunk +0 lines, -94 lines 0 comments Download
D components/mus/common/event_param_traits.cc View 1 chunk +0 lines, -277 lines 0 comments Download
D components/mus/common/event_param_traits_macros.h View 1 chunk +0 lines, -101 lines 0 comments Download
D components/mus/common/event_param_traits_unittest.cc View 1 chunk +0 lines, -468 lines 0 comments Download
D components/mus/common/mus_ipc_export.h View 1 chunk +0 lines, -29 lines 0 comments Download
M ui/events/event.h View 13 chunks +3 lines, -67 lines 0 comments Download
M ui/events/event.cc View 1 chunk +0 lines, -3 lines 0 comments Download
M ui/events/events.gyp View 2 chunks +1 line, -2 lines 0 comments Download
M ui/events/gesture_event_details.h View 3 chunks +0 lines, -18 lines 0 comments Download
M ui/events/ipc/BUILD.gn View 1 chunk +0 lines, -1 line 0 comments Download
M ui/events/keycodes/dom/dom_key.h View 2 chunks +0 lines, -3 lines 0 comments Download
M ui/gfx/ipc/gfx_param_traits.h View 3 chunks +0 lines, -3 lines 0 comments Download
M ui/gfx/ipc/gfx_param_traits.cc View 3 chunks +0 lines, -16 lines 0 comments Download
M ui/metro_viewer/metro_viewer_messages.h View 2 chunks +19 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Dmitry Titov
Created Revert of IPC::ParamTraits for ui::Event (towards ui::Events over mojo IPC)
4 years, 9 months ago (2016-03-02 21:39:41 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1755703004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1755703004/1
4 years, 9 months ago (2016-03-02 21:40:12 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 9 months ago (2016-03-02 21:41:10 UTC) #4
commit-bot: I haz the power
4 years, 9 months ago (2016-03-02 21:45:04 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/f0444bfd6b3c6b43f34da7709debdbc248395ef5
Cr-Commit-Position: refs/heads/master@{#378838}

Powered by Google App Engine
This is Rietveld 408576698