Bug 789248. Fix failing social tests by only updating the status panel on tests that require it and are sure to clear it up afterwards. r=felipe

This commit is contained in:
Shane Caraveo 2012-09-06 13:14:34 -07:00
parent f4119f617d
commit 207baefbc9
3 changed files with 7 additions and 3 deletions

View File

@ -59,6 +59,8 @@ var tests = {
// The sidebar message will always come first, since it loads by default
ok(true, "got sidebar message");
gotSidebarMessage = true;
// load a status panel
port.postMessage({topic: "test-ambient-notification"});
checkNext();
break;
}

View File

@ -7,8 +7,8 @@ function test() {
let manifest = { // normal provider
name: "provider 1",
origin: "https://example1.com",
workerURL: "https://example1.com/worker.js",
origin: "https://example.com",
workerURL: "https://example.com/browser/browser/base/content/test/social_worker.js",
iconURL: "chrome://branding/content/icon48.png"
};
runSocialTestWithProvider(manifest, function (finishcb) {

View File

@ -77,13 +77,15 @@ onconnect = function(e) {
profileURL: "http://en.wikipedia.org/wiki/Kuma_Lisa"
};
port.postMessage({topic: "social.user-profile", data: profile});
break;
case "test-ambient-notification":
let icon = {
name: "testIcon",
iconURL: "chrome://branding/content/icon48.png",
contentPanel: "https://example.com/browser/browser/base/content/test/social_panel.html",
counter: 1
};
port.postMessage({topic: "social.ambient-notification", data: icon});
apiPort.postMessage({topic: "social.ambient-notification", data: icon});
break;
case "test-isVisible":
sidebarPort.postMessage({topic: "test-isVisible"});