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

ios/chrome/browser/bookmarks/bookmarks_utils.cc

Index: ios/chrome/browser/bookmarks/bookmarks_utils.cc
diff --git a/ios/chrome/browser/bookmarks/bookmarks_utils.cc b/ios/chrome/browser/bookmarks/bookmarks_utils.cc
index a7ece33186c047547a51343b61d6793d414e3911..961e299c4a24728612b27fa09258844bf163e55f 100644
--- a/ios/chrome/browser/bookmarks/bookmarks_utils.cc
+++ b/ios/chrome/browser/bookmarks/bookmarks_utils.cc
@@ -6,6 +6,7 @@
 
 #include "base/logging.h"
 #include "base/metrics/histogram_macros.h"
+#include "base/stl_util.h"
 #include "components/bookmarks/browser/bookmark_model.h"
 #include "components/prefs/pref_service.h"
 #include "ios/chrome/browser/bookmarks/bookmark_model_factory.h"
@@ -72,8 +73,7 @@ std::vector<const BookmarkNode*> RootLevelFolders(BookmarkModel* model) {
 
 bool IsPrimaryPermanentNode(const BookmarkNode* node, BookmarkModel* model) {
   std::vector<const BookmarkNode*> primary_nodes(PrimaryPermanentNodes(model));
-  if (std::find(primary_nodes.begin(), primary_nodes.end(), node) !=
-      primary_nodes.end()) {
+  if (base::ContainsValue(primary_nodes, node)) {

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

This can be changed to the following:

  return base::ContainsValue(primary_nodes, node);

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

Done.

     return true;
   }
   return false;
@@ -88,9 +88,7 @@ const BookmarkNode* RootLevelFolderForNode(const BookmarkNode* node,
 
   const std::vector<const BookmarkNode*> root_folders(RootLevelFolders(model));
   const BookmarkNode* top = node;
-  while (top &&
-         std::find(root_folders.begin(), root_folders.end(), top) ==
-             root_folders.end()) {
+  while (top && !base::ContainsValue(root_folders, top)) {
     top = top->parent();
   }
   return top;