[Button] Fix progress indicator is not shown when set as the icon

MaterialButton is using TextViewCompat.setCompoundDrawablesRelative() under the hood to implement setIcon() operation. TextViewCompat won't call its compound drawables' setVisible() method, on which we rely to start progress indicator drawables automatically.

Calls setVisible() explicitly to start progressing.

Also adds a demo to catalog.

Resolves #2095

PiperOrigin-RevId: 383483364

Author: drchen

catalog/java/io/material/catalog/progressindicator/ProgressIndicatorStandaloneDemoFragment.java

@@ -24,6 +24,7 @@
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.annotation.StyleRes;
+import com.google.android.material.button.MaterialButton;
 import com.google.android.material.chip.Chip;
 import com.google.android.material.progressindicator.CircularProgressIndicatorSpec;
 import com.google.android.material.progressindicator.IndeterminateDrawable;
@@ -50,17 +51,20 @@ public View onCreateDemoView(
 
     CircularProgressIndicatorSpec spec =
         new CircularProgressIndicatorSpec(getContext(), /*attrs=*/ null, 0, getSpecStyleResId());
+    Chip chip = view.findViewById(R.id.cat_progress_indicator_chip);
+    chip.setChipIcon(IndeterminateDrawable.createCircularDrawable(getContext(), spec));
+
     IndeterminateDrawable<CircularProgressIndicatorSpec> progressIndicatorDrawable =
         IndeterminateDrawable.createCircularDrawable(getContext(), spec);
-
-    Chip chip = view.findViewById(R.id.cat_progress_indicator_chip);
-    chip.setChipIcon(progressIndicatorDrawable);
+    MaterialButton button = view.findViewById(R.id.cat_progress_indicator_button);
+    button.setIcon(progressIndicatorDrawable);
 
     SwitchMaterial chipIconSwitch =
         view.findViewById(R.id.cat_progress_indicator_standalone_chip_switch);
     chipIconSwitch.setOnCheckedChangeListener(
         (buttonView, isChecked) -> {
           chip.setChipIconVisible(isChecked);
+          button.setIcon(isChecked ? progressIndicatorDrawable : null);
         });
     return view;
   }

catalog/java/io/material/catalog/progressindicator/res/layout/cat_progress_indicator_standalone_fragment.xml

@@ -32,6 +32,14 @@
     android:ellipsize="end"
     android:text="@string/cat_progress_indicator_standalone_indeterminate_chip"/>
 
+  <com.google.android.material.button.MaterialButton
+    android:id="@+id/cat_progress_indicator_button"
+    style="@style/Widget.MaterialComponents.Button.OutlinedButton.Icon"
+    android:layout_width="wrap_content"
+    android:layout_height="wrap_content"
+    android:ellipsize="end"
+    android:text="@string/cat_progress_indicator_standalone_indeterminate_button"/>
+
   <com.google.android.material.switchmaterial.SwitchMaterial
     android:id="@+id/cat_progress_indicator_standalone_chip_switch"
     android:layout_width="wrap_content"

catalog/java/io/material/catalog/progressindicator/res/values/strings.xml

@@ -120,6 +120,10 @@
     description="An example of using progress indicator drawable as chip icon [CHAR LIMIT=NONE]">
     Indeterminate Chip
   </string>
+  <string name="cat_progress_indicator_standalone_indeterminate_button"
+    description="An example of using progress indicator drawable as button icon [CHAR LIMIT=NONE]">
+    Indeterminate Button
+  </string>
   <string name="cat_progress_indicator_standalone_switch_label"
     description="A material switch to toggle progress indicator drawable on chip [CHAR LIMIT=NONE]">
     Show Icon

lib/java/com/google/android/material/button/MaterialButton.java

@@ -761,6 +761,7 @@ private void updateIcon(boolean needsIconReset) {
       int width = iconSize != 0 ? iconSize : icon.getIntrinsicWidth();
       int height = iconSize != 0 ? iconSize : icon.getIntrinsicHeight();
       icon.setBounds(iconLeft, iconTop, iconLeft + width, iconTop + height);
+      icon.setVisible(true, needsIconReset);
     }
 
     // Forced icon update