Pass rietveld as the default patch_storage.

Pass rietveld as the default patch_storage.

Passing around empty strings as command line arguments requires
us to be really careful about command line quoting, parsing, etc.

It seems easy enough here to just supply the implied value.

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

Committed: https://skia.googlesource.com/skia/+/0973a371ecb55a97d84d18c236df6180bca41e6d

[mtklein_C, 2016-09-09 18:59:55]

Description was changed from

==========
Pass rietveld as the default patch_storage.

Empty string parameters like we're passing here before this CL mean we
have to be particularly careful about quoting.  Seems easy enough to
just supply the implied value.

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

to

==========
Pass rietveld as the default patch_storage.

Empty string parameters like we're passing here before this CL mean we
have to be particularly careful about quoting.  Seems easy enough to
just supply the implied value.

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

[mtklein_C, 2016-09-09 19:02:39]

Description was changed from

==========
Pass rietveld as the default patch_storage.

Empty string parameters like we're passing here before this CL mean we
have to be particularly careful about quoting.  Seems easy enough to
just supply the implied value.

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

to

==========
Pass rietveld as the default patch_storage.

Passing around empty strings as command line arguments requires
us to be really careful about command line quoting, parsing, etc.

It seems easy enough here to just supply the implied value.

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

[mtklein_C, 2016-09-09 19:08:50]

mtklein@chromium.org changed reviewers:
+ jcgregorio@google.com, rmistry@google.com

[mtklein_C, 2016-09-09 19:08:50]

The CQ bit was checked by mtklein@chromium.org

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

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-09 19:08:59]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2016-09-10 01:08 UTC

[commit-bot: I haz the power, 2016-09-10 01:08:56]

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

[commit-bot: I haz the power, 2016-09-10 01:08:58]

No LGTM from a valid reviewer yet. Please ask for an LGTM from a
full Skia committer

[rmistry, 2016-09-10 02:01:26]

lgtm

[mtklein_C, 2016-09-15 15:04:15]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-09-15 15:04:18]

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-15 15:27:40]

Description was changed from

==========
Pass rietveld as the default patch_storage.

Passing around empty strings as command line arguments requires
us to be really careful about command line quoting, parsing, etc.

It seems easy enough here to just supply the implied value.

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

to

==========
Pass rietveld as the default patch_storage.

Passing around empty strings as command line arguments requires
us to be really careful about command line quoting, parsing, etc.

It seems easy enough here to just supply the implied value.

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

Committed:
https://skia.googlesource.com/skia/+/0973a371ecb55a97d84d18c236df6180bca41e6d
==========

[commit-bot: I haz the power, 2016-09-15 15:27:41]

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