Remove obsolete QUIC Compressed Cert Set v1.

Remove obsolete QUIC Compressed Cert Set v1.

Merge internal change: 130089774

BUG=636423
Committed: https://crrev.com/aa7a8e79bcfc1680fabf697c9f473e8e7908ccaa
Cr-Commit-Position: refs/heads/master@{#411744}

[Ryan Hamilton, 2016-08-11 21:54:48]

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

[Ryan Hamilton, 2016-08-11 21:55:06]

rch@chromium.org changed reviewers:
+ phajdan.jr@chromium.org, zhongyi@chromium.org

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

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-08-11 22:06:19]

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

[commit-bot: I haz the power, 2016-08-11 22:06:20]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Zhongyi Shi, 2016-08-11 23:05:00]

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
File net/quic/core/crypto/common_cert_set_1a.inc (left):

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
net/quic/core/crypto/common_cert_set_1a.inc:5: 
Looks like this file will only contain the license header. Remove the file
maybe?

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
File net/quic/core/crypto/common_cert_set_1b.inc (left):

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
net/quic/core/crypto/common_cert_set_1b.inc:5: 
ditto

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
File net/quic/core/crypto/common_cert_set_test.cc (left):

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
net/quic/core/crypto/common_cert_set_test.cc:12: 
Removed unintentionally? I thought these should be kept as before.

[Ryan Hamilton, 2016-08-11 23:28:33]

cleanup

[Ryan Hamilton, 2016-08-11 23:29:29]

Thanks.

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
File net/quic/core/crypto/common_cert_set_1a.inc (left):

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
net/quic/core/crypto/common_cert_set_1a.inc:5: 
On 2016/08/11 23:05:00, Zhongyi Shi wrote:
> Looks like this file will only contain the license header. Remove the file
> maybe?

Weird! I'm not sure how this happened. But yeah, removed.

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
File net/quic/core/crypto/common_cert_set_1b.inc (left):

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
net/quic/core/crypto/common_cert_set_1b.inc:5: 
On 2016/08/11 23:05:00, Zhongyi Shi wrote:
> ditto

Done.

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
File net/quic/core/crypto/common_cert_set_test.cc (left):

https://codereview.chromium.org/2242623002/diff/1/net/quic/core/crypto/common...
net/quic/core/crypto/common_cert_set_test.cc:12: 
On 2016/08/11 23:05:00, Zhongyi Shi wrote:
> Removed unintentionally? I thought these should be kept as before.

Done.

[Ryan Hamilton, 2016-08-11 23:29:34]

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

[commit-bot: I haz the power, 2016-08-11 23:30:51]

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

[Zhongyi Shi, 2016-08-12 00:04:01]

lgtm

[commit-bot: I haz the power, 2016-08-12 00:47:44]

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

[commit-bot: I haz the power, 2016-08-12 00:47:44]

Dry run: This issue passed the CQ dry run.

[Ryan Hamilton, 2016-08-12 13:34:41]

Description was changed from

==========
Remove obsolete QUIC Compressed Cert Set v1.

Merge internal change: 130034370
==========

to

==========
Remove obsolete QUIC Compressed Cert Set v1.

Merge internal change: 130089774
==========

[Ryan Hamilton, 2016-08-12 16:33:06]

Description was changed from

==========
Remove obsolete QUIC Compressed Cert Set v1.

Merge internal change: 130089774
==========

to

==========
Remove obsolete QUIC Compressed Cert Set v1.

Merge internal change: 130089774

BUG=636423
==========

[Ryan Hamilton, 2016-08-12 17:38:47]

rch@chromium.org changed reviewers:
+ torne@chromium.org
- phajdan.jr@chromium.org

[Torne, 2016-08-12 17:48:32]

copyright scanner whitelist LGTM

[Ryan Hamilton, 2016-08-12 17:52:01]

Patchset #1 (id:1) has been deleted

[Ryan Hamilton, 2016-08-12 17:52:07]

Patchset #2 (id:40001) has been deleted

[Ryan Hamilton, 2016-08-12 17:56:01]

Patchset #1 (id:20001) has been deleted

[Ryan Hamilton, 2016-08-12 17:56:09]

Patchset #1 (id:60001) has been deleted

[Ryan Hamilton, 2016-08-12 17:58:01]

Patchset #1 (id:80001) has been deleted

[Ryan Hamilton, 2016-08-12 18:00:27]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2016-08-12 18:00:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
zhongyi@chromium.org, torne@chromium.org
Link to the patchset: https://codereview.chromium.org/2242623002/#ps40002
(title: "Rebase")

[commit-bot: I haz the power, 2016-08-12 18:00:50]

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-08-12 18:27:57]

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

[commit-bot: I haz the power, 2016-08-12 18:27:58]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[Ryan Hamilton, 2016-08-12 19:16:15]

The CQ bit was checked by rch@chromium.org

[commit-bot: I haz the power, 2016-08-12 19:16:44]

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-08-12 19:58:31]

Description was changed from

==========
Remove obsolete QUIC Compressed Cert Set v1.

Merge internal change: 130089774

BUG=636423
==========

to

==========
Remove obsolete QUIC Compressed Cert Set v1.

Merge internal change: 130089774

BUG=636423
==========

[commit-bot: I haz the power, 2016-08-12 19:58:36]

Committed patchset #2 (id:40002)

[commit-bot: I haz the power, 2016-08-12 20:00:49]

Description was changed from

==========
Remove obsolete QUIC Compressed Cert Set v1.

Merge internal change: 130089774

BUG=636423
==========

to

==========
Remove obsolete QUIC Compressed Cert Set v1.

Merge internal change: 130089774

BUG=636423

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

[commit-bot: I haz the power, 2016-08-12 20:00:50]

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