fix: revert #2713

Closes #2719

examples/lazy-loading-before-mount/app.js

@@ -23,7 +23,9 @@ const router = new VueRouter({
   ]
 })
 
-router.push('/async')
+router.onReady(() => {
+  router.push('/async')
+})
 
 document.getElementById('load-button').addEventListener('click', (event) => {
   new Vue({

examples/lazy-loading/app.js

@@ -43,6 +43,18 @@ const router = new VueRouter({
     { path: '/', component: Home },
     // Just use them normally in the route config
     { path: '/foo', component: Foo },
+    // mulitple parameters, `/` should not be encoded. The name is also important
+    // https://github.com/vuejs/vue-router/issues/2719
+    { path: '/a/:tags*', name: 'tagged', component: () => new Promise(resolve => {
+      setTimeout(() => {
+        resolve({
+          template: `<div>
+            <h2>Lazy with params</h2>
+            <pre id="tagged-path">{{ $route.path }}</pre>
+          </div>`
+        })
+      }, 200)
+    }) },
     // Bar and Baz belong to the same root route
     // and grouped in the same async chunk.
     { path: '/bar', component: Bar,
@@ -63,6 +75,7 @@ new Vue({
         <li><router-link to="/foo">/foo</router-link></li>
         <li><router-link to="/bar">/bar</router-link></li>
         <li><router-link to="/bar/baz">/bar/baz</router-link></li>
+        <li><router-link to="/a/b/c">/a/b/c</router-link></li>
       </ul>
       <router-view class="view"></router-view>
     </div>

src/util/resolve-components.js

@@ -30,7 +30,7 @@ export function resolveAsyncComponents (matched: Array<RouteRecord>): Function {
           match.components[key] = resolvedDef
           pending--
           if (pending <= 0) {
-            next(to)
+            next()
           }
         })
 

test/e2e/specs/lazy-loading.js

@@ -3,7 +3,7 @@ module.exports = {
     browser
       .url('http://localhost:8080/lazy-loading/')
       .waitForElementVisible('#app', 1000)
-      .assert.count('li a', 4)
+      .assert.count('li a', 5)
       .assert.containsText('.view', 'home')
 
       .click('li:nth-child(2) a')
@@ -28,6 +28,18 @@ module.exports = {
       .waitForElementVisible('#app', 1000)
       .assert.containsText('.view', 'This is Bar!')
       .assert.containsText('.view h3', 'Baz')
+
+      // lazy loading with dynamic params: https://github.com/vuejs/vue-router/issues/2719
+      // direct visit
+      .url('http://localhost:8080/lazy-loading/a/b/c')
+      .waitForElementVisible('#app', 1000)
+      .assert.containsText('.view', '/a/b/c')
+      // coming from another url
+      .url('http://localhost:8080/lazy-loading/')
+      .waitForElementVisible('#app', 1000)
+      .click('li:nth-child(5) a')
+      .waitForElementVisible('#tagged-path', 1000)
+      .assert.containsText('.view', '/a/b/c')
       .end()
   }
 }