Moving service urls to base.

Moving service urls to base.

BUG=699788
Review-Url: https://codereview.chromium.org/2779373003
Cr-Commit-Position: refs/heads/master@{#461246}
Committed: https://chromium.googlesource.com/chromium/src/+/a36b97239e429ff829dd785fc8a3f9b5f7d145e9

[nicholss, 2017-03-29 22:29:07]

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

[nicholss, 2017-03-29 22:29:28]

Description was changed from

==========
Moving service urls to base.

BUG=699788
==========

to

==========
Moving service urls to base.

BUG=699788
==========

[nicholss, 2017-03-29 22:29:28]

nicholss@chromium.org changed reviewers:
+ sergeyu@chromium.org

[commit-bot: I haz the power, 2017-03-29 22:30:11]

Dry run: 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, 2017-03-29 22:47:39]

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

[commit-bot: I haz the power, 2017-03-29 22:47:40]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[nicholss, 2017-03-29 23:10:23]

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

[commit-bot: I haz the power, 2017-03-29 23:11:36]

Dry run: 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, 2017-03-29 23:27:24]

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

[commit-bot: I haz the power, 2017-03-29 23:27:25]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[nicholss, 2017-03-29 23:40:21]

On 2017/03/29 23:27:25, commit-bot: I haz the power wrote:
> Dry run: Try jobs failed on following builders:
>   linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
> (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

I am not sure how to resolve this, pnacl will not link with net, but pnacl will
also not link without IsValidBotJid which came from remoting_bot, but got merged
into service_urls, which needs net to build.

Should I back out the change to remove remoting_bot? Move the service urls into
the signaling dir? (whatever that is for)

[nicholss, 2017-03-30 16:56:35]

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

[commit-bot: I haz the power, 2017-03-30 16:57:07]

Dry run: 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, 2017-03-30 18:30:31]

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

[commit-bot: I haz the power, 2017-03-30 18:30:32]

Dry run: This issue passed the CQ dry run.

[Sergey Ulanov, 2017-03-31 18:38:49]

lgtm

https://codereview.chromium.org/2779373003/diff/40001/remoting/base/service_u...
File remoting/base/service_urls.cc (right):

https://codereview.chromium.org/2779373003/diff/40001/remoting/base/service_u...
remoting/base/service_urls.cc:1: // Copyright 2016 The Chromium Authors. All
rights reserved.
update year

https://codereview.chromium.org/2779373003/diff/40001/remoting/base/service_u...
File remoting/base/service_urls.h (right):

https://codereview.chromium.org/2779373003/diff/40001/remoting/base/service_u...
remoting/base/service_urls.h:1: // Copyright 2016 The Chromium Authors. All
rights reserved.
update year

[nicholss, 2017-03-31 20:51:00]

The CQ bit was checked by nicholss@chromium.org

[nicholss, 2017-03-31 20:51:00]

The patchset sent to the CQ was uploaded after l-g-t-m from sergeyu@chromium.org
Link to the patchset: https://codereview.chromium.org/2779373003/#ps60001
(title: "Updating moved file c dates.")

[commit-bot: I haz the power, 2017-03-31 20:51:48]

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, 2017-03-31 22:12:32]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490993460443660,
"parent_rev": "7cfd3f9eb9dd016c64853f1ae8af84980f9d6f64", "commit_rev":
"a36b97239e429ff829dd785fc8a3f9b5f7d145e9"}

[commit-bot: I haz the power, 2017-03-31 22:13:18]

Description was changed from

==========
Moving service urls to base.

BUG=699788
==========

to

==========
Moving service urls to base.

BUG=699788

Review-Url: https://codereview.chromium.org/2779373003
Cr-Commit-Position: refs/heads/master@{#461246}
Committed:
https://chromium.googlesource.com/chromium/src/+/a36b97239e429ff829dd785fc8a3...
==========

[commit-bot: I haz the power, 2017-03-31 22:13:23]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/a36b97239e429ff829dd785fc8a3...