Fix 2 issues that were hitting dchecks when enabling ModuleDatabase

chrome/browser/conflicts/module_database_win.cc

 // Copyright 2017 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 "chrome/browser/conflicts/module_database_win.h"
 
 #include <algorithm>
 #include <tuple>
 
 #include "base/bind.h"
@@ skipping 208 matching lines @@
 // static
 void ModuleDatabase::RemoveLoadAddressById(
     ModuleId module_id,
     ModuleLoadAddresses* load_addresses) {
   if (load_addresses->empty())
     return;
 
   // This handles the special case of removing the max element in O(1), as
   // FindLoadAddressIndexById processes the elements in reverse order.
   size_t i = FindLoadAddressIndexById(module_id, *load_addresses);
-  RemoveLoadAddressByIndex(i, load_addresses);
+  if (i != kInvalidIndex)
+    RemoveLoadAddressByIndex(i, load_addresses);
 }
 
 // static
 void ModuleDatabase::RemoveLoadAddressByIndex(
     size_t index,
     ModuleLoadAddresses* load_addresses) {
   DCHECK_LT(index, load_addresses->size());
 
   // Special case: removing the only remaining element.
   if (load_addresses->size() == 1) {
@@ skipping 134 matching lines @@
 }
 
 // ModuleDatabase::ProcessInfoData ---------------------------------------------
 
 ModuleDatabase::ProcessInfoData::ProcessInfoData() = default;
 
 ModuleDatabase::ProcessInfoData::ProcessInfoData(const ProcessInfoData& other) =
     default;
 
 ModuleDatabase::ProcessInfoData::~ProcessInfoData() = default;