Switch components/arc/common/net.mojom to use STL types. |
||||||||
Issue descriptionSwitch components/arc/common/net.mojom to use STL types. crbug.com/662510 has more context. Here are CLs that do the same for other mojom files. app.mojom: https://codereview.chromium.org/2503633003/ enterprise_reporting.mojom: https://codereview.chromium.org/2504013003/ video_accelerator.mojom: https://codereview.chromium.org/2505733003/
,
Nov 16 2016
,
Nov 18 2016
As of now, net.mojom is the only interface in arc/ that uses the old binding. abhishekbh@, please remove the arc_bindings_old_types GN target altogether when you migrate the interface.
,
Nov 22 2016
Thanks for working on this! This should be the last piece of the entire codebase. After this I can remove the support for the old mode. :)
,
Nov 27 2016
Was OOO. Will be done this week.
,
Dec 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cc2b33c4b4af32e831b227f81ccd0be0a04c056d commit cc2b33c4b4af32e831b227f81ccd0be0a04c056d Author: abhishekbh <abhishekbh@google.com> Date: Thu Dec 15 01:38:02 2016 arc: enable use_new_wrapper_types for net.mojom BUG= 665983 TEST=try Review-Url: https://codereview.chromium.org/2571143003 Cr-Commit-Position: refs/heads/master@{#438711} [modify] https://crrev.com/cc2b33c4b4af32e831b227f81ccd0be0a04c056d/components/arc/BUILD.gn [modify] https://crrev.com/cc2b33c4b4af32e831b227f81ccd0be0a04c056d/components/arc/net/arc_net_host_impl.cc [modify] https://crrev.com/cc2b33c4b4af32e831b227f81ccd0be0a04c056d/components/arc/net/arc_net_host_impl.h
,
Dec 15 2016
,
Mar 4 2017
,
Apr 17 2017
,
May 30 2017
,
Aug 1 2017
,
Oct 14 2017
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by yusukes@chromium.org
, Nov 16 2016