Refactor duplicated function expressions for firing events

winjs · Oct 8, 2015 · 3ba3487 · 3ba3487
1 parent 0540950
commit 3ba3487
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/src/js/WinJS/Controls/ListView/_BrowseMode.js b/src/js/WinJS/Controls/ListView/_BrowseMode.js
@@ -662,24 +662,22 @@ define([
                     return dropTarget;
                 },
 
-                _fireDragBetweenEvent: function (index, insertAfterIndex, dataTransfer) {
+                _fireEvent: function (index, insertAfterIndex, dataTransfer, eventName) {
                     var event = _Global.document.createEvent("CustomEvent");
-                    event.initCustomEvent("itemdragbetween", true, true, {
+                    event.initCustomEvent(eventName, true, true, {
                         index: index,
                         insertAfterIndex: insertAfterIndex,
                         dataTransfer: dataTransfer
                     });
                     return this.site.element.dispatchEvent(event);
                 },
 
+                _fireDragBetweenEvent: function (index, insertAfterIndex, dataTransfer) {
+                    return this._fireEvent(index, insertAfterIndex, dataTransfer, "itemdragbetween");
+                },
+
                 _fireDropEvent: function (index, insertAfterIndex, dataTransfer) {
-                    var event = _Global.document.createEvent("CustomEvent");
-                    event.initCustomEvent("itemdragdrop", true, true, {
-                        index: index,
-                        insertAfterIndex: insertAfterIndex,
-                        dataTransfer: dataTransfer
-                    });
-                    return this.site.element.dispatchEvent(event);
+                    return this._fireEvent(index, insertAfterIndex, dataTransfer, "itemdragdrop");
                 },
 
                 _handleExitEvent: function () {