Reland: WKWebView: Added cert verification API to web controller.

Reland: WKWebView: Added cert verification API to web controller.

This code is just a skeleton for verification and verification method
is not used for making security decisions or presenting security UI.

The decision to use CertVerifier instead of iOS cert verification API
has not been made yet. But using CertVerifier is easier for
-[WKWebView certificateChain] verification, so this CL uses
CertVerifier.

Original URL: https://codereview.chromium.org/1230033005/

BUG=462427, 462425
Committed: https://crrev.com/80d735a2bf710864c6f738b8fd05244c54c140ca
Cr-Commit-Position: refs/heads/master@{#348699}

[Eugene But (OOO till 7-30), 2015-09-04 18:07:17]

eugenebut@chromium.org changed reviewers:
+ stuartmorgan@chromium.org

[Eugene But (OOO till 7-30), 2015-09-04 18:07:18]

Original patch failed (and reverted) because of dependency CL, which I forgot to
land. However I had to make some include changes to make it compile downstream.
PTAL.

[stuartmorgan, 2015-09-04 18:22:32]

Diff lgtm

https://codereview.chromium.org/1316063008/diff/20001/ios/web/web_state/ui/cr...
File ios/web/web_state/ui/crw_wk_web_view_web_controller.mm (right):

https://codereview.chromium.org/1316063008/diff/20001/ios/web/web_state/ui/cr...
ios/web/web_state/ui/crw_wk_web_view_web_controller.mm:46: #if
!defined(ENABLE_CHROME_NET_STACK_FOR_WKWEBVIEW)
How about we just go ahead and eliminate this whole section, folding everything
up into the main includes? I don't think we need to worry about having extra
headers in a config we're trying to eliminate.

[Eugene But (OOO till 7-30), 2015-09-04 18:39:54]

Thanks!

https://codereview.chromium.org/1316063008/diff/20001/ios/web/web_state/ui/cr...
File ios/web/web_state/ui/crw_wk_web_view_web_controller.mm (right):

https://codereview.chromium.org/1316063008/diff/20001/ios/web/web_state/ui/cr...
ios/web/web_state/ui/crw_wk_web_view_web_controller.mm:46: #if
!defined(ENABLE_CHROME_NET_STACK_FOR_WKWEBVIEW)
On 2015/09/04 18:22:32, stuartmorgan wrote:
> How about we just go ahead and eliminate this whole section, folding
everything
> up into the main includes? I don't think we need to worry about having extra
> headers in a config we're trying to eliminate.

Done.

[Eugene But (OOO till 7-30), 2015-09-14 18:15:28]

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

[commit-bot: I haz the power, 2015-09-14 18:16:37]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1316063008/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1316063008/80001

[commit-bot: I haz the power, 2015-09-14 19:12:06]

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

[commit-bot: I haz the power, 2015-09-14 19:12:07]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2015-09-14 20:02:43]

The CQ bit was checked by eugenebut@chromium.org

[Eugene But (OOO till 7-30), 2015-09-14 20:02:44]

The patchset sent to the CQ was uploaded after l-g-t-m from
stuartmorgan@chromium.org
Link to the patchset: https://codereview.chromium.org/1316063008/#ps80001
(title: "Gracefully handle null NetLog.")

[commit-bot: I haz the power, 2015-09-14 20:03:16]

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

[commit-bot: I haz the power, 2015-09-14 20:14:23]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-09-14 20:15:06]

Patchset 5 (id:??) landed as
https://crrev.com/80d735a2bf710864c6f738b8fd05244c54c140ca
Cr-Commit-Position: refs/heads/master@{#348699}

[commit-bot: I haz the power, 2015-09-23 12:36:29]

Patchset 5 (id:??) landed as
https://crrev.com/80d735a2bf710864c6f738b8fd05244c54c140ca
Cr-Commit-Position: refs/heads/master@{#348699}