SharesheetTargetButton has multi-line label

We avoid the check failure
Check failed: GetMultiLine()
in Label::SetMaximumWidth(),
called by SharesheetTargetButton::SetLabelProperties()


Bug: 1132748
Change-Id: I9d5c12e01954f1abb80592a70967218ab37c8d2c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2440724
Commit-Queue: Eric Willigers <ericwilligers@chromium.org>
Commit-Queue: Melissa Zhang <melzhang@chromium.org>
Reviewed-by: Melissa Zhang <melzhang@chromium.org>
Auto-Submit: Eric Willigers <ericwilligers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#812612}

chrome/browser/ui/views/sharesheet/sharesheet_target_button.cc

@@ -76,10 +76,11 @@ SharesheetTargetButton::SharesheetTargetButton(
     SetLabelProperties(secondary_label);
     accessible_name = base::StrCat(
         {display_name, base::ASCIIToUTF16(" "), secondary_display_name});
-  } else {
-    // If there is no secondary label, let the initial label stretch across
+    // As there is a secondary label, don't let the initial label stretch across
     // multiple lines.
-    label->SetMultiLine(true);
+    label->SetMultiLine(false);
+    secondary_label->SetMultiLine(false);
+  } else {
     label->SetMaxLines(kButtonMaxLines);
   }
 
@@ -91,11 +92,11 @@ SharesheetTargetButton::SharesheetTargetButton(
 
 void SharesheetTargetButton::SetLabelProperties(views::Label* label) {
   label->SetLineHeight(kButtonLineHeight);
+  label->SetMultiLine(true);
   label->SetMaximumWidth(kButtonTextMaxWidth);
   label->SetBackgroundColor(SK_ColorTRANSPARENT);
   label->SetHandlesTooltips(true);
   label->SetTooltipText(label->GetText());
-  label->SetMultiLine(false);
   label->SetAutoColorReadabilityEnabled(false);
   label->SetHorizontalAlignment(gfx::ALIGN_CENTER);
 }