Remove file_util::kPathSeparator from posix.

chrome/browser/safe_browsing/safe_browsing_database_unittest.cc

 // Copyright (c) 2006-2008 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.
 //
 // Unit tests for the SafeBrowsing storage system.
 
 #include "base/command_line.h"
+#include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/path_service.h"
 #include "base/process_util.h"
 #include "base/sha2.h"
 #include "base/stats_counters.h"
 #include "base/string_util.h"
 #include "base/time.h"
 #include "chrome/browser/safe_browsing/protocol_parser.h"
 #include "chrome/browser/safe_browsing/safe_browsing_database.h"
 #include "chrome/common/chrome_switches.h"
 #include "googleurl/src/gurl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using base::Time;
 
-static const wchar_t kSafeBrowsingTestDatabase[] = L"SafeBrowsingTestDatabase";
 static const wchar_t kBloomSuffix[] = L" Bloom";
 
 namespace {
   SBPrefix Sha256Prefix(const std::string& str) {
     SBPrefix hash;
     base::SHA256HashString(str, &hash, sizeof(hash));
     return hash;
   }
 
   // Helper function to do an AddDel or SubDel command.
@@ skipping 17 matching lines @@
   }
 
   void SubDelChunk(SafeBrowsingDatabase* db,
                    const std::string& list,
                    int chunk_id) {
     DelChunk(db, list, chunk_id, true);
   }
 
   // Common database test set up code.
   std::wstring GetTestDatabaseName() {
-    std::wstring filename;
+    FilePath filename;
     PathService::Get(base::DIR_TEMP, &filename);
-    filename.push_back(file_util::kPathSeparator);
-    filename.append(kSafeBrowsingTestDatabase);
-    return filename;
+    filename = filename.Append(FILE_PATH_LITERAL("SafeBrowsingTestDatabase"));
+    return filename.ToWStringHack();
   }
 
   SafeBrowsingDatabase* SetupTestDatabase() {
     std::wstring filename = GetTestDatabaseName();
 
     // In case it existed from a previous run.
     file_util::Delete(filename + kBloomSuffix, false);
     file_util::Delete(filename, false);
 
     SafeBrowsingDatabase* database = SafeBrowsingDatabase::Create();
@@ skipping 958 matching lines @@
 
 void PeformUpdate(const std::wstring& initial_db,
                   const std::vector<ChunksInfo>& chunks,
                   std::vector<SBChunkDelete>* deletes) {
 // TODO(pinkerton): I don't think posix has any concept of IO counters, but
 // we can uncomment this when we implement ProcessMetrics::GetIOCounters
 #if defined(OS_WIN) || defined(OS_LINUX)
   IoCounters before, after;
 #endif
 
-  std::wstring filename;
-  PathService::Get(base::DIR_TEMP, &filename);
-  filename.push_back(file_util::kPathSeparator);
-  filename.append(L"SafeBrowsingTestDatabase");
+  FilePath path;
+  PathService::Get(base::DIR_TEMP, &path);
+  path = path.Append(FILE_PATH_LITERAL("SafeBrowsingTestDatabase"));
+  std::wstring filename = path.ToWStringHack();
+
   // In case it existed from a previous run.
   file_util::Delete(filename, false);
 
   if (!initial_db.empty()) {
     std::wstring full_initial_db = GetFullSBDataPath(initial_db);
     ASSERT_TRUE(file_util::CopyFile(full_initial_db, filename));
   }
 
   SafeBrowsingDatabase* database = SafeBrowsingDatabase::Create();
   database->SetSynchronous();
@@ skipping 174 matching lines @@
 TEST(SafeBrowsingDatabase, DISABLED_DatabaseOldLotsofDeletesIO) {
   std::vector<ChunksInfo> chunks;
   std::vector<SBChunkDelete>* deletes = new std::vector<SBChunkDelete>;
   SBChunkDelete del;
   del.is_sub_del = false;
   del.list_name = safe_browsing_util::kMalwareList;
   del.chunk_del.push_back(ChunkRange(3539, 3579));
   deletes->push_back(del);
   PeformUpdate(GetOldSafeBrowsingPath(), chunks, deletes);
 }