Remove brackets from gm names.

Remove brackets from gm names.

BUG=skia:2634
Committed: https://skia.googlesource.com/skia/+/ef470f185f0dcd94943d0c470ddf8cc109f30325

[borenet, 2014-06-03 13:17:26]

LGTM but wishing we had an easy pre-flight to make sure we don't accidentally
add more.

[scroggo, 2014-06-03 13:41:03]

On 2014/06/03 13:17:26, borenet wrote:
> LGTM but wishing we had an easy pre-flight to make sure we don't accidentally
> add more.

You mean to make sure we don't add more in the future? I double checked this
one,
and realized that I left a couple brackets. Patch set two removes those, while
leaving any others that I looked over....

[scroggo, 2014-06-03 13:41:09]

The CQ bit was checked by scroggo@google.com

[commit-bot: I haz the power, 2014-06-03 13:41:22]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/scroggo@google.com/307373003/20001

[commit-bot: I haz the power, 2014-06-03 13:50:30]

Change committed as ef470f185f0dcd94943d0c470ddf8cc109f30325

[epoger, 2014-06-03 13:52:47]

On 2014/06/03 13:17:26, borenet wrote:
> LGTM but wishing we had an easy pre-flight to make sure we don't accidentally
> add more.

At least for GMs, when a new GM test is added it won't have any expectations. 
So it will upload its results on first run for sure, triggering this error.

I think a new test like you describe in
https://code.google.com/p/skia/issues/detail?id=2634#c15 ("running each of the
binaries which produce files, forcing them to always write everything to files,
and then asserting that all of the produced files had valid names") is a good
idea, but I don't know where it fits in the global priority scheme.  Filed
http://skbug.com/2636 ('add unittest for all our tools to confirm that filenames
written out are legal') for that idea.