Custom install me2me host on Linux Swarming bots.

Custom install me2me host, built as part of chromoting-integration-tests, on Linux Swarming bots.

Instead of installing the me2me debian package, we'll temporarily place the required files in a custom location and run a subset of tests with it.
Because the me2me host is launched from a custom location, the local-machine shows up as a remote-machine, so we'll now run connect-to-remote-host test.

Most of the IT2Me tests fail because we have to update the path to the native-messaging-host in the manifest file. I tested with the manifest files thus modified but couldn't get the tests passing on the Swarming bots, so will investigate that separately (noted in tracking bug).
This version should get at least some tests passing again on the Linux builder.

This whole exercise is a temporary solution until we get a proper method in place to update the host. That work is tracked in http://crbug/461920

BUG=517695
Committed: https://crrev.com/ee696a52970133eebae11bf44e769c3552b5b72e
Cr-Commit-Position: refs/heads/master@{#342516}

[anandc, 2015-08-07 21:49:27]

Patchset #1 (id:1) has been deleted

[anandc, 2015-08-07 21:50:41]

anandc@chromium.org changed reviewers:
+ anandc@chromium.org, joedow@chromium.org

[anandc, 2015-08-07 21:50:41]

Hi Joe, PTAL. Thanks.

[joedow, 2015-08-07 22:16:19]

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
File testing/chromoting/browser_test_commands_linux.txt (right):

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
testing/chromoting/browser_test_commands_linux.txt:4: cd
../../remoting/internal/config && /usr/bin/python ./is_valid_json.py
nit: Now that we have several isolate targets, I think the validation checks
should be moved into their own...not a huge deal though as I may be moving these
files into another repo at which point this is moot anyway.

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
File testing/chromoting/browser_tests_launcher.py (right):

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
testing/chromoting/browser_tests_launcher.py:32: ISOLATE_TEMP_FOLDER = ''
Can you init this variable up here instead of using a global?

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
testing/chromoting/browser_tests_launcher.py:162: cwd = os.getcwd()
can you use previous_directory here?  It is no longer the cwd once you change.

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
testing/chromoting/browser_tests_launcher.py:170: # Go back to starting folder.
nit: remove comment, I think it will be obvious if you cwd to the
previous_directory.

[anandc, 2015-08-07 22:26:41]

Thanks Joe. PTAL.

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
File testing/chromoting/browser_test_commands_linux.txt (right):

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
testing/chromoting/browser_test_commands_linux.txt:4: cd
../../remoting/internal/config && /usr/bin/python ./is_valid_json.py
On 2015/08/07 22:16:19, joedow wrote:
> nit: Now that we have several isolate targets, I think the validation checks
> should be moved into their own...not a huge deal though as I may be moving
these
> files into another repo at which point this is moot anyway.

Acknowledged.

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
File testing/chromoting/browser_tests_launcher.py (right):

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
testing/chromoting/browser_tests_launcher.py:32: ISOLATE_TEMP_FOLDER = ''
On 2015/08/07 22:16:19, joedow wrote:
> Can you init this variable up here instead of using a global?

Done.

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
testing/chromoting/browser_tests_launcher.py:162: cwd = os.getcwd()
On 2015/08/07 22:16:19, joedow wrote:
> can you use previous_directory here?  It is no longer the cwd once you change.

Done.

https://codereview.chromium.org/1279823004/diff/20001/testing/chromoting/brow...
testing/chromoting/browser_tests_launcher.py:170: # Go back to starting folder.
On 2015/08/07 22:16:19, joedow wrote:
> nit: remove comment, I think it will be obvious if you cwd to the
> previous_directory.

Done.

[joedow, 2015-08-07 22:33:25]

lgtm

[anandc, 2015-08-07 22:36:31]

The CQ bit was checked by anandc@chromium.org

[commit-bot: I haz the power, 2015-08-07 22:37:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1279823004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1279823004/40001

[commit-bot: I haz the power, 2015-08-08 00:22:03]

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

[commit-bot: I haz the power, 2015-08-08 00:22:04]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[anandc, 2015-08-08 00:24:05]

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

[commit-bot: I haz the power, 2015-08-08 00:24:20]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1279823004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1279823004/40001

[commit-bot: I haz the power, 2015-08-08 01:19:17]

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

[commit-bot: I haz the power, 2015-08-08 01:19:18]

Dry run: This issue passed the CQ dry run.

[anandc, 2015-08-08 04:54:39]

The CQ bit was checked by anandc@chromium.org

[commit-bot: I haz the power, 2015-08-08 04:54:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1279823004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1279823004/40001

[commit-bot: I haz the power, 2015-08-08 04:58:59]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-08-08 04:59:37]

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