perf($urlRouter): only call $location.port() once

nateabele · nateabele · commit 101012e6c376 · 2014-04-17T06:58:14.000-04:00
diff --git a/src/urlRouter.js b/src/urlRouter.js
@@ -393,8 +393,8 @@ function $UrlRouterProvider(   $locationProvider,   $urlMatcherFactory) {
           return url;
         }
 
-        var slash = (!isHtml5 && url ? '/' : ''),
-            port  = $location.port() == 80 || $location.port() == 443 ? '' : ':' + $location.port();
+        var slash = (!isHtml5 && url ? '/' : ''), port = $location.port();
+        port = (port === 80 || port === 443 ? '' : ':' + port);
 
         return [$location.protocol(), '://', $location.host(), port, slash, url].join('');
       }

Original file line number	Diff line number	Diff line change
`@@ -393,8 +393,8 @@ function $UrlRouterProvider( $locationProvider, $urlMatcherFactory) {`
`393`	`393`	`return url;`
`394`	`394`	`}`
`395`	`395`
`396`		`- var slash = (!isHtml5 && url ? '/' : ''),`
`397`		`- port = $location.port() == 80 \|\| $location.port() == 443 ? '' : ':' + $location.port();`
	`396`	`+ var slash = (!isHtml5 && url ? '/' : ''), port = $location.port();`
	`397`	`+ port = (port === 80 \|\| port === 443 ? '' : ':' + port);`
`398`	`398`
`399`	`399`	`return [$location.protocol(), '://', $location.host(), port, slash, url].join('');`
`400`	`400`	`}`