From bf93427cbb9c64fda8a0d0aa7fc4d0e3426ea640 Mon Sep 17 00:00:00 2001 From: Charles Kerr Date: Fri, 28 Mar 2025 21:34:24 -0500 Subject: [PATCH] perf: avoid double map lookup in HidChooserContext::DeviceRemoved() (#46343) --- shell/browser/hid/hid_chooser_context.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/shell/browser/hid/hid_chooser_context.cc b/shell/browser/hid/hid_chooser_context.cc index e31d77267caa..6e288fa19e54 100644 --- a/shell/browser/hid/hid_chooser_context.cc +++ b/shell/browser/hid/hid_chooser_context.cc @@ -253,10 +253,10 @@ void HidChooserContext::DeviceAdded(device::mojom::HidDeviceInfoPtr device) { void HidChooserContext::DeviceRemoved(device::mojom::HidDeviceInfoPtr device) { DCHECK(device); - DCHECK(devices_.contains(device->guid)); // Update the device list. - devices_.erase(device->guid); + const size_t n_erased = devices_.erase(device->guid); + DCHECK_EQ(n_erased, 1U); // Notify all device observers. for (auto& observer : device_observer_list_)