[REF] observer: move batch code out of store, add notifycb

src/observer.ts

@@ -18,7 +18,7 @@ const ModifiedArrayProto = Object.create(ArrayProto);
 for (let method of methodsToPatch) {
   const initialMethod = ArrayProto[method];
   ModifiedArrayProto[method] = function(...args) {
-    this.__observer__.rev++;
+    this.__observer__.notifyChange();
     this.__owl__.rev++;
     let parent = this;
     do {
@@ -46,6 +46,19 @@ for (let method of methodsToPatch) {
 export class Observer {
   rev: number = 1;
   allowMutations: boolean = true;
+  dirty: boolean = false;
+
+  notifyCB() {}
+  notifyChange() {
+    this.rev++;
+    this.dirty = true;
+    Promise.resolve().then(() => {
+      if (this.dirty) {
+        this.dirty = false;
+        this.notifyCB();
+      }
+    });
+  }
 
   observe(value: any, parent?: any) {
     if (value === null) {
@@ -69,7 +82,7 @@ export class Observer {
   set(target: any, key: number | string, value: any) {
     this._addProp(target, key, value);
     target.__owl__.rev++;
-    this.rev++;
+    this.notifyChange();
   }
 
   unobserve(target: any) {
@@ -119,11 +132,11 @@ export class Observer {
           value = newVal;
           self.observe(newVal, obj);
           obj.__owl__.rev!++;
-          self.rev++;
           let parent = obj;
           do {
             parent.__owl__.deepRev++;
           } while ((parent = parent.__owl__.parent));
+          self.notifyChange();
         }
       }
     });

src/store.ts

@@ -22,7 +22,6 @@ export class Store extends EventBus {
   actions: any;
   mutations: any;
   _commitLevel: number = 0;
-  _isMutating: boolean = false;
   history: any[] = [];
   debug: boolean;
   env: any;
@@ -37,6 +36,7 @@ export class Store extends EventBus {
     this.mutations = config.mutations;
     this.env = config.env;
     this.observer = new Observer();
+    this.observer.notifyCB = this.trigger.bind(this, "update");
     this.observer.allowMutations = false;
     this.observer.observe(this.state);
 
@@ -72,8 +72,6 @@ export class Store extends EventBus {
       throw new Error(`[Error] mutation ${type} is undefined`);
     }
     this._commitLevel++;
-    const currentRev = this.observer.rev;
-    this._isMutating = true;
     this.observer.allowMutations = true;
 
     const res = this.mutations[type].call(
@@ -95,14 +93,6 @@ export class Store extends EventBus {
           payload: payload
         });
       }
-      Promise.resolve().then(() => {
-        if (this._isMutating) {
-          this._isMutating = false;
-          if (currentRev !== this.observer.rev) {
-            this.trigger("update", this.state);
-          }
-        }
-      });
     }
     this._commitLevel--;
     return res;