Wire up certificate policies support in PathBuilder.

Wire up certificate policies support in PathBuilder.

BUG=634456, 634453, 634452
Review-Url: https://codereview.chromium.org/2898303005
Cr-Commit-Position: refs/heads/master@{#476827}
Committed: https://chromium.googlesource.com/chromium/src/+/e0b219a58b5fa18ad52b6b9dbeca8d08d55fccfa

[eroman, 2017-05-25 19:07:48]

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

[commit-bot: I haz the power, 2017-05-25 19:08:30]

Dry run: CQ is trying da patch.

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

[eroman, 2017-05-25 19:08:37]

eroman@chromium.org changed reviewers:
+ mattm@chromium.org

[commit-bot: I haz the power, 2017-05-25 21:31:51]

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

[commit-bot: I haz the power, 2017-05-25 21:31:52]

Dry run: This issue passed the CQ dry run.

[eroman, 2017-05-30 19:25:36]

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

[commit-bot: I haz the power, 2017-05-30 19:26:40]

Dry run: 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, 2017-05-30 19:48:04]

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

[commit-bot: I haz the power, 2017-05-30 19:48:05]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_cronet on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[eroman, 2017-06-02 01:39:21]

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

[commit-bot: I haz the power, 2017-06-02 01:40:09]

Dry run: CQ is trying da patch.

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

[mattm, 2017-06-02 02:54:49]

lgtm w/nit

https://codereview.chromium.org/2898303005/diff/40001/net/cert/internal/path_...
File net/cert/internal/path_builder_unittest.cc (right):

https://codereview.chromium.org/2898303005/diff/40001/net/cert/internal/path_...
net/cert/internal/path_builder_unittest.cc:659: 
extra line

[commit-bot: I haz the power, 2017-06-02 03:33:06]

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

[commit-bot: I haz the power, 2017-06-02 03:33:07]

Dry run: This issue passed the CQ dry run.

[eroman, 2017-06-02 17:58:53]

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

[eroman, 2017-06-02 17:58:59]

https://codereview.chromium.org/2898303005/diff/40001/net/cert/internal/path_...
File net/cert/internal/path_builder_unittest.cc (right):

https://codereview.chromium.org/2898303005/diff/40001/net/cert/internal/path_...
net/cert/internal/path_builder_unittest.cc:659: 
On 2017/06/02 02:54:49, mattm wrote:
> extra line

Done.

[commit-bot: I haz the power, 2017-06-02 17:59:16]

Dry run: 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, 2017-06-02 18:02:17]

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

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

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[eroman, 2017-06-02 18:02:32]

eroman@chromium.org changed reviewers:
+ dougsteed@chromium.org

[eroman, 2017-06-02 18:02:33]

+dougsteed@ for components/cast_certificate/* approval.

There is no functional change for Cast; this just adds new parameters that to
match the //net interface changes (whose values behave the same as what was
previously happening).

I plan to follow-up with a Cast-only CL that addresses the TODO I added there.

[dougsteed, 2017-06-02 18:38:16]

lgtm

[eroman, 2017-06-02 20:48:07]

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

[commit-bot: I haz the power, 2017-06-02 20:48:32]

Dry run: CQ is trying da patch.

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

[eroman, 2017-06-02 21:26:06]

The CQ bit was unchecked by eroman@chromium.org

[eroman, 2017-06-02 21:26:10]

The CQ bit was checked by eroman@chromium.org

[eroman, 2017-06-02 21:26:10]

The patchset sent to the CQ was uploaded after l-g-t-m from mattm@chromium.org
Link to the patchset: https://codereview.chromium.org/2898303005/#ps60001
(title: "remove extra space")

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

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, 2017-06-02 22:55:58]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1496438770452000,
"parent_rev": "75cec4c72223c895e6a3593faa46b3171905215d", "commit_rev":
"befd38736857aa44a2c7584d5b4815516833deb8"}

[commit-bot: I haz the power, 2017-06-02 22:56:11]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1496438770452000,
"parent_rev": "819697cf41f496d914415cc88a39e39afe4dfe29", "commit_rev":
"e0b219a58b5fa18ad52b6b9dbeca8d08d55fccfa"}

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

Description was changed from

==========
Wire up certificate policies support in PathBuilder.

BUG=634456, 634453, 634452
==========

to

==========
Wire up certificate policies support in PathBuilder.

BUG=634456, 634453, 634452

Review-Url: https://codereview.chromium.org/2898303005
Cr-Commit-Position: refs/heads/master@{#476827}
Committed:
https://chromium.googlesource.com/chromium/src/+/e0b219a58b5fa18ad52b6b9dbeca...
==========

[commit-bot: I haz the power, 2017-06-02 22:56:24]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/e0b219a58b5fa18ad52b6b9dbeca...