From 9a51af82db9700913e267c774fa7b6ab049653aa Mon Sep 17 00:00:00 2001 From: "morse%netscape.com" Date: Thu, 4 Apr 2002 21:41:22 +0000 Subject: [PATCH] bug 135046, remove p3p performance penalty, r=dp, sr=waterson, a=asa, adt=putterman --- .../resources/content/cookieTasksOverlay.xul | 22 ++++++++++--------- .../content/cookieTasksOverlay.xul | 22 ++++++++++--------- 2 files changed, 24 insertions(+), 20 deletions(-) diff --git a/extensions/cookie/resources/content/cookieTasksOverlay.xul b/extensions/cookie/resources/content/cookieTasksOverlay.xul index e74b2dc0fdb4..c5f0729bc81d 100644 --- a/extensions/cookie/resources/content/cookieTasksOverlay.xul +++ b/extensions/cookie/resources/content/cookieTasksOverlay.xul @@ -54,16 +54,18 @@ } } - function TestIconVisibility() { - var cookieservice = Components.classes["@mozilla.org/cookieService;1"].getService(); - cookieservice = cookieservice.QueryInterface(Components.interfaces.nsICookieService); - if (cookieservice.cookieIconIsVisible) { - var cookieIcon = document.getElementById("privacy-button"); - if (cookieIcon) { - cookieIcon.removeAttribute("hidden"); + var listener = { + handleEvent: function(event) { + var cookieservice = Components.classes["@mozilla.org/cookieService;1"].getService(); + cookieservice = cookieservice.QueryInterface(Components.interfaces.nsICookieService); + if (cookieservice.cookieIconIsVisible) { + var cookieIcon = document.getElementById("privacy-button"); + if (cookieIcon) { + cookieIcon.removeAttribute("hidden"); + } } } - } + }; function viewP3PDialog() { if (!gObserverService) { @@ -96,8 +98,8 @@ observerService.addObserver(cookieIconObserver, "cookieIcon", false); // determine whether or not cookie icon in this new window should be displayed - // we need to delay before displaying icon because it hasn't been created yet - setTimeout(TestIconVisibility, 0); + // we need to wait until window is created before displaying icon + addEventListener("load", listener, false); } } ]]> diff --git a/xpfe/components/permissions/content/cookieTasksOverlay.xul b/xpfe/components/permissions/content/cookieTasksOverlay.xul index e74b2dc0fdb4..c5f0729bc81d 100644 --- a/xpfe/components/permissions/content/cookieTasksOverlay.xul +++ b/xpfe/components/permissions/content/cookieTasksOverlay.xul @@ -54,16 +54,18 @@ } } - function TestIconVisibility() { - var cookieservice = Components.classes["@mozilla.org/cookieService;1"].getService(); - cookieservice = cookieservice.QueryInterface(Components.interfaces.nsICookieService); - if (cookieservice.cookieIconIsVisible) { - var cookieIcon = document.getElementById("privacy-button"); - if (cookieIcon) { - cookieIcon.removeAttribute("hidden"); + var listener = { + handleEvent: function(event) { + var cookieservice = Components.classes["@mozilla.org/cookieService;1"].getService(); + cookieservice = cookieservice.QueryInterface(Components.interfaces.nsICookieService); + if (cookieservice.cookieIconIsVisible) { + var cookieIcon = document.getElementById("privacy-button"); + if (cookieIcon) { + cookieIcon.removeAttribute("hidden"); + } } } - } + }; function viewP3PDialog() { if (!gObserverService) { @@ -96,8 +98,8 @@ observerService.addObserver(cookieIconObserver, "cookieIcon", false); // determine whether or not cookie icon in this new window should be displayed - // we need to delay before displaying icon because it hasn't been created yet - setTimeout(TestIconVisibility, 0); + // we need to wait until window is created before displaying icon + addEventListener("load", listener, false); } } ]]>