OLD | NEW |
---|---|
(Empty) | |
1 // Copyright 2014 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #include "base/command_line.h" | |
6 #include "base/files/file_path.h" | |
7 #include "base/memory/ref_counted.h" | |
8 #include "base/strings/stringprintf.h" | |
9 #include "chrome/browser/extensions/api/storage/settings_storage_quota_enforcer. h" | |
10 #include "chrome/browser/extensions/api/storage/settings_test_util.h" | |
11 #include "chrome/browser/extensions/api/storage/storage_api.h" | |
12 #include "chrome/browser/extensions/extension_api_unittest.h" | |
13 #include "chrome/browser/extensions/extension_service.h" | |
14 #include "chrome/browser/extensions/test_extension_system.h" | |
15 #include "chrome/browser/value_store/leveldb_value_store.h" | |
16 #include "chrome/browser/value_store/value_store.h" | |
17 #include "extensions/browser/event_router.h" | |
18 #include "extensions/browser/extension_prefs.h" | |
19 #include "extensions/browser/extension_system.h" | |
20 #include "extensions/common/id_util.h" | |
21 #include "extensions/common/manifest.h" | |
22 #include "extensions/common/test_util.h" | |
23 #include "third_party/leveldatabase/src/include/leveldb/db.h" | |
24 #include "third_party/leveldatabase/src/include/leveldb/write_batch.h" | |
25 | |
26 namespace extensions { | |
27 | |
28 class StorageApiUnittest : public ExtensionApiUnittest { | |
29 public: | |
30 virtual void SetUp() OVERRIDE { | |
31 ExtensionApiUnittest::SetUp(); | |
32 TestExtensionSystem* extension_system = | |
33 static_cast<TestExtensionSystem*>(ExtensionSystem::Get(profile())); | |
34 ExtensionService* extension_service = extension_system->extension_service(); | |
35 if (!extension_service) { | |
36 extension_service = extension_system->CreateExtensionService( | |
37 CommandLine::ForCurrentProcess(), base::FilePath(), false); | |
38 ASSERT_TRUE(extension_service); | |
39 } | |
40 | |
41 if (!extension_system->event_router()) { | |
42 extension_system->SetEventRouter(scoped_ptr<EventRouter>( | |
43 new EventRouter(profile(), ExtensionPrefs::Get(profile())))); | |
44 } | |
45 | |
46 scoped_refptr<Extension> extension = | |
47 test_util::CreateExtensionWithID(id_util::GenerateId("ext")); | |
48 set_extension(extension); | |
49 extension_service->AddExtension(extension.get()); | |
50 } | |
51 | |
52 protected: | |
53 // Runs the storage.set() API function with local storage. | |
54 void RunSetFunction(const std::string& key, const std::string& value) { | |
55 RunFunction( | |
56 new StorageStorageAreaSetFunction(), | |
57 base::StringPrintf( | |
58 "[\"local\", {\"%s\": \"%s\"}]", key.c_str(), value.c_str())); | |
59 } | |
60 | |
61 // Runs the storage.get() API function with the local storage, and populates | |
62 // |value| with the string result. Returns true on success, and false on | |
63 // failure. | |
64 bool RunGetFunction(const std::string& key, std::string* value) { | |
not at google - send to devlin
2014/02/19 21:25:29
return a testing::AssertionResult here. It would l
Devlin
2014/02/19 23:12:28
Done.
| |
65 scoped_ptr<base::Value> result = RunFunctionAndReturnValue( | |
66 new StorageStorageAreaGetFunction(), | |
67 base::StringPrintf("[\"local\", \"%s\"]", key.c_str())); | |
68 base::DictionaryValue* dict = NULL; | |
69 if (!result->GetAsDictionary(&dict) || !dict->GetString(key, value)) | |
70 return false; | |
71 return true; | |
72 } | |
73 }; | |
74 | |
75 TEST_F(StorageApiUnittest, RestoreCorruptedStorage) { | |
76 const char kKey[] = "key"; | |
77 const char kValue[] = "value"; | |
78 std::string result; | |
79 | |
80 // Do a simple set/get combo to make sure the API works. | |
81 RunSetFunction(kKey, kValue); | |
82 EXPECT_TRUE(RunGetFunction(kKey, &result)); | |
83 EXPECT_EQ(kValue, result); | |
84 | |
85 // Corrupt the store. This is not as pretty as ideal, because we use knowledge | |
86 // of the underlying structure, but there's no real good way to corrupt a | |
87 // store other than directly modifying the files. | |
88 ValueStore* store = | |
89 settings_test_util::GetStorage(extension()->id(), | |
90 settings_namespace::LOCAL, | |
91 ExtensionSystem::Get(profile()) | |
92 ->extension_service() | |
93 ->settings_frontend()); | |
94 ASSERT_TRUE(store); | |
95 SettingsStorageQuotaEnforcer* quota_store = | |
96 static_cast<SettingsStorageQuotaEnforcer*>(store); | |
97 LeveldbValueStore* leveldb_store = | |
98 static_cast<LeveldbValueStore*>(quota_store->get_delegate_for_test()); | |
not at google - send to devlin
2014/02/19 21:25:29
this does make me very uneasy given there is no sa
Devlin
2014/02/19 23:12:28
Per long in-person conversation, there's no pretty
| |
99 leveldb::WriteBatch batch; | |
100 batch.Put(kKey, "[{(.*+\"\'\\"); | |
101 EXPECT_TRUE(leveldb_store->WriteToDbForTest(&batch)); | |
102 EXPECT_TRUE(leveldb_store->Get(kKey)->IsCorrupted()); | |
103 | |
104 // Running another set should end up working (even though it will restore the | |
105 // store behind the scenes). | |
106 RunSetFunction(kKey, kValue); | |
107 EXPECT_TRUE(RunGetFunction(kKey, &result)); | |
108 EXPECT_EQ(kValue, result); | |
109 } | |
110 | |
111 } // namespace extensions | |
OLD | NEW |