From fa0b15d4cece945fb440641742ad4a3685b8edd6 Mon Sep 17 00:00:00 2001
From: Alexandre Lissy <lissyx+github@lissyx.dyndns.org>
Date: Thu, 10 Apr 2014 08:07:10 +0200
Subject: [PATCH] Bug 994262 - Make sure ModalDialog has title set

---
 apps/system/js/modal_dialog.js             |  4 ++--
 apps/system/test/unit/modal_dialog_test.js | 11 +++++++++++
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/apps/system/js/modal_dialog.js b/apps/system/js/modal_dialog.js
index e13782c0f6bd..53705e158757 100644
--- a/apps/system/js/modal_dialog.js
+++ b/apps/system/js/modal_dialog.js
@@ -199,7 +199,7 @@ var ModalDialog = {
         elements.prompt.classList.add('visible');
         elements.promptInput.value = evt.detail.initialValue;
         elements.promptMessage.innerHTML = message;
-        this.setTitle('prompt', '');
+        this.setTitle('prompt', title);
         elements.promptOk.textContent = evt.yesText ? evt.yesText : _('ok');
         elements.promptCancel.textContent = evt.noText ?
           evt.noText : _('cancel');
@@ -209,7 +209,7 @@ var ModalDialog = {
       case 'confirm':
         elements.confirm.classList.add('visible');
         elements.confirmMessage.innerHTML = message;
-        this.setTitle('confirm', '');
+        this.setTitle('confirm', title);
         elements.confirmOk.textContent = evt.yesText ? evt.yesText : _('ok');
         elements.confirmCancel.textContent = evt.noText ?
           evt.noText : _('cancel');
diff --git a/apps/system/test/unit/modal_dialog_test.js b/apps/system/test/unit/modal_dialog_test.js
index b95b25b8b261..bf9326d37f65 100644
--- a/apps/system/test/unit/modal_dialog_test.js
+++ b/apps/system/test/unit/modal_dialog_test.js
@@ -75,6 +75,9 @@ suite('system/ModalDialog >', function() {
     });
 
     assert.isTrue(ModalDialog.elements.selectOne.classList.contains('visible'));
+    assert.isNull(
+      ModalDialog.elements.selectOneTitle.innerHTML.match(
+        testObject.dialogTitle));
 
     ModalDialogCleanUp();
   });
@@ -89,6 +92,9 @@ suite('system/ModalDialog >', function() {
     );
 
     assert.isTrue(ModalDialog.elements.confirm.classList.contains('visible'));
+    assert.isNotNull(
+      ModalDialog.elements.confirmTitle.innerHTML.match(
+        testObject.dialogTitle));
     assert.isNotNull(
       ModalDialog.elements.confirmMessage.innerHTML.match(
         testObject.dialogText));
@@ -108,6 +114,8 @@ suite('system/ModalDialog >', function() {
       testObject.dialogTitle);
 
     assert.isTrue(ModalDialog.elements.alert.classList.contains('visible'));
+    assert.isNotNull(
+      ModalDialog.elements.alertTitle.innerHTML.match(testObject.dialogTitle));
     assert.isNotNull(
       ModalDialog.elements.alertMessage.innerHTML.match(testObject.dialogText));
 
@@ -124,6 +132,9 @@ suite('system/ModalDialog >', function() {
     );
 
     assert.isTrue(ModalDialog.elements.prompt.classList.contains('visible'));
+    assert.isNotNull(
+      ModalDialog.elements.promptTitle.innerHTML.match(
+        testObject.dialogTitle));
     assert.isNotNull(
       ModalDialog.elements.promptMessage.innerHTML.match(
         testObject.dialogText));