extract ensureURL logic

Author: yyx990803

src/history/base.js

@@ -17,6 +17,7 @@ export class History {
   go: (n: number) => void;
   push: (loc: RawLocation) => void;
   replace: (loc: RawLocation) => void;
+  ensureURL: () => void;
 
   constructor (router: VueRouter, base: ?string) {
     this.router = router
@@ -37,11 +38,13 @@ export class History {
     this.confirmTransition(route, () => {
       this.updateRoute(route)
       cb && cb(route)
+      this.ensureURL()
     })
   }
 
   confirmTransition (route: Route, cb: Function) {
     if (isSameRoute(route, this.current)) {
+      this.ensureURL()
       return
     }
 

src/history/hash.js

@@ -15,12 +15,7 @@ export class HashHistory extends History {
     }
 
     ensureSlash()
-    this.transitionTo(getHash(), route => {
-      // possible redirect on start
-      if (getHash() !== route.fullPath) {
-        replaceHash(route.fullPath)
-      }
-    })
+    this.transitionTo(getHash())
 
     window.addEventListener('hashchange', () => {
       this.onHashChange()
@@ -37,6 +32,12 @@ export class HashHistory extends History {
     }
   }
 
+  ensureURL () {
+    if (getHash() !== this.current.fullPath) {
+      replaceHash(this.current.fullPath)
+    }
+  }
+
   onHashChange () {
     if (!ensureSlash()) {
       return

src/history/html5.js

@@ -19,14 +19,7 @@ export class HTML5History extends History {
   constructor (router: VueRouter, base: ?string) {
     super(router, base)
 
-    const initialLocation = getLocation(this.base)
-    this.transitionTo(initialLocation, route => {
-      // possible redirect on start
-      const url = cleanPath(this.base + this.current.fullPath)
-      if (initialLocation !== url) {
-        replaceState(url)
-      }
-    })
+    this.transitionTo(getLocation(this.base))
 
     const expectScroll = router.options.scrollBehavior
     window.addEventListener('popstate', e => {
@@ -66,6 +59,12 @@ export class HTML5History extends History {
     })
   }
 
+  ensureURL () {
+    if (getLocation(this.base) !== this.current.fullPath) {
+      replaceState(cleanPath(this.base + this.current.fullPath))
+    }
+  }
+
   handleScroll (to: Route, from: Route, isPop: boolean) {
     const router = this.router
     if (!router.app) {