Added HostConfig class. Changed SimpleHost to use it.

Added HostConfig class. Changed SimpleHost to use it.
BUG=none
TEST=unittests

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=49852

[Alpha Left Google, 2010-06-14 20:26:06]

small nits, otherwise LGTM.

http://codereview.chromium.org/2810002/diff/2001/1009
File remoting/host/heartbeat_sender.h (right):

http://codereview.chromium.org/2810002/diff/2001/1009#newcode28
remoting/host/heartbeat_sender.h:28: void Start(JingleClient* jingle_client,
const HostConfig* config);
const HostConfig&

http://codereview.chromium.org/2810002/diff/2001/1012
File remoting/host/simple_host.h (right):

http://codereview.chromium.org/2810002/diff/2001/1012#newcode57
remoting/host/simple_host.h:57: SimpleHost(const HostConfig* config, Capturer*
capturer, Encoder* encoder,
const HostConfig& config

[Sergey Ulanov, 2010-06-14 20:43:20]

http://codereview.chromium.org/2810002/diff/2001/1009
File remoting/host/heartbeat_sender.h (right):

http://codereview.chromium.org/2810002/diff/2001/1009#newcode28
remoting/host/heartbeat_sender.h:28: void Start(JingleClient* jingle_client,
const HostConfig* config);
On 2010/06/14 20:26:06, Alpha wrote:
> const HostConfig&
I believe it is better to use pointer here instead of reference because the
object saves the pointer after Start() returns. That would not be visible enough
if I used reference here. I've changed order of arguments for this method.

http://codereview.chromium.org/2810002/diff/2001/1012
File remoting/host/simple_host.h (right):

http://codereview.chromium.org/2810002/diff/2001/1012#newcode57
remoting/host/simple_host.h:57: SimpleHost(const HostConfig* config, Capturer*
capturer, Encoder* encoder,
On 2010/06/14 20:26:06, Alpha wrote:
> const HostConfig& config
Same here, I believe pointer is more adequate here.

[Alpha Left Google, 2010-06-15 01:11:49]

LGTM.