watcher errors should be handled in production mode as well

yyx990803 · yyx990803 · commit d4a167e7c6c4 · 2016-07-06T16:09:01.000-04:00
diff --git a/src/core/observer/watcher.js b/src/core/observer/watcher.js
@@ -99,12 +99,12 @@ export default class Watcher {
             this.vm
           )
         }
-        /* istanbul ignore else */
-        if (config.errorHandler) {
-          config.errorHandler.call(null, e, this.vm)
-        } else {
-          throw e
-        }
+      }
+      /* istanbul ignore else */
+      if (config.errorHandler) {
+        config.errorHandler.call(null, e, this.vm)
+      } else {
+        throw e
       }
       // return old value when evaluation fails so the current UI is preserved
       // if the error was somehow handled by user

Original file line number	Diff line number	Diff line change
`@@ -99,12 +99,12 @@ export default class Watcher {`
`99`	`99`	`this.vm`
`100`	`100`	`)`
`101`	`101`	`}`
`102`		`- /* istanbul ignore else */`
`103`		`- if (config.errorHandler) {`
`104`		`- config.errorHandler.call(null, e, this.vm)`
`105`		`- } else {`
`106`		`- throw e`
`107`		`- }`
	`102`	`+ }`
	`103`	`+ /* istanbul ignore else */`
	`104`	`+ if (config.errorHandler) {`
	`105`	`+ config.errorHandler.call(null, e, this.vm)`
	`106`	`+ } else {`
	`107`	`+ throw e`
`108`	`108`	`}`
`109`	`109`	`// return old value when evaluation fails so the current UI is preserved`
`110`	`110`	`// if the error was somehow handled by user`