More cleanups in remoting.gyp

More cleanups in remoting.gyp

- Merged remoting_jingle_glue into remoting_protocol.
- Merged remoting_host_logging and remoting_host_event_logger
  into remoting_host.

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

[Lambros, 2014-01-25 00:31:58]

lgtm

https://codereview.chromium.org/145003019/diff/1/remoting/remoting.gyp
File remoting/remoting.gyp (right):

https://codereview.chromium.org/145003019/diff/1/remoting/remoting.gyp#newcod...
remoting/remoting.gyp:326: 'host/daemon_process.cc',
These files were Windows-only, but will now get pulled into all hosts on all
platforms. I guess it's OK because the buildbots were still compiling all these
files on all platforms anyway for remoting_unittests. And the linker should
discard this unused code when linking the host binaries. Still, it will slightly
lengthen the link time if you're just building a host. I assume we're OK with
this?

[Sergey Ulanov, 2014-01-25 01:24:15]

https://codereview.chromium.org/145003019/diff/1/remoting/remoting.gyp
File remoting/remoting.gyp (right):

https://codereview.chromium.org/145003019/diff/1/remoting/remoting.gyp#newcod...
remoting/remoting.gyp:326: 'host/daemon_process.cc',
On 2014/01/25 00:31:59, Lambros wrote:
> These files were Windows-only, but will now get pulled into all hosts on all
> platforms. I guess it's OK because the buildbots were still compiling all
these
> files on all platforms anyway for remoting_unittests. And the linker should
> discard this unused code when linking the host binaries. Still, it will
slightly
> lengthen the link time if you're just building a host. I assume we're OK with
> this?

These files should be cross-platform, but we currently use them only on windows
and linker should strip unused code code from the binaries it produces.
Previously they were compiled twice on windows, which is wrong.

[commit-bot: I haz the power, 2014-01-25 01:24:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sergeyu@chromium.org/145003019/220001

[commit-bot: I haz the power, 2014-01-25 01:48:11]

Retried try job too often on android_clang_dbg for step(s) slave_steps
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[commit-bot: I haz the power, 2014-01-25 02:21:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sergeyu@chromium.org/145003019/430001

[commit-bot: I haz the power, 2014-01-25 02:51:05]

Retried try job too often on android_clang_dbg for step(s) slave_steps
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[commit-bot: I haz the power, 2014-01-25 02:58:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sergeyu@chromium.org/145003019/610001

[commit-bot: I haz the power, 2014-01-25 08:01:33]

Change committed as 247102