fiddle: fix build

fiddle: fix build

Remove the lock from updateCurrent to avoid deadlocking on the mutex.
Also simplify the code by keeping the list of hashes in the same order as they are found in the goodbuilds.txt file.

BUG=skia:
TBR=rmistry

Committed: https://skia.googlesource.com/buildbot/+/628dbb95c47455646bae270e6c1075428adb05fb

[jcgregorio, 2016-04-29 20:24:02]

Description was changed from

==========
fiddle: fix build

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

to

==========
fiddle: fix build

Remove the lock from updateCurrent to avoid deadlocking on the mutex.

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

[jcgregorio, 2016-04-29 20:24:03]

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

[rmistry, 2016-04-29 20:32:36]

lgtm

[jcgregorio, 2016-04-29 21:12:33]

Description was changed from

==========
fiddle: fix build

Remove the lock from updateCurrent to avoid deadlocking on the mutex.

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

to

==========
fiddle: fix build

Remove the lock from updateCurrent to avoid deadlocking on the mutex.
Also simplify the code by keeping the list of hashes in the same order as they
are found in the goodbuilds.txt file.

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

[jcgregorio, 2016-04-29 21:12:49]

The CQ bit was checked by jcgregorio@google.com

[jcgregorio, 2016-04-29 21:12:49]

The patchset sent to the CQ was uploaded after l-g-t-m from rmistry@google.com
Link to the patchset: https://codereview.chromium.org/1933943002/#ps20001
(title: "remove all the reversing")

[commit-bot: I haz the power, 2016-04-29 21:12:55]

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

[commit-bot: I haz the power, 2016-04-29 21:15:38]

Description was changed from

==========
fiddle: fix build

Remove the lock from updateCurrent to avoid deadlocking on the mutex.
Also simplify the code by keeping the list of hashes in the same order as they
are found in the goodbuilds.txt file.

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

to

==========
fiddle: fix build

Remove the lock from updateCurrent to avoid deadlocking on the mutex.
Also simplify the code by keeping the list of hashes in the same order as they
are found in the goodbuilds.txt file.

BUG=skia:
TBR=rmistry

Committed:
https://skia.googlesource.com/buildbot/+/628dbb95c47455646bae270e6c1075428adb...
==========

[commit-bot: I haz the power, 2016-04-29 21:15:39]

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