Merge branch 'async-improvements' into dev

yyx990803 · yyx990803 · commit e7a066fe6164 · 2017-04-05T16:04:58.000+08:00
diff --git a/src/core/vdom/helpers/resolve-async-component.js b/src/core/vdom/helpers/resolve-async-component.js
@@ -1,62 +1,100 @@
 /* @flow */
 
-// () => ({
-//   component: import('./xxx.vue'),
-//   delay: 200,
-//   loading: LoadingComponent,
-//   error: ErrorComponent
-// })
-
 import {
   warn,
+  once,
+  isDef,
+  isUndef,
+  isTrue,
   isObject
 } from 'core/util/index'
 
+function ensureCtor (comp, base) {
+  return isObject(comp)
+    ? base.extend(comp)
+    : comp
+}
+
 export function resolveAsyncComponent (
   factory: Function,
   baseCtor: Class<Component>,
   context: Component
 ): Class<Component> | void {
-  if (factory.resolved) {
+  if (isTrue(factory.error) && isDef(factory.errorComp)) {
+    return factory.errorComp
+  }
+
+  if (isDef(factory.resolved)) {
     return factory.resolved
   }
 
-  const cb = () => context.$forceUpdate()
-  if (factory.requested) {
-    // pool callbacks
-    factory.pendingCallbacks.push(cb)
+  if (isTrue(factory.loading) && isDef(factory.loadingComp)) {
+    return factory.loadingComp
+  }
+
+  if (isDef(factory.contexts)) {
+    // already pending
+    factory.contexts.push(context)
   } else {
-    factory.requested = true
-    const cbs = factory.pendingCallbacks = [cb]
+    const contexts = factory.contexts = [context]
     let sync = true
 
-    const resolve = (res: Object | Class<Component>) => {
-      if (isObject(res)) {
-        res = baseCtor.extend(res)
+    const forceRender = () => {
+      for (let i = 0, l = contexts.length; i < l; i++) {
+        contexts[i].$forceUpdate()
       }
+    }
+
+    const resolve = once((res: Object | Class<Component>) => {
       // cache resolved
-      factory.resolved = res
+      factory.resolved = ensureCtor(res, baseCtor)
       // invoke callbacks only if this is not a synchronous resolve
       // (async resolves are shimmed as synchronous during SSR)
       if (!sync) {
-        for (let i = 0, l = cbs.length; i < l; i++) {
-          cbs[i](res)
-        }
+        forceRender()
       }
-    }
+    })
 
-    const reject = reason => {
+    const reject = once(reason => {
       process.env.NODE_ENV !== 'production' && warn(
         `Failed to resolve async component: ${String(factory)}` +
         (reason ? `\nReason: ${reason}` : '')
       )
-    }
+      if (isDef(factory.errorComp)) {
+        factory.error = true
+        forceRender()
+      }
+    })
 
     const res = factory(resolve, reject)
 
-    // handle promise
-    if (res && typeof res.then === 'function' && !factory.resolved) {
-      res.then(resolve, reject)
+    if (isObject(res)) {
+      if (typeof res.then === 'function') {
+        // () => Promise
+        if (isUndef(factory.resolved)) {
+          res.then(resolve, reject)
+        }
+      } else if (isDef(res.component) && typeof res.component.then === 'function') {
+        if (isDef(res.error)) {
+          factory.errorComp = ensureCtor(res.error, baseCtor)
+        }
+
+        if (isDef(res.loading)) {
+          factory.loadingComp = ensureCtor(res.loading, baseCtor)
+          setTimeout(() => {
+            if (isUndef(factory.resolved) && isUndef(factory.error)) {
+              factory.loading = true
+              forceRender()
+            }
+          }, res.delay || 200)
+        }
+
+        if (isDef(res.timeout)) {
+          setTimeout(reject, res.timeout)
+        }
+
+        res.component.then(resolve, reject)
+      }
     }
 
     sync = false
diff --git a/src/shared/util.js b/src/shared/util.js
@@ -250,10 +250,10 @@ export function looseIndexOf (arr: Array<mixed>, val: mixed): number {
  */
 export function once (fn: Function): Function {
   let called = false
-  return () => {
+  return function () {
     if (!called) {
       called = true
-      fn()
+      fn.apply(this, arguments)
     }
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -250,10 +250,10 @@ export function looseIndexOf (arr: Array<mixed>, val: mixed): number {`
`250`	`250`	`*/`
`251`	`251`	`export function once (fn: Function): Function {`
`252`	`252`	`let called = false`
`253`		`- return () => {`
	`253`	`+ return function () {`
`254`	`254`	`if (!called) {`
`255`	`255`	`called = true`
`256`		`- fn()`
	`256`	`+ fn.apply(this, arguments)`
`257`	`257`	`}`
`258`	`258`	`}`
`259`	`259`	`}`