Implement simple http server with set cookie functionality.

Implement simple http server with set cookie functionality.

Add functionality to http server utility so that it can include Set-Coockie in
the header. Some tests need cookies to be set, sets cookie through http response
can better simulate real world user behavior than through javascript.

BUG=

Committed: https://crrev.com/f4eee1cd7196932caa0261c4fed6723242d4d4a8
Cr-Commit-Position: refs/heads/master@{#415322}

[liaoyuke, 2016-08-27 00:15:52]

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

[commit-bot: I haz the power, 2016-08-27 00:16:42]

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

[liaoyuke, 2016-08-27 00:17:26]

liaoyuke@chromium.org changed reviewers:
+ baxley@chromium.org

[liaoyuke, 2016-08-27 00:17:27]

I created some tests locally to verify that the functionality does set cookies.

[commit-bot: I haz the power, 2016-08-27 01:07:44]

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

[commit-bot: I haz the power, 2016-08-27 01:07:44]

Dry run: This issue passed the CQ dry run.

[baxley, 2016-08-30 15:52:40]

lgtm

[liaoyuke, 2016-08-30 16:05:35]

The CQ bit was checked by liaoyuke@chromium.org

[commit-bot: I haz the power, 2016-08-30 16:06:15]

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-08-30 16:52:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 16:54:51]

Description was changed from

==========
Implement simple http server with set cookie functionality.

Add functionality to http server utility so that it can include Set-Coockie in
the header. Some tests need cookies to be set, sets cookie through http response
can better simulate real world user behavior than through javascript.

BUG=
==========

to

==========
Implement simple http server with set cookie functionality.

Add functionality to http server utility so that it can include Set-Coockie in
the header. Some tests need cookies to be set, sets cookie through http response
can better simulate real world user behavior than through javascript.

BUG=

Committed: https://crrev.com/f4eee1cd7196932caa0261c4fed6723242d4d4a8
Cr-Commit-Position: refs/heads/master@{#415322}
==========

[commit-bot: I haz the power, 2016-08-30 16:54:52]

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

[pkl (ping after 24h if needed), 2016-08-30 21:24:04]

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

[pkl (ping after 24h if needed), 2016-08-30 21:24:04]

https://codereview.chromium.org/2283203003/diff/1/ios/web/public/test/respons...
File ios/web/public/test/response_providers/html_response_provider.h (right):

https://codereview.chromium.org/2283203003/diff/1/ios/web/public/test/respons...
ios/web/public/test/response_providers/html_response_provider.h:32: const
std::string& cookie);
I don't think "explicit" on a constructor w/ more than 1 parameter means
anything. Do you?
See HtmlResponseProviderImpl as well.

https://codereview.chromium.org/2283203003/diff/1/ios/web/public/test/respons...
File ios/web/public/test/response_providers/response_provider.h (right):

https://codereview.chromium.org/2283203003/diff/1/ios/web/public/test/respons...
ios/web/public/test/response_providers/response_provider.h:57: // response code.
This comment does not match what this method actually does. It should describe
what it does with |cookie|.

[liaoyuke, 2016-09-12 18:58:13]

https://codereview.chromium.org/2283203003/diff/1/ios/web/public/test/respons...
File ios/web/public/test/response_providers/html_response_provider.h (right):

https://codereview.chromium.org/2283203003/diff/1/ios/web/public/test/respons...
ios/web/public/test/response_providers/html_response_provider.h:32: const
std::string& cookie);
On 2016/08/30 21:24:04, pklpkl wrote:
> I don't think "explicit" on a constructor w/ more than 1 parameter means
> anything. Do you?
> See HtmlResponseProviderImpl as well.

Done.

https://codereview.chromium.org/2283203003/diff/1/ios/web/public/test/respons...
File ios/web/public/test/response_providers/response_provider.h (right):

https://codereview.chromium.org/2283203003/diff/1/ios/web/public/test/respons...
ios/web/public/test/response_providers/response_provider.h:57: // response code.
On 2016/08/30 21:24:04, pklpkl wrote:
> This comment does not match what this method actually does. It should describe
> what it does with |cookie|.

Acknowledged.