From 18578e925108d6af5c5de320fbe1fcec8a3748c6 Mon Sep 17 00:00:00 2001 From: drebs Date: Mon, 4 Nov 2013 15:33:54 -0200 Subject: Fix items from mozilla review. Closes #4374. --- Makefile | 4 ++++ changes/feature-4374_fix-items-from-review | 1 + chrome/content/accountWizard/accountWizard.js | 4 +--- chrome/content/accountWizard/accountWizard.xul | 6 +----- install.rdf.template | 1 + 5 files changed, 8 insertions(+), 8 deletions(-) create mode 100644 changes/feature-4374_fix-items-from-review diff --git a/Makefile b/Makefile index 6b63456..fb59643 100755 --- a/Makefile +++ b/Makefile @@ -41,14 +41,18 @@ all: clean $(TARGET) $(TARGET): clean install.rdf mkdir -p $(TEMPDIR) + mkdir -p `dirname $@` cp -r $(PREFIX)/{$(FILES_TO_PACKAGE)} $(TEMPDIR)/ + rm -rf $(TEMPDIR)/.gitignore (cd $(TEMPDIR) && zip -r $(TARGET) ./) rm -rf $(TEMPDIR) (cd build/ && sha512sum $(PKGNAME) > SHA512SUMS && gpg -a --default-key $(DEFAULTKEY) --detach-sign SHA512SUMS) signed: clean mkdir -p $(TEMPDIR) + mkdir -p `dirname $@` cp -r $(PREFIX)/{$(FILES_TO_PACKAGE)} $(TEMPDIR)/ + rm -rf $(TEMPDIR)/.gitignore signtool -d $(CERTDIR) -k $(CERTNAME) $(TEMPDIR)/ (cd $(TEMPDIR) && zip $(TARGET) ./$(RSA_FILE) && zip -r -D $(TARGET) ./ -x ./$(RSA_FILE)) rm -rf $(TEMPDIR) diff --git a/changes/feature-4374_fix-items-from-review b/changes/feature-4374_fix-items-from-review new file mode 100644 index 0000000..6c4b3bf --- /dev/null +++ b/changes/feature-4374_fix-items-from-review @@ -0,0 +1 @@ + o Fix items from Mozilla editors review. Closes #4374. diff --git a/chrome/content/accountWizard/accountWizard.js b/chrome/content/accountWizard/accountWizard.js index b2b533f..da49536 100644 --- a/chrome/content/accountWizard/accountWizard.js +++ b/chrome/content/accountWizard/accountWizard.js @@ -28,7 +28,6 @@ Cu.import("resource:///modules/gloda/log4moz.js"); let gEmailWizardLogger = Log4Moz.getConfiguredLogger("mail.wizard"); var gStringsBundle; -var gMessengerBundle; var gBrandShortName; function e(elementID) @@ -125,8 +124,7 @@ BitmaskAccountWizard.prototype = gEmailWizardLogger.info("Email account setup dialog loaded."); - gStringsBundle = e("strings"); - gMessengerBundle = e("bundle_messenger"); + gStringsBundle = e("accountCreationStrings"); gBrandShortName = e("bundle_brand").getString("brandShortName"); // admin-locked prefs hurray diff --git a/chrome/content/accountWizard/accountWizard.xul b/chrome/content/accountWizard/accountWizard.xul index 0d42c0d..f9ff6f4 100644 --- a/chrome/content/accountWizard/accountWizard.xul +++ b/chrome/content/accountWizard/accountWizard.xul @@ -31,12 +31,8 @@ - - -