GitLab

CVJ::OnHashesReady seems to be missing lock while accessing member
variable. Add lock to the method. OnHashesReady also calls
BytesRead which already had lock. Turn that method to private
BytesReadImpl, remove lock from it, and provide lock to the
publicly callable version (new BytesRead).
This CL also protects CVJ::time_spent_.

Bug: 814966
Test: Internal change only. See bug for running unit_tests.
Change-Id: I632dccf2ee921d1c84392b44c3ab8e0ca52dde3e
Reviewed-on: https://chromium-review.googlesource.com/933245Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: Istiaque Ahmed <lazyboy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#538970}