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

components/webdata/common/web_database_migration_unittest.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 <string>
 
 #include "base/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/guid.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 139 matching lines @@
                        sql::MetaTable* meta_table) {
   std::string value;
   EXPECT_FALSE(meta_table->GetValue(
       "Default Search Provider ID Backup", &value));
   EXPECT_FALSE(meta_table->GetValue(
       "Default Search Provider ID Backup Signature", &value));
   EXPECT_FALSE(connection.DoesTableExist("keywords_backup"));
 }
 
 std::string RemoveQuotes(const std::string& has_quotes) {
+  std::string no_quotes;
   // SQLite quotes: http://www.sqlite.org/lang_keywords.html
-  static const char kQuotes[] = "\"[]`";
-  std::string no_quotes;
-  base::RemoveChars(has_quotes, kQuotes, &no_quotes);
+  base::RemoveChars(has_quotes, "\"[]`", &no_quotes);
   return no_quotes;
 }
 
 }  // anonymous namespace
 
 // The WebDatabaseMigrationTest encapsulates testing of database migrations.
 // Specifically, these tests are intended to exercise any schema changes in
 // the WebDatabase and data migrations that occur in
 // |WebDatabase::MigrateOldVersionsAsNeeded()|.
 class WebDatabaseMigrationTest : public testing::Test {
@@ skipping 2490 matching lines @@
     EXPECT_EQ(base::string16(), s_profiles.ColumnString16(7));
     EXPECT_EQ(ASCIIToUTF16("US"), s_profiles.ColumnString16(8));
     EXPECT_EQ(1395948829, s_profiles.ColumnInt(9));
     EXPECT_EQ(ASCIIToUTF16("Chrome settings"), s_profiles.ColumnString16(10));
     EXPECT_EQ(std::string(), s_profiles.ColumnString(11));
 
     // No more entries expected.
     ASSERT_FALSE(s_profiles.Step());
   }
 }