-
Takeshi Yoshino authored
This reverts commit 77c59a87. Reason for revert: Broke build. See the comment on the original review. Original change's description: > Mojo EDK: Refactor layered message serialization > > Consolidates and simplifies user message serialization within the EDK. > Namely: > > * The ports layer no longer operates under the assumption that all > events exist exclusively in serialized form. This eliminates the > complexity of layered message allocation at the cost of a small amount > of extra copying in some cases. > > * ports::Message is now more aptly named ports::Event, and different > event types are subclasses. > > * Introduces a ports::UserMessage base class for ports embedder > message types. These can be attached to ports::UserMessageEvent > objects. > > * The MessageForTransit and PortsMessage types have been flattened > into a single UserMessageImpl type, a subclass of ports::UserMessage. > > * Nearly all user message serialization and deserialization logic has > been consolidated into UserMessageImpl. A few small pieces remain > isolated in node_controller.cc and will eventually move elsewhere > after additional refactoring. > > All of this work is a precursor to supporting lazy serialization. > > BUG=725321 > > Change-Id: Ib9e66cd0d5b88ac0e6dc898f248c958039078023 > Reviewed-on: https://chromium-review.googlesource.com/516402 > Commit-Queue: Ken Rockot <rockot@chromium.org> > Reviewed-by: Jay Civelli <jcivelli@chromium.org> > Cr-Commit-Position: refs/heads/master@{#475731} TBR=jcivelli@chromium.org,rockot@chromium.org No-Presubmit: true No-Tree-Checks: true No-Try: true BUG=725321 Change-Id: I84f950290e6c45dc830f2bc3177af09ae955f432 Reviewed-on: https://chromium-review.googlesource.com/518742Reviewed-by:
Takeshi Yoshino <tyoshino@chromium.org> Commit-Queue: Takeshi Yoshino <tyoshino@chromium.org> Cr-Commit-Position: refs/heads/master@{#475805}
9a9457b3