Blacklist more characters when sending a try job.

Blacklist more characters when sending a try job.

Otherwise files containing these characters can be created on the file system,
making it harder to get rid of those files.

TEST=none
BUG=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=64782

[M-A Ruel, 2010-11-02 13:51:47]

Adding Jochen, whoever is the faster to review. :)

[jochen (gone - plz use gerrit), 2010-11-02 18:51:46]

http://codereview.chromium.org/4127014/diff/1/2
File trychange.py (right):

http://codereview.chromium.org/4127014/diff/1/2#newcode98
trychange.py:98: .replace(':', '-'))
hum, what about a whitelist instead of a blacklist? [a-zA-Z0-9_-] are kept and
the rest is mapped to -?

[M-A Ruel, 2010-11-02 18:53:34]

http://codereview.chromium.org/4127014/diff/1/2
File trychange.py (right):

http://codereview.chromium.org/4127014/diff/1/2#newcode98
trychange.py:98: .replace(':', '-'))
On 2010/11/02 18:51:46, jochen wrote:
> hum, what about a whitelist instead of a blacklist? [a-zA-Z0-9_-] are kept and
> the rest is mapped to -?

and all our nice utf-8 characters !?!? :D

Ok I can switch to a whitelist.

[M-A Ruel, 2010-11-02 19:12:41]

Use a white list instead of a black list and fixed unit test.

[jochen (gone - plz use gerrit), 2010-11-02 19:12:50]

lgtm, thanks