Use stricter type checking in UMA_HISTOGRAM_ENUMERATION

components/leveldb/public/cpp/util.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/leveldb/public/cpp/util.h"
 
 #include "third_party/leveldatabase/env_chromium.h"
 #include "third_party/leveldatabase/src/include/leveldb/status.h"
 
 namespace leveldb {
@@ skipping 28 matching lines @@
       return leveldb::Status::InvalidArgument(msg, msg2);
     case mojom::DatabaseError::IO_ERROR:
       return leveldb::Status::IOError(msg, msg2);
   }
 
   // This will never be reached, but we still have configurations which don't
   // do switch enum checking.
   return leveldb::Status::InvalidArgument(msg, msg2);
 }
 
-int GetLevelDBStatusUMAValue(mojom::DatabaseError status) {
+leveldb_env::LevelDBStatusValue GetLevelDBStatusUMAValue(
+    mojom::DatabaseError status) {
   switch (status) {
     case mojom::DatabaseError::OK:
       return leveldb_env::LEVELDB_STATUS_OK;
     case mojom::DatabaseError::NOT_FOUND:
       return leveldb_env::LEVELDB_STATUS_NOT_FOUND;
     case mojom::DatabaseError::CORRUPTION:
       return leveldb_env::LEVELDB_STATUS_CORRUPTION;
     case mojom::DatabaseError::NOT_SUPPORTED:
       return leveldb_env::LEVELDB_STATUS_NOT_SUPPORTED;
     case mojom::DatabaseError::INVALID_ARGUMENT:
@@ skipping 23 matching lines @@
 std::string Uint8VectorToStdString(const std::vector<uint8_t>& input) {
   return std::string(reinterpret_cast<const char*>(input.data()), input.size());
 }
 
 std::vector<uint8_t> StdStringToUint8Vector(const std::string& input) {
   const uint8_t* data = reinterpret_cast<const uint8_t*>(input.data());
   return std::vector<uint8_t>(data, data + input.size());
 }
 
 }  // namespace leveldb