Obliterate legacy cronet API. All of its users have been migrated.

Obliterate legacy cronet API. All of its users have been migrated.

Committed: https://crrev.com/f62a203e0c8926aa7975f004b354837a351221c2
Cr-Commit-Position: refs/heads/master@{#420716}

[Charles, 2016-09-23 15:11:08]

Description was changed from

==========
Obliterate legacy cronet API. All of its users have been migrated.

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

to

==========
Obliterate legacy cronet API. All of its users have been migrated.
==========

[Charles, 2016-09-23 15:11:08]

clm@google.com changed reviewers:
+ mef@chromium.org

[Charles, 2016-09-23 15:11:31]

PTAL

[mef, 2016-09-23 16:25:18]

mef@chromium.org changed reviewers:
+ kapishnikov@chromium.org

[mef, 2016-09-23 16:25:19]

Nice! Andrei has removal of legacy API as part of builder API changes CL, so
I'll move this review to him for coordination.

[kapishnikov, 2016-09-23 17:34:43]

On 2016/09/23 15:11:31, Charles wrote:
> PTAL

Charles here is the pending CL that also removes the deprecated API:
https://codereview.chromium.org/2339223002. I see that I have missed some of the
files, e.g. wrapped_channel_upload_element_reader.cc. Would you like me to
incorporate your changes into my CL or should we submit it separately? I think
your CL looks good and can be submitted separately. LGTM if the corresponding
work in google3 is done and all tests passing.

[Charles, 2016-09-23 17:41:36]

On 2016/09/23 17:34:43, kapishnikov wrote:
> On 2016/09/23 15:11:31, Charles wrote:
> > PTAL
> 
> Charles here is the pending CL that also removes the deprecated API:
> https://codereview.chromium.org/2339223002. I see that I have missed some of
the
> files, e.g. wrapped_channel_upload_element_reader.cc. Would you like me to
> incorporate your changes into my CL or should we submit it separately? I think
> your CL looks good and can be submitted separately. LGTM if the corresponding
> work in google3 is done and all tests passing.

Google3 work is done as of yesterday, and all the remaining tests are passing.
Submitting!

[Charles, 2016-09-23 17:41:49]

The CQ bit was checked by clm@google.com

[commit-bot: I haz the power, 2016-09-23 17:43:01]

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-09-23 20:30:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-23 20:32:53]

Description was changed from

==========
Obliterate legacy cronet API. All of its users have been migrated.
==========

to

==========
Obliterate legacy cronet API. All of its users have been migrated.

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

[commit-bot: I haz the power, 2016-09-23 20:32:55]

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