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

ios/chrome/browser/passwords/password_generation_agent.mm

Index: ios/chrome/browser/passwords/password_generation_agent.mm
diff --git a/ios/chrome/browser/passwords/password_generation_agent.mm b/ios/chrome/browser/passwords/password_generation_agent.mm
index 67671677d07eeb5e1f7dfb59827b3b2c9ee849c7..377b36bd1c112c8e76a7b0c6e70bdd76a40c78f0 100644
--- a/ios/chrome/browser/passwords/password_generation_agent.mm
+++ b/ios/chrome/browser/passwords/password_generation_agent.mm
@@ -8,6 +8,7 @@
 
 #include "base/mac/foundation_util.h"
 #include "base/mac/scoped_block.h"
+#include "base/stl_util.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/browser/password_generator.h"
@@ -44,7 +45,7 @@ const size_t kMinimumTextFieldsForAccountCreation = 3;
 
 // Returns true if |urls| contains |url|.
 bool VectorContainsURL(const std::vector<GURL>& urls, const GURL& url) {

[sdefresne, 2017/06/23 14:31:21]

There is only one call site for VectorContainsURL (in the current file). Maybe
inline the base::ContainsValue() and remove the function (as it is just a
specialized instantiation of base::ContainsValue).

[Tripta, 2017/06/27 06:41:28]

Done.

-  return std::find(urls.begin(), urls.end(), url) != urls.end();
+  return base::ContainsValue(urls, url);
 }
 
 // Returns whether |field| should be considered a text field. Implementation