[IMP] web: performance improvement with bus

The bus class listen for many mouse/keyboard events and trigger them on
itself.  This was intended for the main bus, but not for custom buses.

This commit should improve the performance when using many buses.

1 files changed, 15 insertions, 15 deletions

diff --git a/addons/web/static/src/js/framework/core.js b/addons/web/static/src/js/framework/core.js
index 437bfffb87f..7bd4800ec98 100644
--- a/addons/web/static/src/js/framework/core.js
+++ b/addons/web/static/src/js/framework/core.js
@@ -21,23 +21,23 @@ var _lt = translation._lt;
 var Bus = Class.extend(mixins.EventDispatcherMixin, {
     init: function() {
         mixins.EventDispatcherMixin.init.call(this, parent);
-        var self = this;
-        // TODO fme: allow user to bind keys for some global actions.
-        //           check gtk bindings
-        // http://unixpapa.com/js/key.html
-        _.each('click,dblclick,keydown,keypress,keyup'.split(','), function(evtype) {
-            $('html').on(evtype, function(ev) {
-                self.trigger(evtype, ev);
-            });
-        });
-        _.each('resize,scroll'.split(','), function(evtype) {
-            $(window).on(evtype, function(ev) {
-                self.trigger(evtype, ev);
-            });
-        });
     },
 });
 
+var main_bus = new Bus ();
+
+_.each('click,dblclick,keydown,keypress,keyup'.split(','), function(evtype) {
+    $('html').on(evtype, function(ev) {
+        main_bus.trigger(evtype, ev);
+    });
+});
+_.each('resize,scroll'.split(','), function(evtype) {
+    $(window).on(evtype, function(ev) {
+        main_bus.trigger(evtype, ev);
+    });
+});
+
+
 // Underscore customization
 //-------------------------------------------------------------------------
 _.str.toBoolElse = function (str, elseValues, trueValues, falseValues) {
@@ -172,7 +172,7 @@ return {
     Class: Class,
     Bus: Bus,
     mixins: mixins,
-    bus: new Bus (),
+    bus: main_bus,
     _t: _t,
     _lt: _lt,