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

chromecast/graphics/cast_focus_client_aura.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 "base/stl_util.h"

[alokp, 2017/06/01 19:01:21]

base/stl_util.h should be included after cast_focus_client_aura.h

[Tripta, 2017/06/02 04:40:22]

Have made the changes.

 #include "chromecast/graphics/cast_focus_client_aura.h"
 
 #include "ui/aura/window.h"
 
 #define LOG_WINDOW_INFO(top_level, window)                              \
   "top-level: " << (top_level)->id() << ": '" << (top_level)->GetName() \
                 << "', window: " << (window)->id() << ": '"             \
                 << (window)->GetName() << "'"
 
 namespace chromecast {
@@ skipping 102 matching lines @@
 // focus request if it isn't for the topmost window.  If it is for a lower
 // window, then we'll track it to focus it later when it rises to the top.
 void CastFocusClientAura::FocusWindow(aura::Window* window) {
   if (window) {
     if (!window->CanFocus()) {
       return;
     }
     aura::Window* top_level = GetZOrderWindow(window);
     DCHECK(top_level);
     DLOG(INFO) << "Requesting focus for " << LOG_WINDOW_INFO(top_level, window);
-    auto iter =
-        std::find(focusable_windows_.begin(), focusable_windows_.end(), window);
-    if (iter == focusable_windows_.end()) {
+    if(!base::ContainsValue(focusable_windows_, window)) {
       // We're not yet tracking this focusable window, so start tracking it as a
       // potential focus target.
       window->AddObserver(this);
       focusable_windows_.push_back(window);
     }
   }
 
   // Check whether this new window is the most appropriate to focus.
   UpdateWindowFocus();
 }
@@ skipping 102 matching lines @@
 
 aura::Window* CastFocusClientAura::GetToplevelWindow(aura::Window* window) {
   return GetZOrderWindow(window);
 }
 
 bool CastFocusClientAura::CanActivateWindow(aura::Window* window) const {
   return true;
 }
 
 }  // namespace chromecast