Deprecate FLAGS_quic_reloadable_flag_enable_async_get_proof

Deprecate FLAGS_quic_reloadable_flag_enable_async_get_proof

Merge internal change: 146243587

BUG=

[Fan Yang, 2017-02-06 21:01:28]

Description was changed from

==========
Deprecate FLAGS_quic_reloadable_flag_enable_async_get_proof

Merge internal change: 146243587

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

to

==========
Deprecate FLAGS_quic_reloadable_flag_enable_async_get_proof

Merge internal change: 146243587

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

[Fan Yang, 2017-02-06 21:01:28]

fayang@chromium.org changed reviewers:
+ gredner@chromium.org, rch@chromium.org

[Fan Yang, 2017-02-06 21:01:55]

fayang@chromium.org changed reviewers:
+ gredner@google.com
- gredner@chromium.org

[Fan Yang, 2017-02-06 21:02:16]

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

[commit-bot: I haz the power, 2017-02-06 21:02:58]

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

[gredner, 2017-02-06 21:07:54]

LGTM modulo a request for a few comments.  Thanks for taking care of this! 
Hopefully future merges in this area will be much more straightforward.

https://codereview.chromium.org/2679783003/diff/1/net/quic/chromium/crypto/pr...
File net/quic/chromium/crypto/proof_test_chromium.cc (right):

https://codereview.chromium.org/2679783003/diff/1/net/quic/chromium/crypto/pr...
net/quic/chromium/crypto/proof_test_chromium.cc:153:
source->GetProof(server_addr, hostname, server_config, quic_version,
For safety, I think it's a good idea to comment that this code expects the async
method to invoke the callback synchronously (here and elsewhere).

[Fan Yang, 2017-02-06 21:43:20]

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

[Fan Yang, 2017-02-06 21:43:30]

On 2017/02/06 21:07:54, gredner wrote:
> LGTM modulo a request for a few comments.  Thanks for taking care of this! 
> Hopefully future merges in this area will be much more straightforward.
> 
>
https://codereview.chromium.org/2679783003/diff/1/net/quic/chromium/crypto/pr...
> File net/quic/chromium/crypto/proof_test_chromium.cc (right):
> 
>
https://codereview.chromium.org/2679783003/diff/1/net/quic/chromium/crypto/pr...
> net/quic/chromium/crypto/proof_test_chromium.cc:153:
> source->GetProof(server_addr, hostname, server_config, quic_version,
> For safety, I think it's a good idea to comment that this code expects the
async
> method to invoke the callback synchronously (here and elsewhere).

Done. Thank you.

[commit-bot: I haz the power, 2017-02-06 21:44:13]

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

[gredner, 2017-02-06 21:47:13]

lgtm

[gredner, 2017-02-06 21:47:14]

lgtm

[commit-bot: I haz the power, 2017-02-06 22:59:37]

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

[commit-bot: I haz the power, 2017-02-06 22:59:38]

Dry run: This issue passed the CQ dry run.

[Ryan Hamilton, 2017-02-06 23:15:17]

lgtm