fuzz: list API fuzzing options if -t api and -n matches nothing.

fuzz: list API fuzzing options if -t api and -n matches nothing.

Today we segfault if --name is empty.  This fixes that too.

This updates some terms: -t api lets us fuzz an API.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1617713003

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

[mtklein_C, 2016-01-21 13:51:21]

Description was changed from

==========
fuzz: list API fuzzing options if -t api and -n matches nothing.

Today we segfault if --name is empty.  This fixes that too.

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

to

==========
fuzz: list API fuzzing options if -t api and -n matches nothing.

Today we segfault if --name is empty.  This fixes that too.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[mtklein_C, 2016-01-21 13:51:56]

Description was changed from

==========
fuzz: list API fuzzing options if -t api and -n matches nothing.

Today we segfault if --name is empty.  This fixes that too.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
fuzz: list API fuzzing options if -t api and -n matches nothing.

Today we segfault if --name is empty.  This fixes that too.

This updates some terms: -t api lets us fuzz an API.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[mtklein_C, 2016-01-21 13:52:46]

mtklein@chromium.org changed reviewers:
+ kjlubick@google.com

[mtklein_C, 2016-01-21 13:52:51]

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

[commit-bot: I haz the power, 2016-01-21 13:53:01]

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

[kjlubick, 2016-01-21 13:57:31]

https://codereview.chromium.org/1617713003/diff/1/fuzz/fuzz.cpp
File fuzz/fuzz.cpp (right):

https://codereview.chromium.org/1617713003/diff/1/fuzz/fuzz.cpp#newcode50
fuzz/fuzz.cpp:50: switch (FLAGS_type[0][0]) {
This segfaults if --type is empty.  Can you print the usage if --type is empty?

[mtklein, 2016-01-21 13:59:35]

On 2016/01/21 13:57:31, kjlubick wrote:
> https://codereview.chromium.org/1617713003/diff/1/fuzz/fuzz.cpp
> File fuzz/fuzz.cpp (right):
> 
> https://codereview.chromium.org/1617713003/diff/1/fuzz/fuzz.cpp#newcode50
> fuzz/fuzz.cpp:50: switch (FLAGS_type[0][0]) {
> This segfaults if --type is empty.  Can you print the usage if --type is
empty?

Unlike --name, type's default doesn't segfault.

[mtklein, 2016-01-21 14:00:53]

mtklein@google.com changed reviewers:
+ mtklein@google.com

[mtklein, 2016-01-21 14:00:53]

https://codereview.chromium.org/1617713003/diff/1/fuzz/fuzz.cpp
File fuzz/fuzz.cpp (right):

https://codereview.chromium.org/1617713003/diff/1/fuzz/fuzz.cpp#newcode50
fuzz/fuzz.cpp:50: switch (FLAGS_type[0][0]) {
On 2016/01/21 13:57:31, kjlubick wrote:
> This segfaults if --type is empty.  Can you print the usage if --type is
empty?

Done.

[kjlubick, 2016-01-21 14:02:35]

On 2016/01/21 at 14:00:53, mtklein wrote:
> https://codereview.chromium.org/1617713003/diff/1/fuzz/fuzz.cpp
> File fuzz/fuzz.cpp (right):
> 
> https://codereview.chromium.org/1617713003/diff/1/fuzz/fuzz.cpp#newcode50
> fuzz/fuzz.cpp:50: switch (FLAGS_type[0][0]) {
> On 2016/01/21 13:57:31, kjlubick wrote:
> > This segfaults if --type is empty.  Can you print the usage if --type is
empty?
> 
> Done.

lgtm

[mtklein, 2016-01-21 14:03:09]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2016-01-21 14:03:17]

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

[commit-bot: I haz the power, 2016-01-21 14:13:54]

Description was changed from

==========
fuzz: list API fuzzing options if -t api and -n matches nothing.

Today we segfault if --name is empty.  This fixes that too.

This updates some terms: -t api lets us fuzz an API.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
fuzz: list API fuzzing options if -t api and -n matches nothing.

Today we segfault if --name is empty.  This fixes that too.

This updates some terms: -t api lets us fuzz an API.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

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

[commit-bot: I haz the power, 2016-01-21 14:13:55]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/d4387ea99369ee1b89bbc9ffc0d8c203a8923515