cc: Remove all blimp code from cc.

cc: Remove all blimp code from cc.

This change removes all code in cc/blimp and cc that was used only by
blimp, including the proto files, serialization code and related tests.

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

Review-Url: https://codereview.chromium.org/2646623002
Cr-Commit-Position: refs/heads/master@{#445161}
Committed: https://chromium.googlesource.com/chromium/src/+/54203a1937cde8d7305b2f94110a91298a4bc2c3

[Khushal, 2017-01-18 21:41:31]

Description was changed from

==========
cc: Remove all blimp code from cc.

This change removes all code in cc/blimp and cc that was used only by
blimp, including the proto files, serialization code and related tests.

BUG=677556
==========

to

==========
cc: Remove all blimp code from cc.

This change removes all code in cc/blimp and cc that was used only by
blimp, including the proto files, serialization code and related tests.

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

[Khushal, 2017-01-18 21:42:26]

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

[Khushal, 2017-01-18 21:42:27]

This depends on the change that removes all cc/blimp dependencies from content
first.

[enne (OOO), 2017-01-18 22:20:00]

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

[enne (OOO), 2017-01-18 22:20:01]

lgtm

[vmpstr, 2017-01-19 23:26:07]

lgtm

[Khushal, 2017-01-20 01:37:50]

The CQ bit was checked by khushalsagar@chromium.org

[commit-bot: I haz the power, 2017-01-20 01:38:37]

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, 2017-01-20 02:45:59]

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

[commit-bot: I haz the power, 2017-01-20 02:46:01]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Khushal, 2017-01-20 02:57:24]

The CQ bit was checked by khushalsagar@chromium.org

[Khushal, 2017-01-20 02:57:25]

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

[commit-bot: I haz the power, 2017-01-20 02:57:52]

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, 2017-01-20 04:04:58]

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

[commit-bot: I haz the power, 2017-01-20 04:04:59]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Khushal, 2017-01-20 05:56:18]

The CQ bit was checked by khushalsagar@chromium.org

[commit-bot: I haz the power, 2017-01-20 05:56: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, 2017-01-20 07:10:13]

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

[commit-bot: I haz the power, 2017-01-20 07:10:15]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Khushal, 2017-01-20 18:10:15]

Why is this failing on files I didn't change? T_T

e:\b\c\b\win\src\cc\quads\draw_polygon.cc(107): error C2220: warning treated as
error - no 'object' file generated
e:\b\c\b\win\src\cc\quads\draw_polygon.cc(107): warning C4267: 'initializing':
conversion from 'size_t' to 'int', possible loss of data

[Khushal, 2017-01-20 18:43:55]

The CQ bit was checked by khushalsagar@chromium.org

[Khushal, 2017-01-20 18:43:56]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org,
enne@chromium.org
Link to the patchset: https://codereview.chromium.org/2646623002/#ps40001
(title: "build fix")

[commit-bot: I haz the power, 2017-01-20 18:44:18]

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, 2017-01-20 19:41:19]

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

[commit-bot: I haz the power, 2017-01-20 19:41:20]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Khushal, 2017-01-20 19:47:44]

The CQ bit was checked by khushalsagar@chromium.org

[Khushal, 2017-01-20 19:47:44]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org,
enne@chromium.org
Link to the patchset: https://codereview.chromium.org/2646623002/#ps50001
(title: "test build")

[commit-bot: I haz the power, 2017-01-20 19:48:26]

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, 2017-01-20 21:06:02]

CQ is committing da patch.

Bot data: {"patchset_id": 50001, "attempt_start_ts": 1484941664279790,
"parent_rev": "8df74b7ca28739c0ec5ec29657063ad1b71e69e4", "commit_rev":
"54203a1937cde8d7305b2f94110a91298a4bc2c3"}

[commit-bot: I haz the power, 2017-01-20 21:06:34]

Description was changed from

==========
cc: Remove all blimp code from cc.

This change removes all code in cc/blimp and cc that was used only by
blimp, including the proto files, serialization code and related tests.

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

to

==========
cc: Remove all blimp code from cc.

This change removes all code in cc/blimp and cc that was used only by
blimp, including the proto files, serialization code and related tests.

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

Review-Url: https://codereview.chromium.org/2646623002
Cr-Commit-Position: refs/heads/master@{#445161}
Committed:
https://chromium.googlesource.com/chromium/src/+/54203a1937cde8d7305b2f94110a...
==========

[commit-bot: I haz the power, 2017-01-20 21:06:35]

Committed patchset #4 (id:50001) as
https://chromium.googlesource.com/chromium/src/+/54203a1937cde8d7305b2f94110a...

[Wez, 2017-02-08 17:57:43]

Description was changed from

==========
cc: Remove all blimp code from cc.

This change removes all code in cc/blimp and cc that was used only by
blimp, including the proto files, serialization code and related tests.

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

Review-Url: https://codereview.chromium.org/2646623002
Cr-Commit-Position: refs/heads/master@{#445161}
Committed:
https://chromium.googlesource.com/chromium/src/+/54203a1937cde8d7305b2f94110a...
==========

to

==========
cc: Remove all blimp code from cc.

This change removes all code in cc/blimp and cc that was used only by
blimp, including the proto files, serialization code and related tests.

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

Review-Url: https://codereview.chromium.org/2646623002
Cr-Commit-Position: refs/heads/master@{#445161}
Committed:
https://chromium.googlesource.com/chromium/src/+/54203a1937cde8d7305b2f94110a...
==========

[Wez, 2017-02-08 17:57:43]

wez@chromium.org changed reviewers:
+ avi@chromium.org

[Wez, 2017-02-08 17:59:35]

Hallo avi@chromium.org!
Due to a depot_tools patch which mistakenly removed the OWNERS check for
non-source files (see crbug.com/684270), the following files landed in this CL
and need a retrospective review from you:
	content/renderer/BUILD.gn
Thanks,
Wez

[Avi (use Gerrit), 2017-02-08 18:00:52]

On 2017/02/08 17:59:35, Wez wrote:
> Hallo mailto:avi@chromium.org!
> Due to a depot_tools patch which mistakenly removed the OWNERS check for
> non-source files (see crbug.com/684270), the following files landed in this CL
> and need a retrospective review from you:
> 	content/renderer/BUILD.gn
> Thanks,
> Wez

This removes a DEPS entry, so BUILD.gn has my L👍TM.

lgtm