Replace cookie lines with CanonicalCookie in CanSetCookie delegates.

This is a follow-up of https://crrev.com/c/771510 and finishes the
replacement of the std::string ("Set-Cookie:" HTTP header line) argument
with net::CanonicalCookie in network delegates that give embedders an
opportunity to block cookies.

This CL removes duplicate parsing of Set-Cookie: HTTP header lines,
which was an opportunity for bugs.

Bug: 756093
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ibe4ea727cb55665f1973483345c97e4b2122b7ea
Reviewed-on: https://chromium-review.googlesource.com/775606
Reviewed-by: Alex Clarke <alexclarke@chromium.org>
Reviewed-by: Mark Cogan <marq@chromium.org>
Reviewed-by: Bo <boliu@chromium.org>
Reviewed-by: Randy Smith <rdsmith@chromium.org>
Reviewed-by: Misha Efimov <mef@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517837}
35 files changed