cc/blimp: Proto Cleanup.

cc/blimp: Proto Cleanup.

Remove all unused protos and serialization code.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/4762e7cc7d6ff39382c0fa9266d1db0ab0467bf7
Cr-Commit-Position: refs/heads/master@{#431782}

[Khushal, 2016-11-10 19:49:50]

Description was changed from

==========
cc/blimp: Proto Cleanup.

Remove all unused protos and serialization code.

BUG=
==========

to

==========
cc/blimp: Proto Cleanup.

Remove all unused protos and serialization code.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Khushal, 2016-11-10 19:50:28]

khushalsagar@chromium.org changed reviewers:
+ ajuma@chromium.org

[Khushal, 2016-11-10 19:50:29]

And final cleanup patch. Updating the tests on this too.

[Khushal, 2016-11-10 21:56:39]

The CQ bit was checked by khushalsagar@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 21:57:20]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-10 22:23:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-10 22:23:19]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Khushal, 2016-11-11 00:35:45]

khushalsagar@chromium.org changed reviewers:
+ enne@chromium.org

[Khushal, 2016-11-11 00:35:46]

+enne on this one. Since it looks like ajuma@ is OOO.

[Khushal, 2016-11-11 03:44:04]

Description was changed from

==========
cc/blimp: Proto Cleanup.

Remove all unused protos and serialization code.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc/blimp: Proto Cleanup.

Remove all unused protos and serialization code.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[enne (OOO), 2016-11-11 19:24:55]

Yeah, lgtm.  I was a little bit surprised to see property tree proto code go
away, but I guess this is internal to cc at the moment.  We can resurrect it in
the future if we need it.

[Khushal, 2016-11-11 19:52:52]

On 2016/11/11 19:24:55, enne wrote:
> Yeah, lgtm.  I was a little bit surprised to see property tree proto code go
> away, but I guess this is internal to cc at the moment.  We can resurrect it
in
> the future if we need it.

Yeah. I wasn't sure what to do about it either. But having to maintain it at the
moment when we're not actively working on it looked like an unnecessary burden.
Thanks!

[Khushal, 2016-11-12 08:11:28]

The CQ bit was checked by khushalsagar@chromium.org

[Khushal, 2016-11-12 08:11:29]

The patchset sent to the CQ was uploaded after l-g-t-m from enne@chromium.org
Link to the patchset: https://codereview.chromium.org/2493853002/#ps80001
(title: "Rebase")

[commit-bot: I haz the power, 2016-11-12 08:11:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-12 09:22:19]

Description was changed from

==========
cc/blimp: Proto Cleanup.

Remove all unused protos and serialization code.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc/blimp: Proto Cleanup.

Remove all unused protos and serialization code.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[commit-bot: I haz the power, 2016-11-12 09:22:20]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-11-12 09:24:40]

Description was changed from

==========
cc/blimp: Proto Cleanup.

Remove all unused protos and serialization code.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc/blimp: Proto Cleanup.

Remove all unused protos and serialization code.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/4762e7cc7d6ff39382c0fa9266d1db0ab0467bf7
Cr-Commit-Position: refs/heads/master@{#431782}
==========

[commit-bot: I haz the power, 2016-11-12 09:24:41]

Patchset 5 (id:??) landed as
https://crrev.com/4762e7cc7d6ff39382c0fa9266d1db0ab0467bf7
Cr-Commit-Position: refs/heads/master@{#431782}

[agrieve, 2016-11-16 15:57:21]

On 2016/11/12 09:24:41, commit-bot: I haz the power wrote:
> Patchset 5 (id:??) landed as
> https://crrev.com/4762e7cc7d6ff39382c0fa9266d1db0ab0467bf7
> Cr-Commit-Position: refs/heads/master@{#431782}

Nice work! This change (plus your proto one before it) shrunk libchrome.so on
Android by ~80kb!

[Khushal, 2016-11-16 23:57:12]

That's awesome! :D
+Kevin who is also working on restricting proto usage in the blimp layer to
minimize the overhead we add to the apk size.