Bug 1226133 - Synchronise pages container on window resize. r=alberto…

…pq, a=mahe

apps/homescreen/js/pages.js

@@ -31,6 +31,7 @@
     this.pages.addEventListener('click', this);
     this.pages.addEventListener('contextmenu', this);
     this.pages.addEventListener('keydown', this);
+    window.addEventListener('resize', this);
 
     this.done.addEventListener('click', e => {
       e.preventDefault();
@@ -259,6 +260,10 @@
             this.launchCard(e.target);
         }
         break;
+
+      case 'resize':
+        this.pages.synchronise();
+        break;
       }
     }
   };

apps/homescreen/test/unit/pages_test.js

@@ -397,5 +397,14 @@ suite('Pages', () => {
         assert.isFalse(editModeStub.called);
       });
     });
+
+    suite('resize', () => {
+      test('should call pages.synchronise()', () => {
+        var synchroniseCalled = false;
+        pages.pages.synchronise = () => { synchroniseCalled = true; };
+        pages.handleEvent(new CustomEvent('resize'));
+        assert.isTrue(synchroniseCalled);
+      });
+    });
   });
 });