From 535aa8adfcea6d0258d324b809f494e23c0f0d0a Mon Sep 17 00:00:00 2001
From: Shane Tomlinson <stomlinson@mozilla.com>
Date: Thu, 2 Aug 2012 14:05:26 +0100
Subject: [PATCH] Test to make sure emails, siteInfo and managePage all have
default values after clear.
---
resources/static/test/cases/common/js/storage.js | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/resources/static/test/cases/common/js/storage.js b/resources/static/test/cases/common/js/storage.js
index cf1cff158..cc98a58d9 100644
--- a/resources/static/test/cases/common/js/storage.js
+++ b/resources/static/test/cases/common/js/storage.js
@@ -71,12 +71,19 @@
});
- test("clear", function() {
+ test("clear - there should be default values", function() {
storage.addEmail("testuser@testuser.com", {priv: "key"});
storage.clear();
var emails = storage.getEmails();
equal(_.size(emails), 0, "object should have no items");
+
+ // all fields *MUST* have default values or else synchronization of
+ // localStorage in IE8 across multiple browsing contexts becomes a problem.
+ // See issue #2206 and #1637
+ notEqual(typeof localStorage.emails, "undefined", "emails is defined");
+ notEqual(typeof localStorage.siteInfo, "undefined", "siteInfo is defined");
+ notEqual(typeof localStorage.managePage, "undefined", "managePage is defined");
});
test("invalidateEmail with valid email address", function() {
--
GitLab