From e4cc4a944fa7680e862fab6b3c1cbd0e2c7932c1 Mon Sep 17 00:00:00 2001
From: Andrew Bonventre <andybons@chromium.org>
Date: Wed, 9 Dec 2015 14:34:21 -0500
Subject: [PATCH] Don't send request for avatar unless they are enabled

While the avatar img tag was itself hidden correctly,
a request was still being made to fetch it when a message
was stamped causig noise in the console. This fixes that.

Change-Id: I94aac49bf5de21538ef277c79903cbff88e96998
---
 polygerrit-ui/app/elements/gr-message.html | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/polygerrit-ui/app/elements/gr-message.html b/polygerrit-ui/app/elements/gr-message.html
index 4a36e162e6..f07542c2db 100644
--- a/polygerrit-ui/app/elements/gr-message.html
+++ b/polygerrit-ui/app/elements/gr-message.html
@@ -116,7 +116,7 @@ limitations under the License.
       }
     </style>
     <div class$="[[_computeClass(expanded, showAvatar)]]">
-      <img class="avatar" src$="[[_computeAvatarURL(message.author)]]">
+      <img class="avatar" src$="[[_computeAvatarURL(message.author, showAvatar)]]">
       <div class="contentContainer">
         <div class="name" id="name">[[message.author.name]]</div>
         <div class="content">
@@ -223,8 +223,8 @@ limitations under the License.
         return classes.join(' ');
       },
 
-      _computeAvatarURL: function(author) {
-        if (!author) { return '' }
+      _computeAvatarURL: function(author, showAvatar) {
+        if (!showAvatar || !author) { return '' }
         return '/accounts/' + author.email + '/avatar?s=100';
       },