Remove Windows XP SHA-256 and ECDSA logic.

Remove Windows XP SHA-256 and ECDSA logic.

We can assume the system verifier handles both SHA-256 and ECDSA now.
No need to inject SHA-256 support or disable ECDSA ciphers.

BUG=554410
Committed: https://crrev.com/3c0aa77c46d827484a01662cc0eb383e7de6d93f
Cr-Commit-Position: refs/heads/master@{#371329}

[davidben, 2016-01-21 22:40:49]

Description was changed from

==========
Remove Windows XP SHA-256 and ECDSA logic.

We can assume the system verifier handles both SHA-256 and ECDSA now.

BUG=554410
==========

to

==========
Remove Windows XP SHA-256 and ECDSA logic.

We can assume the system verifier handles both SHA-256 and ECDSA now. No need to
inject SHA-256 support or disable
ECDSA ciphers.

BUG=554410
==========

[Ryan Sleevi, 2016-01-21 22:43:06]

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

[Ryan Sleevi, 2016-01-21 22:43:06]

\(^o^)/

LGTM

https://codereview.chromium.org/1610833004/diff/40001/content/browser/browser...
File content/browser/browser_main_runner.cc (right):

https://codereview.chromium.org/1610833004/diff/40001/content/browser/browser...
content/browser/browser_main_runner.cc:88: if (base::win::GetVersion() <
base::win::VERSION_VISTA) {
Oooh, we can delete this code too.

But not in this CL :P

[Will Harris, 2016-01-21 22:43:55]

On 2016/01/21 22:43:06, Ryan Sleevi wrote:
> \(^o^)/
> 
> LGTM
> 
>
https://codereview.chromium.org/1610833004/diff/40001/content/browser/browser...
> File content/browser/browser_main_runner.cc (right):
> 
>
https://codereview.chromium.org/1610833004/diff/40001/content/browser/browser...
> content/browser/browser_main_runner.cc:88: if (base::win::GetVersion() <
> base::win::VERSION_VISTA) {
> Oooh, we can delete this code too.
> 
> But not in this CL :P

I love this CL.

[davidben, 2016-01-21 22:44:56]

Description was changed from

==========
Remove Windows XP SHA-256 and ECDSA logic.

We can assume the system verifier handles both SHA-256 and ECDSA now. No need to
inject SHA-256 support or disable
ECDSA ciphers.

BUG=554410
==========

to

==========
Remove Windows XP SHA-256 and ECDSA logic.

We can assume the system verifier handles both SHA-256 and ECDSA now.
No need to inject SHA-256 support or disable ECDSA ciphers.

BUG=554410
==========

[davidben, 2016-01-21 22:50:45]

davidben@chromium.org changed reviewers:
+ asargent@chromium.org, cpu@chromium.org

[davidben, 2016-01-21 22:50:46]

+cpu for base/win and components/component_updater
+asargent for chrome/browser/extensions/updater

https://codereview.chromium.org/1610833004/diff/40001/content/browser/browser...
File content/browser/browser_main_runner.cc (right):

https://codereview.chromium.org/1610833004/diff/40001/content/browser/browser...
content/browser/browser_main_runner.cc:88: if (base::win::GetVersion() <
base::win::VERSION_VISTA) {
On 2016/01/21 22:43:06, Ryan Sleevi wrote:
> Oooh, we can delete this code too.
> 
> But not in this CL :P

Oh, didn't even notice that. I'll go upload that CL too. It'll take 5 minutes...

[asargent_no_longer_on_chrome, 2016-01-21 23:06:40]

hooray, lgtm

+cc sorin as FYI

[cpu_(ooo_6.6-7.5), 2016-01-22 17:16:37]

cpu@chromium.org changed reviewers:
+ sorin@chromium.org

[cpu_(ooo_6.6-7.5), 2016-01-22 17:17:08]

+Sorin anyhow.

[cpu_(ooo_6.6-7.5), 2016-01-22 17:17:40]

base/win lgtm

[Sorin Jianu, 2016-01-25 19:15:09]

lgtm

thank you!

[davidben, 2016-01-25 19:16:37]

The CQ bit was checked by davidben@chromium.org

[commit-bot: I haz the power, 2016-01-25 19:19:27]

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

[commit-bot: I haz the power, 2016-01-25 22:08:06]

Description was changed from

==========
Remove Windows XP SHA-256 and ECDSA logic.

We can assume the system verifier handles both SHA-256 and ECDSA now.
No need to inject SHA-256 support or disable ECDSA ciphers.

BUG=554410
==========

to

==========
Remove Windows XP SHA-256 and ECDSA logic.

We can assume the system verifier handles both SHA-256 and ECDSA now.
No need to inject SHA-256 support or disable ECDSA ciphers.

BUG=554410
==========

[commit-bot: I haz the power, 2016-01-25 22:08:06]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-25 22:12:16]

Description was changed from

==========
Remove Windows XP SHA-256 and ECDSA logic.

We can assume the system verifier handles both SHA-256 and ECDSA now.
No need to inject SHA-256 support or disable ECDSA ciphers.

BUG=554410
==========

to

==========
Remove Windows XP SHA-256 and ECDSA logic.

We can assume the system verifier handles both SHA-256 and ECDSA now.
No need to inject SHA-256 support or disable ECDSA ciphers.

BUG=554410

Committed: https://crrev.com/3c0aa77c46d827484a01662cc0eb383e7de6d93f
Cr-Commit-Position: refs/heads/master@{#371329}
==========

[commit-bot: I haz the power, 2016-01-25 22:12:17]

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