diff options
author | kkania@chromium.org <kkania@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-30 23:25:57 +0000 |
---|---|---|
committer | kkania@chromium.org <kkania@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-30 23:25:57 +0000 |
commit | d38f83f98ff29ec940f0bc978a925fa7828fad67 (patch) | |
tree | 2418bcaa6d25a7f5d340f5d2a62c05ac4cea4916 /chrome/test | |
parent | 42f79663e4afdda0e0cd3d9f4d70a9f3273c9957 (diff) | |
download | chromium_src-d38f83f98ff29ec940f0bc978a925fa7828fad67.zip chromium_src-d38f83f98ff29ec940f0bc978a925fa7828fad67.tar.gz chromium_src-d38f83f98ff29ec940f0bc978a925fa7828fad67.tar.bz2 |
Attempt to fix AppApiTest.AppProcess and ExtensionApiTest.Storage flakiness.
BUG=42497,42943
TEST=none
Review URL: http://codereview.chromium.org/1705025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46151 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/test')
-rw-r--r-- | chrome/test/data/extensions/api_test/storage/background.html | 28 | ||||
-rw-r--r-- | chrome/test/data/extensions/api_test/storage/tab.html | 2 |
2 files changed, 19 insertions, 11 deletions
diff --git a/chrome/test/data/extensions/api_test/storage/background.html b/chrome/test/data/extensions/api_test/storage/background.html index 3bd7b24..5e3be5f 100644 --- a/chrome/test/data/extensions/api_test/storage/background.html +++ b/chrome/test/data/extensions/api_test/storage/background.html @@ -1,25 +1,31 @@ <script>
// store some stuff in local storage
localStorage.foo = "bar";
-
+
// store some stuff in a database
var db = window.openDatabase("mydb2", "1.0", "database test", 2048);
if (!db)
chrome.test.fail("failed to open database");
db.transaction(function(tx) {
- tx.executeSql("drop table if exists note");
- tx.executeSql("create table note (body text)", []);
- tx.executeSql("insert into note values ('hotdog')", []);
+ tx.executeSql("drop table if exists note", [], null, onSqlError);
+ tx.executeSql("create table note (body text)", [], null, onSqlError);
+ tx.executeSql("insert into note values ('hotdog')", [], onSqlExecuted,
+ onSqlError);
}, function(error) {
chrome.test.fail(error.message);
});
- // Open a tab. This doesn't really prove we're writing to disk, but it is
- // difficult to prove that without shutting down the process. We'll just
- // trust that if this next trick works, that the real testing for local
- // storage is good enough to catch more subtle errors.
- chrome.tabs.create({
- url: "tab.html"
- });
+ function onSqlError(tx, error) {
+ chrome.test.fail(error.message);
+ }
+ function onSqlExecuted() {
+ // Open a tab. This doesn't really prove we're writing to disk, but it is
+ // difficult to prove that without shutting down the process. We'll just
+ // trust that if this next trick works, that the real testing for local
+ // storage is good enough to catch more subtle errors.
+ chrome.tabs.create({
+ url: "tab.html"
+ });
+ }
</script>
diff --git a/chrome/test/data/extensions/api_test/storage/tab.html b/chrome/test/data/extensions/api_test/storage/tab.html index 1e116e7..675dcaaf 100644 --- a/chrome/test/data/extensions/api_test/storage/tab.html +++ b/chrome/test/data/extensions/api_test/storage/tab.html @@ -13,6 +13,8 @@ chrome.test.runTests([function tab() { chrome.test.assertTrue(results.rows.length == 1);
chrome.test.assertTrue(results.rows.item(0).body == "hotdog");
chrome.test.succeed();
+ }, function(tx, error) {
+ chrome.test.fail(error.message);
});
}, function(error) {
chrome.test.fail(error.message);
|