perf(router): don't create new serializer every time UrlTree.toString…

… is called (#15565)
angular · Mar 29, 2017 · fd61145 · fd61145
1 parent c828511
commit fd61145
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/packages/router/src/url_tree.ts b/packages/router/src/url_tree.ts
@@ -115,15 +115,15 @@ export class UrlTree {
       /** The fragment of the URL */
       public fragment: string) {}
 
-  get queryParamMap() {
+  get queryParamMap(): ParamMap {
     if (!this._queryParamMap) {
       this._queryParamMap = convertToParamMap(this.queryParams);
     }
     return this._queryParamMap;
   }
 
   /** @docsNotRequired */
-  toString(): string { return new DefaultUrlSerializer().serialize(this); }
+  toString(): string { return DEFAULT_SERIALIZER.serialize(this); }
 }
 
 /**
@@ -294,6 +294,8 @@ export class DefaultUrlSerializer implements UrlSerializer {
   }
 }
 
+const DEFAULT_SERIALIZER = new DefaultUrlSerializer();
+
 export function serializePaths(segment: UrlSegmentGroup): string {
   return segment.segments.map(p => serializePath(p)).join('/');
 }