[TopAppBar] Fix app:expanded=false not working

When an AppBarLayout is loaded, it will load the attribute and set up a non-forcing pending action. However when it's measured, the current scroll state will be saved and restored, which will override the non-forcing pending action. This behavior is incorrect - if there's a pending action, we shouldn't restore the scrolling state because it's meant to be changed.

Resolves #2629

PiperOrigin-RevId: 440960254
(cherry picked from commit 05be1b9)

Author: drchen

lib/java/com/google/android/material/appbar/AppBarLayout.java

@@ -537,7 +537,12 @@ private boolean hasCollapsibleChild() {
 
   private void invalidateScrollRanges() {
     // Saves the current scrolling state when we need to recalculate scroll ranges
-    SavedState savedState = behavior == null || totalScrollRange == INVALID_SCROLL_RANGE
+    // If the total scroll range is not known yet, the ABL is never scrolled.
+    // If there's a pending action, we should skip this step and respect the pending action.
+    SavedState savedState =
+        behavior == null
+            || totalScrollRange == INVALID_SCROLL_RANGE
+            || pendingAction != PENDING_ACTION_NONE
             ? null : behavior.saveScrollState(AbsSavedState.EMPTY_STATE, this);
     // Invalidate the scroll ranges
     totalScrollRange = INVALID_SCROLL_RANGE;