Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(360)

Unified Diff: device/generic_sensor/platform_sensor_provider_base.cc

Issue 2370343002: [sensors] Ambient light sensor implementation for ChromeOS and Linux. (Closed)
Patch Set: rebased on top of Mihail's cl Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: device/generic_sensor/platform_sensor_provider_base.cc
diff --git a/device/generic_sensor/platform_sensor_provider_base.cc b/device/generic_sensor/platform_sensor_provider_base.cc
index fd43614d4816896155af189f2a845322c82b6ef2..5c528f3c943e799914cf7a1c779405cecb09c430 100644
--- a/device/generic_sensor/platform_sensor_provider_base.cc
+++ b/device/generic_sensor/platform_sensor_provider_base.cc
@@ -76,8 +76,8 @@ bool PlatformSensorProviderBase::CreateSharedBufferIfNeeded() {
void PlatformSensorProviderBase::RemoveSensor(mojom::SensorType type) {
DCHECK(CalledOnValidThread());
- DCHECK(ContainsKey(sensor_map_, type));
- sensor_map_.erase(type);
+ if (ContainsKey(sensor_map_, type))
+ sensor_map_.erase(type);
if (sensor_map_.empty())
shared_buffer_handle_.reset();

Powered by Google App Engine
This is Rietveld 408576698