Not Robbie, but yes, that sounds right.

Excellent :-)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/c0b4818f62ae008fa858794ac278ce7f65cdf92d

commit c0b4818f62ae008fa858794ac278ce7f65cdf92d
Author: qyearsley <qyearsley@chromium.org>
Date: Mon Oct 17 17:31:05 2016

In archive_layout_test_results, only support archiving to GS.

BUG= 655810 

Review-Url: https://codereview.chromium.org/2414153003

[modify] https://crrev.com/c0b4818f62ae008fa858794ac278ce7f65cdf92d/scripts/slave/chromium/archive_layout_test_results.py
[modify] https://crrev.com/c0b4818f62ae008fa858794ac278ce7f65cdf92d/scripts/slave/chromium/archive_layout_test_results_unittest.py

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/c0b4818f62ae008fa858794ac278ce7f65cdf92d

commit c0b4818f62ae008fa858794ac278ce7f65cdf92d
Author: qyearsley <qyearsley@chromium.org>
Date: Mon Oct 17 17:31:05 2016

In archive_layout_test_results, only support archiving to GS.

BUG= 655810 

Review-Url: https://codereview.chromium.org/2414153003

[modify] https://crrev.com/c0b4818f62ae008fa858794ac278ce7f65cdf92d/scripts/slave/chromium/archive_layout_test_results.py
[modify] https://crrev.com/c0b4818f62ae008fa858794ac278ce7f65cdf92d/scripts/slave/chromium/archive_layout_test_results_unittest.py

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/0885bed6d8ed3456c579460b73a65f22935f0e07

