fix(button): reuse _getHostElement() to avoid redundant elementRef.na…

…tiveElement calls (angular#2625)
crisbeto · Jan 19, 2017 · c7d1c17 · c7d1c17
1 parent e2ad3a0
commit c7d1c17
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/lib/button/button.ts b/src/lib/button/button.ts
@@ -82,7 +82,7 @@ export class MdButton {
 
   _setElementColor(color: string, isAdd: boolean) {
     if (color != null && color != '') {
-      this._renderer.setElementClass(this._elementRef.nativeElement, `md-${color}`, isAdd);
+      this._renderer.setElementClass(this._getHostElement(), `md-${color}`, isAdd);
     }
   }
 
@@ -96,15 +96,15 @@ export class MdButton {
 
   /** Focuses the button. */
   focus(): void {
-    this._renderer.invokeElementMethod(this._elementRef.nativeElement, 'focus');
+    this._renderer.invokeElementMethod(this._getHostElement(), 'focus');
   }
 
   _getHostElement() {
     return this._elementRef.nativeElement;
   }
 
   _isRoundButton() {
-    const el = this._elementRef.nativeElement;
+    const el = this._getHostElement();
     return el.hasAttribute('md-icon-button') ||
         el.hasAttribute('md-fab') ||
         el.hasAttribute('md-mini-fab');