Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1196)

Unified Diff: content/browser/indexed_db/indexed_db_backing_store_unittest.cc

Issue 16870007: Switch database/file_identifier to std::string, remove createFromDatabaseIdentifier calls (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/indexed_db/indexed_db_backing_store_unittest.cc
diff --git a/content/browser/indexed_db/indexed_db_backing_store_unittest.cc b/content/browser/indexed_db/indexed_db_backing_store_unittest.cc
index 314ce23f70ec5a02e52f25a11f0650d752468776..087a32996a3da09fd4b40ad2e16b420c12dc8a82 100644
--- a/content/browser/indexed_db/indexed_db_backing_store_unittest.cc
+++ b/content/browser/indexed_db/indexed_db_backing_store_unittest.cc
@@ -11,10 +11,10 @@
#include "base/strings/utf_string_conversions.h"
#include "content/browser/indexed_db/indexed_db_factory.h"
#include "content/browser/indexed_db/indexed_db_leveldb_coding.h"
+#include "googleurl/src/gurl.h"
#include "testing/gtest/include/gtest/gtest.h"
-#include "third_party/WebKit/public/web/WebSecurityOrigin.h"
+#include "webkit/common/database/database_identifier.h"
-using WebKit::WebSecurityOrigin;
using WebKit::WebIDBKey;
namespace content {
@@ -25,7 +25,7 @@ class IndexedDBBackingStoreTest : public testing::Test {
public:
IndexedDBBackingStoreTest() {}
virtual void SetUp() {
- string16 file_identifier;
+ std::string file_identifier;
backing_store_ = IndexedDBBackingStore::OpenInMemory(file_identifier);
// useful keys and values during tests
@@ -333,9 +333,10 @@ class MockIDBFactory : public IndexedDBFactory {
}
scoped_refptr<IndexedDBBackingStore> TestOpenBackingStore(
- const WebSecurityOrigin& origin,
+ const GURL& origin,
const base::FilePath& data_directory) {
- return OpenBackingStore(origin.databaseIdentifier(), data_directory);
+ return OpenBackingStore(
+ webkit_database::GetIdentifierFromOrigin(origin), data_directory);
}
private:
@@ -343,10 +344,8 @@ class MockIDBFactory : public IndexedDBFactory {
};
TEST(IndexedDBFactoryTest, BackingStoreLifetime) {
- WebSecurityOrigin origin1 =
- WebSecurityOrigin::createFromString("http://localhost:81");
- WebSecurityOrigin origin2 =
- WebSecurityOrigin::createFromString("http://localhost:82");
+ GURL origin1("http://localhost:81");
+ GURL origin2("http://localhost:82");
scoped_refptr<MockIDBFactory> factory = MockIDBFactory::Create();
@@ -371,10 +370,8 @@ TEST(IndexedDBFactoryTest, BackingStoreLifetime) {
}
TEST(IndexedDBFactoryTest, MemoryBackingStoreLifetime) {
- WebSecurityOrigin origin1 =
- WebSecurityOrigin::createFromString("http://localhost:81");
- WebSecurityOrigin origin2 =
- WebSecurityOrigin::createFromString("http://localhost:82");
+ GURL origin1("http://localhost:81");
+ GURL origin2("http://localhost:82");
scoped_refptr<MockIDBFactory> factory = MockIDBFactory::Create();
scoped_refptr<IndexedDBBackingStore> mem_store1 =
@@ -412,15 +409,12 @@ TEST(IndexedDBFactoryTest, RejectLongOrigins)
EXPECT_GT(limit, 0);
std::string origin(limit + 1, 'x');
- WebSecurityOrigin too_long_origin =
- WebSecurityOrigin::createFromString(WebKit::WebString::fromUTF8(
- std::string("http://" + origin + ":81/").c_str()));
+ GURL too_long_origin("http://" + origin + ":81/");
scoped_refptr<IndexedDBBackingStore> diskStore1 =
factory->TestOpenBackingStore(too_long_origin, base_path);
EXPECT_FALSE(diskStore1.get());
- WebSecurityOrigin ok_origin =
- WebSecurityOrigin::createFromString("http://someorigin.com:82/");
+ GURL ok_origin("http://someorigin.com:82/");
scoped_refptr<IndexedDBBackingStore> diskStore2 =
factory->TestOpenBackingStore(ok_origin, base_path);
EXPECT_TRUE(diskStore2.get());

Powered by Google App Engine
This is Rietveld 408576698