Use ContainsValue() instead of std::find() in chromecast/

chromecast/base/observer.h

 // 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.
 
 // Safe Observer/Observable implementation.
 //
 // When using ObserverListThreadSafe, we were running into issues since there
 // was no synchronization between getting the existing value, and registering
 // as an observer. See go/cast-platform-design-synchronized-value for more
 // details.
@@ skipping 96 matching lines @@
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/sequence_checker.h"
 #include "base/sequenced_task_runner.h"
+#include "base/stl_util.h"
 #include "base/synchronization/lock.h"
 #include "base/threading/sequenced_task_runner_handle.h"
 
 namespace chromecast {
 
 namespace subtle {
 template <typename T>
 class ObservableInternals;
 }  // namespace subtle
 
@@ skipping 132 matching lines @@
     SequenceOwnedInfo(const T& value) : value_(value) {}
 
     const T& value() const {
       DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
       return value_;
     }
 
     void AddObserver(Observer<T>* observer) {
       DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
       DCHECK(observer);
-      DCHECK(std::find(observers_.begin(), observers_.end(), observer) ==
-             observers_.end());
+      DCHECK(!base::ContainsValue(observers_, observer));
       observers_.push_back(observer);
     }
 
     void RemoveObserver(Observer<T>* observer) {
       DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
       DCHECK(observer);
-      DCHECK(std::find(observers_.begin(), observers_.end(), observer) !=
-             observers_.end());
+      DCHECK(base::ContainsValue(observers_, observer));
       observers_.erase(
           std::remove(observers_.begin(), observers_.end(), observer),
           observers_.end());
     }
 
     bool Empty() const {
       DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
       return observers_.empty();
     }
 
@@ skipping 131 matching lines @@
 }
 
 template <typename T>
 T Observable<T>::GetValueThreadSafe() const {
   return internals_->GetValueThreadSafe();
 }
 
 }  // namespace chromecast
 
 #endif  // CHROMECAST_BASE_OBSERVER_H_