CrOS GMC: Hide artist label when both artist and artwork are empty.

Bug: 1144325
Change-Id: I59caaf4c860adce66fcda090c46a6dd91dc37ce5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533284Reviewed-by: Tommy Steimel <steimel@chromium.org>
Commit-Queue: Jazz Xu <jazzhsu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#832115}

components/media_message_center/media_notification_view_impl.cc

@@ -432,6 +432,7 @@ void MediaNotificationViewImpl::UpdateWithMediaMetadata(
 
   container_->OnMediaSessionMetadataChanged(metadata);
 
+  MaybeShowOrHideArtistLabel();
   PreferredSizeChanged();
   Layout();
   SchedulePaint();
@@ -463,6 +464,7 @@ void MediaNotificationViewImpl::UpdateWithMediaArtwork(
 
   container_->OnMediaArtworkChanged(image);
 
+  MaybeShowOrHideArtistLabel();
   PreferredSizeChanged();
   Layout();
   SchedulePaint();
@@ -794,6 +796,13 @@ void MediaNotificationViewImpl::ButtonPressed(views::Button* button) {
     item_->OnMediaSessionActionButtonPressed(GetActionFromButtonTag(*button));
 }
 
+void MediaNotificationViewImpl::MaybeShowOrHideArtistLabel() {
+  if (!is_cros_)
+    return;
+
+  artist_label_->SetVisible(!artist_label_->GetText().empty() || has_artwork_);
+}
+
 std::vector<views::View*> MediaNotificationViewImpl::GetButtons() {
   auto buttons = button_row_->children();
   buttons.insert(buttons.cbegin(),

components/media_message_center/media_notification_view_impl.h

@@ -134,6 +134,8 @@ class COMPONENT_EXPORT(MEDIA_MESSAGE_CENTER) MediaNotificationViewImpl
 
   void ButtonPressed(views::Button* button);
 
+  void MaybeShowOrHideArtistLabel();
+
   // Returns the buttons contained in the button row and playback button
   // container.
   std::vector<views::View*> GetButtons();