Mojo service implementation for HTTP server - part 1

Mojo service implementation for HTTP server - part 1

This CL adds HTTP server and connection classes, and implements lifespan management for them.

The following work will be in future CLs:
- convert net::HttpServer::Delegate events about HTTP/WebSocket requests to HttpConnectionDelegate Mojo interface method calls.
- tests.

BUG=478249
TEST=None

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

[yzshen1, 2015-05-11 23:47:07]

yzshen@chromium.org changed reviewers:
+ jam@chromium.org

[yzshen1, 2015-05-11 23:47:08]

Hi, John.

This CL depends on https://codereview.chromium.org/1136773002/

I pipelined this CL before getting your LG for 1136773002. If you think
1136773002 needs any further changes, please feel free to ignore this CL until I
make corresponding changes.

Thanks!

[jam, 2015-05-12 15:50:24]

lgtm with question

https://codereview.chromium.org/1134883002/diff/40001/mojo/services/network/h...
File mojo/services/network/http_server_impl.h (right):

https://codereview.chromium.org/1134883002/diff/40001/mojo/services/network/h...
mojo/services/network/http_server_impl.h:33: static HttpServerImpl*
Create(HttpServerDelegatePtr delegate);
why is there a static factory method which just calls a private constructor?
i.e. why not just have the ctor public and remove this method?

[yzshen1, 2015-05-12 16:02:05]

Thanks, John!

https://codereview.chromium.org/1134883002/diff/40001/mojo/services/network/h...
File mojo/services/network/http_server_impl.h (right):

https://codereview.chromium.org/1134883002/diff/40001/mojo/services/network/h...
mojo/services/network/http_server_impl.h:33: static HttpServerImpl*
Create(HttpServerDelegatePtr delegate);
On 2015/05/12 15:50:23, jam wrote:
> why is there a static factory method which just calls a private constructor?
> i.e. why not just have the ctor public and remove this method?

Making the ctor private has be benefit of preventing user from allocating on
stack.

After some more thoughts, I moved more code into Create() and made the lifespan
of the object an implementation detail.

Please take another look of this method. Thanks!

[jam, 2015-05-12 19:26:09]

lgtm

[yzshen1, 2015-05-12 19:33:48]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2015-05-12 19:36:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1134883002/60001

[commit-bot: I haz the power, 2015-05-12 19:42:00]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-12 19:42:49]

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