[update] modify fullscreenchange event

what: why: how:
Chimeejs · Sep 5, 2017 · 6dc1f30 · 6dc1f30
1 parent 91a5fb0
commit 6dc1f30
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/__tests__/index.js b/__tests__/index.js
@@ -659,6 +659,19 @@ describe('isFullscreen and fullscreenElement', () => {
     document.body.removeChild(wrapper);
   });
 
+  test('fullscreen event and fullscreenchange event', () => {
+    const fn = jest.fn();
+    const changeFn = jest.fn();
+    player.on('fullscreen', fn);
+    player.on('fullscreenchange', changeFn);
+    player.fullscreen();
+    expect(fn).toHaveBeenCalledTimes(1);
+    expect(changeFn).toHaveBeenCalledTimes(1);
+    player.fullscreen(false);
+    expect(fn).toHaveBeenCalledTimes(2);
+    expect(changeFn).toHaveBeenCalledTimes(2);
+  });
+
   describe('$video, $container and $wrapper', () => {
     const wrapper = document.createElement('div');
     player = new Chimee(wrapper);

diff --git a/src/helper/const.js b/src/helper/const.js
@@ -73,10 +73,6 @@ export const domEvents = [
   'select',
   'wheel',
   'fullscreenchange',
-  'webkitfullscreenchange',
-  'mozfullscreenchange',
-  'msfullscreenchange',
-  'MSFullscreenChange',
   'contextmenu'
 ];
 export const selfProcessorEvents = [