Remove Windows Vista/XP specific code from net/quic.

Remove Windows Vista/XP specific code from net/quic.

BUG=580691
Committed: https://crrev.com/e3a795ebed090c2b43e8dcf2af854382b5d3a7b0
Cr-Commit-Position: refs/heads/master@{#394869}

[martijnc, 2016-05-16 13:47:10]

Patchset #1 (id:1) has been deleted

[martijnc, 2016-05-16 15:35:19]

Description was changed from

==========
Remove Windows Vista/XP specific handling from net/quic.

BUG=580691
==========

to

==========
Remove Windows Vista/XP specific code from net/quic.

BUG=580691
==========

[martijnc, 2016-05-16 17:08:49]

martijn@martijnc.be changed reviewers:
+ rch@chromium.org

[martijnc, 2016-05-16 18:48:14]

Patchset #1 (id:20001) has been deleted

[martijnc, 2016-05-16 20:28:31]

Hi, can you review this patch? Thanks!

[Ryan Hamilton, 2016-05-17 21:18:39]

This change looks good, but there's a bit more to be removed. Since this code is
shared with the internal repository, I have a suggestion. Can you prune this CL
to only involve changes to quic_stream_factory.cc and proof_test.cc? Then I'll
pick up the other changes and continue removing code in the internal version,
which will eventually be merged to chrome.  Cool?

[martijnc, 2016-05-18 17:04:53]

On 2016/05/17 at 21:18:39, rch wrote:
> This change looks good, but there's a bit more to be removed. Since this code
is shared with the internal repository, I have a suggestion. Can you prune this
CL to only involve changes to quic_stream_factory.cc and proof_test.cc? Then
I'll pick up the other changes and continue removing code in the internal
version, which will eventually be merged to chrome.  Cool?

Yes, sounds good, I've uploaded patchset 2 which only includes the changes to
proof_test.cc and quic_stream_factory.cc.

[Ryan Hamilton, 2016-05-18 21:35:48]

lgtm

[martijnc, 2016-05-19 16:56:56]

The CQ bit was checked by martijn@martijnc.be

[commit-bot: I haz the power, 2016-05-19 16:57:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1977293002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1977293002/60001

[commit-bot: I haz the power, 2016-05-19 18:54:46]

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

[commit-bot: I haz the power, 2016-05-19 18:54:47]

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

[martijnc, 2016-05-19 19:02:30]

The CQ bit was checked by martijn@martijnc.be

[commit-bot: I haz the power, 2016-05-19 19:03:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1977293002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1977293002/60001

[commit-bot: I haz the power, 2016-05-19 21:22:05]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-05-19 21:23:17]

Description was changed from

==========
Remove Windows Vista/XP specific code from net/quic.

BUG=580691
==========

to

==========
Remove Windows Vista/XP specific code from net/quic.

BUG=580691

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

[commit-bot: I haz the power, 2016-05-19 21:23:19]

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