More plumbing for logging connection IP addresses

More plumbing for logging connection IP addresses

Followup CL to http://codereview.chromium.org/9271026/ - this passes the
RouteChange() notification from ConnectionToClient through the various layers
to the ChromotingHost.


BUG=109682
TEST=Compiles, unit-tests pass.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=119137

[Lambros, 2012-01-25 01:23:36]

This is just plumbing.  I'm not sure I've chosen good names for some of these
functions.

Next CL will update ChromotingHost to notify its HostStatusObservers, and
finally implement the logging :)

[Sergey Ulanov, 2012-01-25 01:56:48]

LGTM when my nits are addressed.

http://codereview.chromium.org/9288010/diff/1/remoting/host/chromoting_host.h
File remoting/host/chromoting_host.h (right):

http://codereview.chromium.org/9288010/diff/1/remoting/host/chromoting_host.h...
remoting/host/chromoting_host.h:28: class IpEndPoint;
remove this as we discussed

http://codereview.chromium.org/9288010/diff/1/remoting/host/client_session.h
File remoting/host/client_session.h (right):

http://codereview.chromium.org/9288010/diff/1/remoting/host/client_session.h#...
remoting/host/client_session.h:57: virtual void
OnSessionIpEndPoint(ClientSession* client,
OnSessionIpAddress?

http://codereview.chromium.org/9288010/diff/1/remoting/host/host_mock_objects.h
File remoting/host/host_mock_objects.h (right):

http://codereview.chromium.org/9288010/diff/1/remoting/host/host_mock_objects...
remoting/host/host_mock_objects.h:22: class IpEndPoint;
remove this

http://codereview.chromium.org/9288010/diff/1/remoting/protocol/connection_to...
File remoting/protocol/connection_to_client.h (right):

http://codereview.chromium.org/9288010/diff/1/remoting/protocol/connection_to...
remoting/protocol/connection_to_client.h:56: virtual void
OnClientIpEndPoint(ConnectionToClient* connection,
Maybe call it OnClientIpAddress?

[Lambros, 2012-01-25 19:27:10]

http://codereview.chromium.org/9288010/diff/1/remoting/host/chromoting_host.h
File remoting/host/chromoting_host.h (right):

http://codereview.chromium.org/9288010/diff/1/remoting/host/chromoting_host.h...
remoting/host/chromoting_host.h:28: class IpEndPoint;
On 2012/01/25 01:56:48, sergeyu wrote:
> remove this as we discussed

Done.

http://codereview.chromium.org/9288010/diff/1/remoting/host/client_session.h
File remoting/host/client_session.h (right):

http://codereview.chromium.org/9288010/diff/1/remoting/host/client_session.h#...
remoting/host/client_session.h:57: virtual void
OnSessionIpEndPoint(ClientSession* client,
On 2012/01/25 01:56:48, sergeyu wrote:
> OnSessionIpAddress?

Done.

http://codereview.chromium.org/9288010/diff/1/remoting/host/host_mock_objects.h
File remoting/host/host_mock_objects.h (right):

http://codereview.chromium.org/9288010/diff/1/remoting/host/host_mock_objects...
remoting/host/host_mock_objects.h:22: class IpEndPoint;
On 2012/01/25 01:56:48, sergeyu wrote:
> remove this

Done.

http://codereview.chromium.org/9288010/diff/1/remoting/protocol/connection_to...
File remoting/protocol/connection_to_client.h (right):

http://codereview.chromium.org/9288010/diff/1/remoting/protocol/connection_to...
remoting/protocol/connection_to_client.h:56: virtual void
OnClientIpEndPoint(ConnectionToClient* connection,
On 2012/01/25 01:56:48, sergeyu wrote:
> Maybe call it OnClientIpAddress?

Done.

[commit-bot: I haz the power, 2012-01-25 19:27:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/lambroslambrou@chromium.org/9288010/1012

[commit-bot: I haz the power, 2012-01-25 19:54:22]

Try job failure for 9288010-1012 (retry) on win_rel for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-01-25 20:22:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/lambroslambrou@chromium.org/9288010/4003

[commit-bot: I haz the power, 2012-01-25 23:11:25]

Try job failure for 9288010-4003 (retry) on win_rel for step "base_unittests".
It's a second try, previously, step "base_unittests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...