commit 0885bed6d8ed3456c579460b73a65f22935f0e07
Author: recipe-roller <recipe-roller@chromium.org>
Date: Mon Oct 17 18:13:47 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/c0b4818f62ae008fa858794ac278ce7f65cdf92d In archive_layout_test_results, only support archiving to GS. (qyearsley@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 655810 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2429493003

[modify] https://crrev.com/0885bed6d8ed3456c579460b73a65f22935f0e07/infra/config/recipes.cfg

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/829daf60e51ee9e6c78bea29918def500427ac76

commit 829daf60e51ee9e6c78bea29918def500427ac76
Author: recipe-roller <recipe-roller@chromium.org>
Date: Mon Oct 17 22:31:12 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/69fc06f12bb5fc602161bd2169cb54a03fe5634b Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/61728af22d26f33d09ed715d0c43f6051ee9c1ea Fix compile on the Blimp Client Engine Integration bot (shenghuazhang@chromium.org)
  https://crrev.com/f7b18bcd210f254d4b61c92b2a94942fe9b83927 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/dd839c91b9b2e7f1b45683dfe1e7ac68f01157a6 Modify 'build_dir' for run_mb() to avoid double 'Debug' (shenghuazhang@chromium.org)
  https://crrev.com/7eea10901de0274a5be22fdf6479e1109582900a Skia trigger recipe: change remote recipe condition (borenet@chromium.org)
  https://crrev.com/0b094b378658afa9436b1107859128456cd1c08d Add GomaCanaryiOS to chromium.fyi (smut@google.com)
  https://crrev.com/97367d4e07794d107d6075ff86ce898631169e73 Add option to use goma module in chromium.compile (tikuta@chromium.org)
  https://crrev.com/d44afde07794016523aceefdd1de603fc2f75459 V8: Run simple-leak-check step on release and trybot (machenbach@chromium.org)
  https://crrev.com/92dfa57e7376ea9ee60ce378cf16ae290cd9aea3 V8: Bump timeout on arm64 sim nosnap (machenbach@chromium.org)
  https://crrev.com/6b546c5afd0b236ef02e3dfb029d2d630582786c Revert of V8: Run simple-leak-check step on release and trybot (patchset #1 id:1 of https://codereview.chromium.org/2415243003/ ) (machenbach@chromium.org)
  https://crrev.com/94aa3062f0f4cc2542d375cf45706c7e2742c714 libyuv: Switch bots over to GN by default. (kjellander@chromium.org)
  https://crrev.com/ab47d254ddf187fa3b3c0231ea1882557b589f7b Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/3d9ee695a52c6ea7c00b60ae7ce8de03bbf626ed libyuv: Switch {client,tryserver}.libyuv to GN (kjellander@chromium.org)
  https://crrev.com/ded9582695d74387b613dcbcad04936bff12d88b libyuv: Disable signing for iOS (kjellander@chromium.org)
  https://crrev.com/1a9cedbf5751bb8b7d4a59ef43f9618b87b0b97f Updating flags necessary for upload_perf_dashboard_results.py (eyaich@chromium.org)
  https://crrev.com/1088f8337678d51a96fa44917bc9efc64270b0e4 Add the Syzygy x64 bots to the Syzygy waterfall. (sebmarchand@chromium.org)
  https://crrev.com/7ae96413206814a587e3e5921017762be6257983 Use the Chromium-side script to run the PGO benchmarks. (sebmarchand@chromium.org)
  https://crrev.com/02bbb17974a70e1bb198ee4243d830ff258d1bb8 Add IGNORE_DEPS_CHANGES flag and coverage builder. (dnj@chromium.org)
  https://crrev.com/78c3859a73773c4d7eff4ed38c796c147bae998c Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/c0b4818f62ae008fa858794ac278ce7f65cdf92d In archive_layout_test_results, only support archiving to GS. (qyearsley@chromium.org)
  https://crrev.com/65dc88015fad41b08d59d3f47ce157c3d5f8b051 V8: Add valgrind deps to linux64 bots (machenbach@chromium.org)
  https://crrev.com/9f623ff560eeca917efb9b799a1ca3137f9dcaa0 Revert of Add IGNORE_DEPS_CHANGES flag and coverage builder. (patchset #3 id:40001 of https://codereview.chromium.org/2418233002/ ) (dnj@chromium.org)
  https://crrev.com/124f02dd3006293789d17557aea29dfbb8f911e0 Remove use of "use_mirror" recipe property. (dnj@chromium.org)
  https://crrev.com/13c03fad59862d90b2b6bb2372a37ab414074228 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/2d442327bbfad0eb211466afbfe2f5a17c4561da Stop archiving redundant copy of {full,failing}_results.json. (qyearsley@chromium.org)
  https://crrev.com/506376486f9bd8973f5b96a02eddd449a3ef52e2 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
depot_tools:
  https://crrev.com/c9ebbd24639c8fb9f7fd2a36032614830e1e5285 Fix logic to detect Gerrit CL in CQ (rmistry@google.com)
  https://crrev.com/1dbb595dbfa4ebf73ee036cd66275af24253a99d Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/48df58136dc1847cd158eaa330f5dd457265d654 Gerrit git cl: make it work with minimal file. (tandrii@chromium.org)
  https://crrev.com/d437d76b64e083f2fb2ff3a7b38638d06642e270 Reland "Remove SVN and File support from gclient" (agable@chromium.org)
  https://crrev.com/aa9e16acd4e74f35dc738b98cd03b2c0b30bfefb Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/1c822ade1f731ec35a3d2d3be8805f773df22c4c Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
recipe_engine:
  https://crrev.com/a15c8474d308784e94fe4e15deb7c7860d717059 Reland: Add post_process hooks to recipe testing facilities. (iannucci@chromium.org)
  https://crrev.com/7076377b62701f77783d2d5e002fe2708ffab273 Fix VerifySubset when comparing empty lists (martiniss@chromium.org)
  https://crrev.com/bb6caf8060139fd4bcefee43f54ae00150e7ce3a Remove 'use_mirror' and 'root' properties. (dnj@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 656555 , chromium:656741 ,654806, chromium:656731 ,641588,chromium:633253, 656573 , 655810 , chromium:652188 , 655864 ,chromium:648308, chromium:656042 , 309849 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2422383003
Cr-Commit-Position: refs/heads/master@{#425793}

[modify] https://crrev.com/829daf60e51ee9e6c78bea29918def500427ac76/infra/config/recipes.cfg

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/ece187bdc4c389344de942753c06aaa69d0e880e

commit ece187bdc4c389344de942753c06aaa69d0e880e
Author: qyearsley <qyearsley@chromium.org>
Date: Tue Oct 18 20:42:48 2016

In archive_layout_test_results.py, simplify argument handling.

As far as I know, archive_layout_test_results.py is currently
always called from the chromium_tests recipe module, after running
layout tests, for the "archive_webkit_tests_results" step. In
practice, this is always called like this:

archive_webkit_tests_results
python -u .../archive_layout_test_results.py \
  --results-dir /b/rr/tmpa5gR1b/w/layout-test-results \
  --build-dir /b/c/b/mac_layout/src/out \
  --build-number 25264 \
  --builder-name 'WebKit Mac10.10' \
  --gs-bucket gs://chromium-layout-test-archives \
  --staging-dir /b/c/chrome_staging

That is, results-dir is always given and is an absolute path,
and build number and builder name are always given.

In this CL:
 - use argparse rather than optparse
 - remove --build-properties and --factory-properties args
 - make build number and builder name required
 - make results dir required, and don't treat it as a
   relative path relative to the build dir (since in practice
   it isn't).

BUG= 655810 

Review-Url: https://codereview.chromium.org/2429583002

[modify] https://crrev.com/ece187bdc4c389344de942753c06aaa69d0e880e/scripts/slave/chromium/archive_layout_test_results.py

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/d9c9f884bd6c8d67c78230e2af7e50f3815996ec

commit d9c9f884bd6c8d67c78230e2af7e50f3815996ec
Author: recipe-roller <recipe-roller@chromium.org>
Date: Tue Oct 18 20:55:26 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5e7f9212eea94ed34e3a5755a2ed6d3f1eb3990e

commit 5e7f9212eea94ed34e3a5755a2ed6d3f1eb3990e
Author: recipe-roller <recipe-roller@chromium.org>
Date: Tue Oct 18 21:07:10 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/ece187bdc4c389344de942753c06aaa69d0e880e In archive_layout_test_results.py, simplify argument handling. (qyearsley@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 655810 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2428123003
Cr-Commit-Position: refs/heads/master@{#426045}

[modify] https://crrev.com/5e7f9212eea94ed34e3a5755a2ed6d3f1eb3990e/infra/config/recipes.cfg

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/f67ef4c2d0648a3310b44d178312ac55007b52e0

commit f67ef4c2d0648a3310b44d178312ac55007b52e0
Author: recipe-roller <recipe-roller@chromium.org>
Date: Tue Oct 18 22:30:26 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/9005b9309742d2a60d08738069909c4eaf6569aa Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/ece187bdc4c389344de942753c06aaa69d0e880e In archive_layout_test_results.py, simplify argument handling. (qyearsley@chromium.org)
depot_tools:
  https://crrev.com/78de30e1bb2a3065ea78c28305be6782cc7c4e43 Revert "git_cl: Allow server overriding." (dnj@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 655810 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2423293004

[modify] https://crrev.com/f67ef4c2d0648a3310b44d178312ac55007b52e0/infra/config/recipes.cfg

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/e22a1387a6f865be9becbc903fcc16d5d05b6a36

commit e22a1387a6f865be9becbc903fcc16d5d05b6a36
Author: qyearsley <qyearsley@chromium.org>
Date: Wed Oct 19 17:10:31 2016

Also archive diff files in layout-test-results.zip.

This is a followup to http://crrev.com/2414153003.
The purpose is to add diff files into the zip archives
along with actual result files.

BUG= 656684 , 655810 

Review-Url: https://chromiumcodereview.appspot.com/2429573002

[modify] https://crrev.com/e22a1387a6f865be9becbc903fcc16d5d05b6a36/scripts/slave/chromium/archive_layout_test_results.py
[modify] https://crrev.com/e22a1387a6f865be9becbc903fcc16d5d05b6a36/scripts/slave/chromium/archive_layout_test_results_unittest.py

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/a742ce6396354d7b63bcb8f09e9d8db03132f5fd

commit a742ce6396354d7b63bcb8f09e9d8db03132f5fd
Author: recipe-roller <recipe-roller@chromium.org>
Date: Wed Oct 19 17:16:01 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/ba3cd0acc9525cfc6bf5bf0980dd6292397f22cd

commit ba3cd0acc9525cfc6bf5bf0980dd6292397f22cd
Author: recipe-roller <recipe-roller@chromium.org>
Date: Wed Oct 19 17:29:17 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/e22a1387a6f865be9becbc903fcc16d5d05b6a36 Also archive diff files in layout-test-results.zip. (qyearsley@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 656684 , 655810 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://chromiumcodereview.appspot.com/2433873004

[modify] https://crrev.com/ba3cd0acc9525cfc6bf5bf0980dd6292397f22cd/infra/config/recipes.cfg

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ed7a15dfc0751bdf2ede6935a560f91af3db581d

commit ed7a15dfc0751bdf2ede6935a560f91af3db581d
Author: recipe-roller <recipe-roller@chromium.org>
Date: Wed Oct 19 17:37:14 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/e22a1387a6f865be9becbc903fcc16d5d05b6a36 Also archive diff files in layout-test-results.zip. (qyearsley@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 656684 , 655810 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://chromiumcodereview.appspot.com/2429253003
Cr-Commit-Position: refs/heads/master@{#426237}

[modify] https://crrev.com/ed7a15dfc0751bdf2ede6935a560f91af3db581d/infra/config/recipes.cfg