Revert of CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in path pruning loop.

Revert of CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in path pruning loop. (patchset #11 id:300001 of https://codereview.chromium.org/2101303005/ )

Reason for revert:
This breaks verification on OS X 10.12 and probably needs some further investigation.

Original issue's description:
> CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in path pruning loop.
>
> This also removes the native hostname checking workarounds.
>
> BUG=570909, 588789, 621684
>
> Committed: https://crrev.com/9cedf75377d817c6b32a01f1d30fbe10663b8bb8
> Cr-Commit-Position: refs/heads/master@{#418732}

TBR=rsleevi@chromium.org,mattm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=570909, 588789, 621684, 647241
Committed: https://crrev.com/03a7397bd4ed8c059a7f5977a7672769fb783626
Cr-Commit-Position: refs/heads/master@{#419245}

[davidben, 2016-09-16 15:21:48]

Created Revert of CertVerifyProcMac: Add Keychain re-ordering hack, check
CRLsets in path pruning loop.

[davidben, 2016-09-16 15:22:16]

Description was changed from

==========
Revert of CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in
path pruning loop. (patchset #11 id:300001 of
https://codereview.chromium.org/2101303005/ )

Reason for revert:
This breaks verification on OS X 10.12 and probably needs some further
investigation.

BUG=647241

Original issue's description:
> CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in path
pruning loop.
> 
> This also removes the native hostname checking workarounds.
> 
> BUG=570909,588789,621684
> 
> Committed: https://crrev.com/9cedf75377d817c6b32a01f1d30fbe10663b8bb8
> Cr-Commit-Position: refs/heads/master@{#418732}

TBR=rsleevi@chromium.org,mattm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=570909,588789,621684
==========

to

==========
Revert of CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in
path pruning loop. (patchset #11 id:300001 of
https://codereview.chromium.org/2101303005/ )

Reason for revert:
This breaks verification on OS X 10.12 and probably needs some further
investigation.

Original issue's description:
> CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in path
pruning loop.
> 
> This also removes the native hostname checking workarounds.
> 
> BUG=570909,588789,621684
> 
> Committed: https://crrev.com/9cedf75377d817c6b32a01f1d30fbe10663b8bb8
> Cr-Commit-Position: refs/heads/master@{#418732}

TBR=rsleevi@chromium.org,mattm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=570909,588789,621684,647241
==========

[davidben, 2016-09-16 15:22:21]

The CQ bit was checked by davidben@chromium.org

[commit-bot: I haz the power, 2016-09-16 15:22:45]

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

[cbentzel, 2016-09-16 15:35:04]

On 2016/09/16 15:22:45, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

LGTM since just revert

[commit-bot: I haz the power, 2016-09-16 16:30:32]

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

[commit-bot: I haz the power, 2016-09-16 16:30:33]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[davidben, 2016-09-16 16:40:17]

On 2016/09/16 16:30:33, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

...that doesn't make any sense. This CL appears to #ifdef out the offending
check.

https://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel...

[davidben, 2016-09-16 18:22:04]

The CQ bit was checked by davidben@chromium.org

[davidben, 2016-09-16 18:22:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
cbentzel@chromium.org
Link to the patchset: https://codereview.chromium.org/2347893002/#ps250001
(title: "git cl patch is broken so try a manual revert instead")

[commit-bot: I haz the power, 2016-09-16 18:22:26]

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-16 19:21:37]

Description was changed from

==========
Revert of CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in
path pruning loop. (patchset #11 id:300001 of
https://codereview.chromium.org/2101303005/ )

Reason for revert:
This breaks verification on OS X 10.12 and probably needs some further
investigation.

Original issue's description:
> CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in path
pruning loop.
> 
> This also removes the native hostname checking workarounds.
> 
> BUG=570909,588789,621684
> 
> Committed: https://crrev.com/9cedf75377d817c6b32a01f1d30fbe10663b8bb8
> Cr-Commit-Position: refs/heads/master@{#418732}

TBR=rsleevi@chromium.org,mattm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=570909,588789,621684,647241
==========

to

==========
Revert of CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in
path pruning loop. (patchset #11 id:300001 of
https://codereview.chromium.org/2101303005/ )

Reason for revert:
This breaks verification on OS X 10.12 and probably needs some further
investigation.

Original issue's description:
> CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in path
pruning loop.
> 
> This also removes the native hostname checking workarounds.
> 
> BUG=570909,588789,621684
> 
> Committed: https://crrev.com/9cedf75377d817c6b32a01f1d30fbe10663b8bb8
> Cr-Commit-Position: refs/heads/master@{#418732}

TBR=rsleevi@chromium.org,mattm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=570909,588789,621684,647241
==========

[commit-bot: I haz the power, 2016-09-16 19:21:38]

Committed patchset #2 (id:250001)

[commit-bot: I haz the power, 2016-09-16 19:23:13]

Description was changed from

==========
Revert of CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in
path pruning loop. (patchset #11 id:300001 of
https://codereview.chromium.org/2101303005/ )

Reason for revert:
This breaks verification on OS X 10.12 and probably needs some further
investigation.

Original issue's description:
> CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in path
pruning loop.
> 
> This also removes the native hostname checking workarounds.
> 
> BUG=570909,588789,621684
> 
> Committed: https://crrev.com/9cedf75377d817c6b32a01f1d30fbe10663b8bb8
> Cr-Commit-Position: refs/heads/master@{#418732}

TBR=rsleevi@chromium.org,mattm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=570909,588789,621684,647241
==========

to

==========
Revert of CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in
path pruning loop. (patchset #11 id:300001 of
https://codereview.chromium.org/2101303005/ )

Reason for revert:
This breaks verification on OS X 10.12 and probably needs some further
investigation.

Original issue's description:
> CertVerifyProcMac: Add Keychain re-ordering hack, check CRLsets in path
pruning loop.
>
> This also removes the native hostname checking workarounds.
>
> BUG=570909,588789,621684
>
> Committed: https://crrev.com/9cedf75377d817c6b32a01f1d30fbe10663b8bb8
> Cr-Commit-Position: refs/heads/master@{#418732}

TBR=rsleevi@chromium.org,mattm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=570909,588789,621684,647241

Committed: https://crrev.com/03a7397bd4ed8c059a7f5977a7672769fb783626
Cr-Commit-Position: refs/heads/master@{#419245}
==========

[commit-bot: I haz the power, 2016-09-16 19:23:14]

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