Merge Firefox frontend test fixes

The fix was done on another branch to show that it completely solves tests
failures introduced by 2be873e3c7.

All remaining failures in this branch are due to other changes made after
**1.6.6**, namely from fe08d2a1db included onwards.

tests/frontend/specs/bold.js

@@ -44,7 +44,7 @@ describe("bold button", function(){
     //select this text element
     $firstTextElement.sendkeys('{selectall}');
 
-    if(inner$(window)[0].bowser.firefox || inner$(window)[0].bowser.modernIE){ // if it's a mozilla or IE 
+    if(inner$(window)[0].bowser.modernIE){ // if it's IE
       var evtType = "keypress";
     }else{
       var evtType = "keydown";

tests/frontend/specs/indentation.js

@@ -15,7 +15,7 @@ describe("indentation button", function(){
     //select this text element
     $firstTextElement.sendkeys('{selectall}');
 
-    if(inner$(window)[0].bowser.firefox || inner$(window)[0].bowser.modernIE){ // if it's a mozilla or IE
+    if(inner$(window)[0].bowser.modernIE){ // if it's IE
       var evtType = "keypress";
     }else{
       var evtType = "keydown";
@@ -325,7 +325,7 @@ describe("indentation button", function(){
 
 function pressEnter(){
   var inner$ = helper.padInner$;
-  if(inner$(window)[0].bowser.firefox || inner$(window)[0].bowser.modernIE){ // if it's a mozilla or IE
+  if(inner$(window)[0].bowser.modernIE){ // if it's IE
     var evtType = "keypress";
   }else{
     var evtType = "keydown";

tests/frontend/specs/italic.js

@@ -44,7 +44,7 @@ describe("italic some text", function(){
     //select this text element
     $firstTextElement.sendkeys('{selectall}');
 
-    if(inner$(window)[0].bowser.firefox || inner$(window)[0].bowser.modernIE){ // if it's a mozilla or IE  
+    if(inner$(window)[0].bowser.modernIE){ // if it's IE
       var evtType = "keypress";
     }else{
       var evtType = "keydown";

tests/frontend/specs/ordered_list.js

@@ -111,7 +111,7 @@ describe("assign ordered list", function(){
 
   var triggerCtrlShiftShortcut = function(shortcutChar) {
     var inner$ = helper.padInner$;
-    if(inner$(window)[0].bowser.firefox || inner$(window)[0].bowser.modernIE) { // if it's a mozilla or IE
+    if(inner$(window)[0].bowser.modernIE) { // if it's IE
       var evtType = "keypress";
     }else{
       var evtType = "keydown";

tests/frontend/specs/scroll.js

@@ -513,7 +513,7 @@ describe('scroll when focus line is out of viewport', function () {
   var pressKey = function(keyCode, shiftIsPressed){
     var inner$ = helper.padInner$;
     var evtType;
-    if(inner$(window)[0].bowser.firefox || inner$(window)[0].bowser.modernIE){ // if it's a mozilla or IE
+    if(inner$(window)[0].bowser.modernIE){ // if it's IE
       evtType = 'keypress';
     }else{
       evtType = 'keydown';

tests/frontend/specs/select_formatting_buttons.js

@@ -88,7 +88,7 @@ describe("select formatting buttons when selection has style applied", function(
     //select this text element
     $firstTextElement.sendkeys('{selectall}');
 
-    if(inner$(window)[0].bowser.firefox || inner$(window)[0].bowser.modernIE){ // if it's a mozilla or IE
+    if(inner$(window)[0].bowser.modernIE){ // if it's IE
       var evtType = "keypress";
     }else{
       var evtType = "keydown";

tests/frontend/specs/undo.js

@@ -44,11 +44,11 @@ describe("undo button", function(){
     var modifiedValue = $firstTextElement.text(); // get the modified value
     expect(modifiedValue).not.to.be(originalValue); // expect the value to change
 
-    if(inner$(window)[0].bowser.firefox){ // if it's a mozilla browser
+    /*
-      var evtType = "keypress";
+     * ACHTUNG: this is the only place in the test codebase in which a keydown
-    }else{
+     * is sent for IE. Everywhere else IE uses keypress.
-      var evtType = "keydown";
+     */
-    }
+    var evtType = "keydown";
 
     var e = inner$.Event(evtType);
     e.ctrlKey = true; // Control key