Skia recipe: Use [depot_tools]/gsutil.py instead of [depot_tools]/third_party/gsutil/gsutil

Skia recipe: Use [depot_tools]/gsutil.py instead of [depot_tools]/third_party/gsutil/gsutil

The latter is being removed in https://codereview.chromium.org/2280023003/
Is also pinned to 3.25, whereas the former is updated more often.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2298293002

Committed: https://skia.googlesource.com/skia/+/f15590f803596f51d0950994b2aeb623d2d1fc24

[Ryan Tseng, 2016-08-31 17:29:29]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[Ryan Tseng, 2016-08-31 17:29:29]

Description was changed from

==========
Skia recipe: Use [depot_tools]/gsutil.py instead of
[depot_tools]/third_party/gsutil/gsutil

The latter is being removed in https://codereview.chromium.org/2280023003/
Is also pinned to 3.25, whereas the former is updated more often.

BUG=skia:
==========

to

==========
Skia recipe: Use [depot_tools]/gsutil.py instead of
[depot_tools]/third_party/gsutil/gsutil

The latter is being removed in https://codereview.chromium.org/2280023003/
Is also pinned to 3.25, whereas the former is updated more often.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2298293002
==========

[commit-bot: I haz the power, 2016-08-31 17:29:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ryan Tseng, 2016-08-31 17:29:48]

hinoka@google.com changed reviewers:
+ borenet@google.com

[Ryan Tseng, 2016-08-31 17:29:49]

ptal, now on the skia side

[commit-bot: I haz the power, 2016-08-31 17:30:21]

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

[commit-bot: I haz the power, 2016-08-31 17:30:22]

Dry run: Try jobs failed on following builders:
  Build-Ubuntu-GCC-Arm64-Debug-Android-Trybot on master.client.skia.compile
(JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm...)
  Build-Ubuntu-GCC-Arm7-Debug-Android-Trybot on master.client.skia.compile
(JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm...)
  Build-Ubuntu-GCC-Mips-Debug-Android-Trybot on master.client.skia.compile
(JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Mip...)

[borenet, 2016-08-31 17:33:25]

LGTM. Aside: CQ is broken due to
https://bugs.chromium.org/p/chromium/issues/detail?id=642796

[Ryan Tseng, 2016-09-19 18:29:54]

The CQ bit was checked by hinoka@google.com

[commit-bot: I haz the power, 2016-09-19 18:30:05]

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, 2016-09-19 19:18:56]

Description was changed from

==========
Skia recipe: Use [depot_tools]/gsutil.py instead of
[depot_tools]/third_party/gsutil/gsutil

The latter is being removed in https://codereview.chromium.org/2280023003/
Is also pinned to 3.25, whereas the former is updated more often.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2298293002
==========

to

==========
Skia recipe: Use [depot_tools]/gsutil.py instead of
[depot_tools]/third_party/gsutil/gsutil

The latter is being removed in https://codereview.chromium.org/2280023003/
Is also pinned to 3.25, whereas the former is updated more often.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2298293002

Committed:
https://skia.googlesource.com/skia/+/f15590f803596f51d0950994b2aeb623d2d1fc24
==========

[commit-bot: I haz the power, 2016-09-19 19:18:57]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/f15590f803596f51d0950994b2aeb623d2d1fc24

[borenet, 2016-09-20 19:33:06]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2355943002/ by borenet@google.com.

The reason for reverting is: Broke all nanobench uploads, eg.
https://uberchromegw.corp.google.com/i/client.skia.android/builders/Perf-Andr....

[hinoka, 2016-09-20 19:44:47]

hinoka@chromium.org changed reviewers:
+ hinoka@chromium.org

[hinoka, 2016-09-20 19:44:47]

A straight revert may be problematic, I've rolled out the depot_tools side
change.

The running script will have to pass in "--" to separate args to the wrapper
from args to the real gsutil.  I'll send out another CL shortly