fiddle: Don't overwrite existing fiddle names unintentionally.

fiddle: Don't overwrite existing fiddle names unintentionally.

BUG=skia:
TBR=rmistry

Committed: https://skia.googlesource.com/buildbot/+/2f0e606a02a22e8671d721c2c1a32e6ef79c1e5a

[jcgregorio, 2016-04-22 14:53:57]

Description was changed from

==========
fiddle: Don't overwrite existing fiddle names unintentionally.

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

to

==========
fiddle: Don't overwrite existing fiddle names unintentionally.

BUG=skia:
TBR=rmistry
==========

[jcgregorio, 2016-04-22 14:53:58]

jcgregorio@google.com changed reviewers:
+ rmistry@google.com

[jcgregorio, 2016-04-22 14:54:04]

The CQ bit was checked by jcgregorio@google.com

[commit-bot: I haz the power, 2016-04-22 14:54:12]

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

[commit-bot: I haz the power, 2016-04-22 14:59:28]

Description was changed from

==========
fiddle: Don't overwrite existing fiddle names unintentionally.

BUG=skia:
TBR=rmistry
==========

to

==========
fiddle: Don't overwrite existing fiddle names unintentionally.

BUG=skia:
TBR=rmistry

Committed:
https://skia.googlesource.com/buildbot/+/2f0e606a02a22e8671d721c2c1a32e6ef79c...
==========

[commit-bot: I haz the power, 2016-04-22 14:59:29]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/buildbot/+/2f0e606a02a22e8671d721c2c1a32e6ef79c...

[rmistry, 2016-04-25 12:59:18]

lgtm

https://codereview.chromium.org/1912793002/diff/80001/fiddle/go/named/named.go
File fiddle/go/named/named.go (right):

https://codereview.chromium.org/1912793002/diff/80001/fiddle/go/named/named.g...
fiddle/go/named/named.go:75: if err == nil {
I had to look at this a few times to see what it was doing. Looks like line 74
returns an err if the name does not exist yet. If there is no error means that a
name already exists. Either way its ok to write has long as overwrite is True.