Make various string_util functions take StringPieces instead of char[].

chrome/renderer/chrome_render_view_observer.cc

 // Copyright (c) 2012 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 "chrome/renderer/chrome_render_view_observer.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
@@ skipping 540 matching lines @@
   sprintf_s(buf, "%d chars retrieved for indexing in %gms\n",
             contents.size(), (end - begin)*1000);
   OutputDebugStringA(buf);
 #endif
 
   // When the contents are clipped to the maximum, we don't want to have a
   // partial word indexed at the end that might have been clipped. Therefore,
   // terminate the string at the last space to ensure no words are clipped.
   if (contents->size() == kMaxIndexChars) {
     size_t last_space_index = contents->find_last_of(base::kWhitespaceUTF16);
-    if (last_space_index == base::string16::npos)
-      return;  // don't index if we got a huge block of text with no spaces
-    contents->resize(last_space_index);
+    if (last_space_index != base::string16::npos)
+      contents->resize(last_space_index);
   }
 }
 
 bool ChromeRenderViewObserver::HasRefreshMetaTag(WebFrame* frame) {
   if (!frame)
     return false;
   WebElement head = frame->document().head();
   if (head.isNull() || !head.hasChildNodes())
     return false;
 
   const WebString tag_name(base::ASCIIToUTF16("meta"));
   const WebString attribute_name(base::ASCIIToUTF16("http-equiv"));
 
   WebNodeList children = head.childNodes();
   for (size_t i = 0; i < children.length(); ++i) {
     WebNode node = children.item(i);
     if (!node.isElementNode())
       continue;
     WebElement element = node.to<WebElement>();
     if (!element.hasTagName(tag_name))
       continue;
     WebString value = element.getAttribute(attribute_name);
     if (value.isNull() || !LowerCaseEqualsASCII(value, "refresh"))
       continue;
     return true;
   }
   return false;
 }