Include string fields when performing a search on a node.

BUG=83849
TEST=


Review URL: http://codereview.chromium.org/7057059

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87541 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 14 insertions, 1 deletions

diff --git a/chrome/browser/sync/engine/syncapi.cc b/chrome/browser/sync/engine/syncapi.cc
index de00056..37f56ab 100644
--- a/chrome/browser/sync/engine/syncapi.cc
+++ b/chrome/browser/sync/engine/syncapi.cc
@@ -1124,7 +1124,20 @@ bool BaseNode::ContainsString(const std::string& lowercase_query) const {
   // Now convert to lower case.
   StringToLowerASCII(&temp);
 
-  return temp.find(lowercase_query) != std::string::npos;
+  if (temp.find(lowercase_query) != std::string::npos)
+    return true;
+
+  // Now go through all the string fields to see if the value is there.
+  for (int i = syncable::STRING_FIELDS_BEGIN; i < syncable::STRING_FIELDS_END;
+       ++i) {
+    std::string value = GetEntry()->Get(
+        static_cast<syncable::StringField>(i));
+
+    StringToLowerASCII(&value);
+    if (value.find(lowercase_query) != std::string::npos)
+      return true;
+  }
+  return false;
 }
 
 SyncManager::ExtraPasswordChangeRecordData::ExtraPasswordChangeRecordData() {}