[LevelDBScopes] Making all scopes lock ranges bytewise compared

The locking system of LevelDB scopes doesn't need to match the key
system of the backing leveldb database. Tieing them together means that
the leveldb comparator has to handle both leveldb and lock keys, and
these aren't necessarily the same.

To avoid making the comparator slower for leveldb by adding more logic,
make the ScopesLockManager locks always be bytewise-compared.

See this doc for more context:
https://docs.google.com/document/d/1jX6OOQJ0aLrWAeqYLtARXPFawUlJPbHNIKAK-0vmFHU/edit#heading=h.yifj0a13oqdj

Bug: 862456
Change-Id: I56ec1f7328eadacda91c68a64047e0d84e458376
Reviewed-on: https://chromium-review.googlesource.com/c/1406288
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#621860}
6 files changed