IndexedDB: Switch key/value handling from vector<char> to std::string

content/browser/indexed_db/leveldb/leveldb_comparator.h

Index: content/browser/indexed_db/leveldb/leveldb_comparator.h
diff --git a/content/browser/indexed_db/leveldb/leveldb_comparator.h b/content/browser/indexed_db/leveldb/leveldb_comparator.h
index 1a804104ddef6f141770dbb523ce5aad7878263d..2e8e3299916ac5cf1397e7351cadb89d04f16598 100644
--- a/content/browser/indexed_db/leveldb/leveldb_comparator.h
+++ b/content/browser/indexed_db/leveldb/leveldb_comparator.h
@@ -5,17 +5,16 @@
 #ifndef CONTENT_BROWSER_INDEXED_DB_LEVELDB_LEVELDB_COMPARATOR_H_
 #define CONTENT_BROWSER_INDEXED_DB_LEVELDB_LEVELDB_COMPARATOR_H_
 
-#include "base/strings/string16.h"
+#include "base/strings/string_piece.h"
 
 namespace content {
 
-class LevelDBSlice;
-
 class LevelDBComparator {

[alecflett, 2013/07/09 18:26:30]

Seems like we could get rid of this adapter class entirely with a little more
work (for another patch...)

[jsbell, 2013/07/09 19:41:46]

Agreed.

  public:
   virtual ~LevelDBComparator() {}
 
-  virtual int Compare(const LevelDBSlice& a, const LevelDBSlice& b) const = 0;
+  virtual int Compare(const base::StringPiece& a,
+                      const base::StringPiece& b) const = 0;
   virtual const char* Name() const = 0;
 };