[Pkg-mozext-commits] [firebug] 78/82: Clear storage synchronously (issue 6143), should fix lib/storageService test

David Prévot taffit at moszumanska.debian.org
Mon Mar 31 22:45:43 UTC 2014


This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to tag fbtest-1.11.2
in repository firebug.

commit b54e6fe0794e77a08b82671661bc6c3449c80a4e
Author: Jan Odvarko <odvarko at gmail.com>
Date:   Thu Jan 3 18:17:28 2013 +0100

    Clear storage synchronously (issue 6143), should fix lib/storageService test
---
 extension/content/firebug/console/commandLineInclude.js | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/extension/content/firebug/console/commandLineInclude.js b/extension/content/firebug/console/commandLineInclude.js
index b27dc81..59a1960 100644
--- a/extension/content/firebug/console/commandLineInclude.js
+++ b/extension/content/firebug/console/commandLineInclude.js
@@ -124,14 +124,14 @@ var CommandLineIncludeRep = domplate(FirebugReps.Table,
         // NOTE: that piece of code has not been tested since deleting aliases through the table 
         // has been disabled.
         // Once it is enabled again, make sure FBTests is available for this feature
-        var store = CommandLine.getStore();
-        if (! Options.get(removeConfirmation))
+        var store = CommandLineInclude.getStore();
+        if (!Options.get(removeConfirmation))
         {
             var check = {value: false};
             var flags = prompts.BUTTON_POS_0 * prompts.BUTTON_TITLE_YES +
             prompts.BUTTON_POS_1 * prompts.BUTTON_TITLE_NO;
 
-            if  (prompts.confirmEx(context.chrome.window, Locale.$STR("Firebug"),
+            if (prompts.confirmEx(context.chrome.window, Locale.$STR("Firebug"),
                 Locale.$STR("commandline.include.confirmDelete"), flags, "", "", "",
                 Locale.$STR("Do_not_show_this_message_again"), check) > 0)
             {
@@ -483,10 +483,8 @@ var CommandLineInclude = Obj.extend(Firebug.Module,
     resetAllOptions: function()
     {
         var store = this.getStore();
-
-        if (!store)
-            return;
-        store.clear();
+        if (store)
+            store.clear(true);
     }
 });
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/firebug.git



More information about the Pkg-mozext-commits mailing list