rename Binding -> Dep

Author: yyx990803

component.json

@@ -18,7 +18,6 @@
     "src/api/global.js",
     "src/api/lifecycle.js",
     "src/batcher.js",
-    "src/binding.js",
     "src/cache.js",
     "src/compiler/compile.js",
     "src/compiler/transclude.js",
@@ -53,6 +52,7 @@
     "src/instance/init.js",
     "src/instance/scope.js",
     "src/observer/array.js",
+    "src/observer/dep.js",
     "src/observer/index.js",
     "src/observer/object.js",
     "src/parsers/directive.js",

src/api/lifecycle.js

@@ -125,7 +125,6 @@ function cleanup (vm) {
   vm.$parent =
   vm.$root =
   vm._children =
-  vm._bindings =
   vm._directives = null
   // call the last hook...
   vm._isDestroyed = true

src/instance/scope.js

@@ -1,6 +1,6 @@
 var _ = require('../util')
 var Observer = require('../observer')
-var Binding = require('../binding')
+var Dep = require('../observer/dep')
 
 /**
  * Setup the scope of an instance, which contains:
@@ -197,20 +197,20 @@ exports._initMeta = function () {
  */
 
 exports._defineMeta = function (key, value) {
-  var binding = new Binding()
+  var dep = new Dep()
   Object.defineProperty(this, key, {
     enumerable: true,
     configurable: true,
     get: function metaGetter () {
       if (Observer.target) {
-        Observer.target.addDep(binding)
+        Observer.target.addDep(dep)
       }
       return value
     },
     set: function metaSetter (val) {
       if (val !== value) {
         value = val
-        binding.notify()
+        dep.notify()
       }
     }
   })

src/binding.js src/observer/dep.js

@@ -1,18 +1,18 @@
 var uid = 0
 
 /**
- * A binding is an observable that can have multiple
+ * A dep is an observable that can have multiple
  * directives subscribing to it.
  *
  * @constructor
  */
 
-function Binding () {
+function Dep () {
   this.id = ++uid
   this.subs = []
 }
 
-var p = Binding.prototype
+var p = Dep.prototype
 
 /**
  * Add a directive subscriber.
@@ -47,4 +47,4 @@ p.notify = function () {
   }
 }
 
-module.exports = Binding
+module.exports = Dep

src/observer/index.js

@@ -1,6 +1,6 @@
 var _ = require('../util')
 var config = require('../config')
-var Binding = require('../binding')
+var Dep = require('./dep')
 var arrayMethods = require('./array')
 var arrayKeys = Object.getOwnPropertyNames(arrayMethods)
 require('./object')
@@ -58,7 +58,7 @@ function Observer (value, type) {
   this.id = ++uid
   this.value = value
   this.active = true
-  this.bindings = []
+  this.deps = []
   _.define(value, '__ob__', this)
   if (type === ARRAY) {
     var augment = config.proto && _.hasProto
@@ -126,10 +126,10 @@ p.walk = function (obj) {
 
 /**
  * Try to carete an observer for a child value,
- * and if value is array, link binding to the array.
+ * and if value is array, link dep to the array.
  *
  * @param {*} val
- * @return {Binding|undefined}
+ * @return {Dep|undefined}
  */
 
 p.observe = function (val) {
@@ -160,9 +160,9 @@ p.observeArray = function (items) {
 p.convert = function (key, val) {
   var ob = this
   var childOb = ob.observe(val)
-  var binding = new Binding()
+  var dep = new Dep()
   if (childOb) {
-    childOb.bindings.push(binding)
+    childOb.deps.push(dep)
   }
   Object.defineProperty(ob.value, key, {
     enumerable: true,
@@ -171,40 +171,40 @@ p.convert = function (key, val) {
       // Observer.target is a watcher whose getter is
       // currently being evaluated.
       if (ob.active && Observer.target) {
-        Observer.target.addDep(binding)
+        Observer.target.addDep(dep)
       }
       return val
     },
     set: function (newVal) {
       if (newVal === val) return
-      // remove binding from old value
+      // remove dep from old value
       var oldChildOb = val && val.__ob__
       if (oldChildOb) {
-        var oldBindings = oldChildOb.bindings
-        oldBindings.splice(oldBindings.indexOf(binding), 1)
+        var oldDeps = oldChildOb.deps
+        oldDeps.splice(oldDeps.indexOf(dep), 1)
       }
       val = newVal
-      // add binding to new value
+      // add dep to new value
       var newChildOb = ob.observe(newVal)
       if (newChildOb) {
-        newChildOb.bindings.push(binding)
+        newChildOb.deps.push(dep)
       }
-      binding.notify()
+      dep.notify()
     }
   })
 }
 
 /**
- * Notify change on all self bindings on an observer.
+ * Notify change on all self deps on an observer.
  * This is called when a mutable value mutates. e.g.
  * when an Array's mutating methods are called, or an
  * Object's $add/$delete are called.
  */
 
 p.notify = function () {
-  var bindings = this.bindings
-  for (var i = 0, l = bindings.length; i < l; i++) {
-    bindings[i].notify()
+  var deps = this.deps
+  for (var i = 0, l = deps.length; i < l; i++) {
+    deps[i].notify()
   }
 }
 

src/watcher.js

@@ -46,18 +46,18 @@ function Watcher (vm, expression, cb, filters, needSet, deep) {
 var p = Watcher.prototype
 
 /**
- * Add a binding dependency to this directive.
+ * Add a dependency to this directive.
  *
- * @param {Binding} binding
+ * @param {Dep} dep
  */
 
-p.addDep = function (binding) {
-  var id = binding.id
+p.addDep = function (dep) {
+  var id = dep.id
   if (!this.newDeps[id]) {
-    this.newDeps[id] = binding
+    this.newDeps[id] = dep
     if (!this.deps[id]) {
-      this.deps[id] = binding
-      binding.addSub(this)
+      this.deps[id] = dep
+      dep.addSub(this)
     }
   }
 }

test/unit/specs/api/lifecycle_spec.js

@@ -168,7 +168,6 @@ if (_.inBrowser) {
         expect(vm.$parent).toBeNull()
         expect(vm.$root).toBeNull()
         expect(vm._children).toBeNull()
-        expect(vm._bindings).toBeNull()
         expect(vm._directives).toBeNull()
         expect(Object.keys(vm._events).length).toBe(0)
       })

test/unit/specs/binding_spec.js

@@ -1,5 +1,5 @@
-var Vue = require('../../../src/vue')
-var filters = require('../../../src/filters')
+var Vue = require('../../../../src/vue')
+var filters = require('../../../../src/filters')
 
 describe('Filters', function () {
 

test/unit/specs/compile/compile_spec.js test/unit/specs/compiler/compile_spec.js

@@ -0,0 +1,34 @@
+var Dep = require('../../../../src/observer/dep')
+
+describe('Dep', function () {
+
+  var d
+  beforeEach(function () {
+    d = new Dep()
+  })
+
+  it('addSub', function () {
+    var sub = {}
+    d.addSub(sub)
+    expect(d.subs.length).toBe(1)
+    expect(d.subs.indexOf(sub)).toBe(0)
+  })
+
+  it('removeSub', function () {
+    var sub = {}
+    d.addSub(sub)
+    d.removeSub(sub)
+    expect(d.subs.length).toBe(0)
+    expect(d.subs.indexOf(sub)).toBe(-1)
+  })
+
+  it('notify', function () {
+    var sub = {
+      update: jasmine.createSpy('sub')
+    }
+    d.addSub(sub)
+    d.notify()
+    expect(sub.update).toHaveBeenCalled()
+  })
+
+})