Mojo C++ bindings: custom type mapping for array - part 2

Mojo C++ bindings: custom type mapping for array - part 2

This CL:
- switches array and map to use the new serialization API.
- gets rid of two layers of indirection: ArraySerializationStrategy and ArraySerializationImpl.
- gets rid of wtf_array_serialization.
- simplifies map serialization by reusing more of the array serialization logic.

BUG=577686
Committed: https://crrev.com/c8c4b08746630b7a551cc685ff6ee425c1943e3b
Cr-Commit-Position: refs/heads/master@{#392159}

[yzshen1, 2016-05-04 18:50:00]

yzshen@chromium.org changed reviewers:
+ rockot@chromium.org

[yzshen1, 2016-05-04 18:50:01]

Hi, Ken.

Would you please take a look? Thanks!

[yzshen1, 2016-05-04 21:34:44]

On 2016/05/04 18:50:01, yzshen1 wrote:
> Hi, Ken.
> 
> Would you please take a look? Thanks!

Please hold off reviewing if you haven't done so.

I need to also convert Map serialization in order to fix the trybot failures, I
think.

[yzshen1, 2016-05-05 20:34:30]

Description was changed from

==========
Mojo C++ bindings: custom type mapping for array - part 2

This CL:
- switches array to use the new serialization API.
- gets rid of two layers of indirection: ArraySerializationStrategy and
ArraySerializationImpl.
- gets rid of wtf_array_serialization.


BUG=577686
==========

to

==========
Mojo C++ bindings: custom type mapping for array - part 2

This CL:
- switches array and map to use the new serialization API.
- gets rid of two layers of indirection: ArraySerializationStrategy and
ArraySerializationImpl.
- gets rid of wtf_array_serialization.
- simplifies map serialization by reusing more of the array serialization logic.

BUG=577686
==========

[yzshen1, 2016-05-05 20:35:00]

On 2016/05/04 21:34:44, yzshen1 wrote:
> On 2016/05/04 18:50:01, yzshen1 wrote:
> > Hi, Ken.
> > 
> > Would you please take a look? Thanks!
> 
> Please hold off reviewing if you haven't done so.
> 
> I need to also convert Map serialization in order to fix the trybot failures,
I
> think.

Hi, Ken.

I think this is ready for review now. Please take a look. Thanks!

[Ken Rockot(use gerrit already), 2016-05-06 20:13:34]

lgtm!

[yzshen1, 2016-05-06 20:23:50]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2016-05-06 20:24:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1953493002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1953493002/120001

[commit-bot: I haz the power, 2016-05-06 21:26:22]

Description was changed from

==========
Mojo C++ bindings: custom type mapping for array - part 2

This CL:
- switches array and map to use the new serialization API.
- gets rid of two layers of indirection: ArraySerializationStrategy and
ArraySerializationImpl.
- gets rid of wtf_array_serialization.
- simplifies map serialization by reusing more of the array serialization logic.

BUG=577686
==========

to

==========
Mojo C++ bindings: custom type mapping for array - part 2

This CL:
- switches array and map to use the new serialization API.
- gets rid of two layers of indirection: ArraySerializationStrategy and
ArraySerializationImpl.
- gets rid of wtf_array_serialization.
- simplifies map serialization by reusing more of the array serialization logic.

BUG=577686
==========

[commit-bot: I haz the power, 2016-05-06 21:26:23]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-05-06 21:29:11]

Description was changed from

==========
Mojo C++ bindings: custom type mapping for array - part 2

This CL:
- switches array and map to use the new serialization API.
- gets rid of two layers of indirection: ArraySerializationStrategy and
ArraySerializationImpl.
- gets rid of wtf_array_serialization.
- simplifies map serialization by reusing more of the array serialization logic.

BUG=577686
==========

to

==========
Mojo C++ bindings: custom type mapping for array - part 2

This CL:
- switches array and map to use the new serialization API.
- gets rid of two layers of indirection: ArraySerializationStrategy and
ArraySerializationImpl.
- gets rid of wtf_array_serialization.
- simplifies map serialization by reusing more of the array serialization logic.

BUG=577686

Committed: https://crrev.com/c8c4b08746630b7a551cc685ff6ee425c1943e3b
Cr-Commit-Position: refs/heads/master@{#392159}
==========

[commit-bot: I haz the power, 2016-05-06 21:29:12]

Patchset 7 (id:??) landed as
https://crrev.com/c8c4b08746630b7a551cc685ff6ee425c1943e3b
Cr-Commit-Position: refs/heads/master@{#392159}