From 40665d5f11d50a2e96c13a8e68798711ba88917f Mon Sep 17 00:00:00 2001
From: "yan.kong" <hcanyz@163.com>
Date: Tue, 7 Jul 2020 09:31:02 +0800
Subject: [PATCH] =?UTF-8?q?fix(setNoScrolling):=20=E8=A7=A3=E5=86=B3Virtua?=
 =?UTF-8?q?lLayoutManager.setNoScrolling(=E8=AE=A9recyclerView=E7=BB=98?=
 =?UTF-8?q?=E5=88=B6=E6=89=80=E6=9C=89item)=E4=B8=8D=E7=94=9F=E6=95=88?=
 =?UTF-8?q?=EF=BC=8C=E5=B9=B6=E4=B8=94=E6=97=A0=E9=99=90=E5=BE=AA=E7=8E=AF?=
 =?UTF-8?q?onMeasure=E9=97=AE=E9=A2=98?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../vlayout/example/VLayoutActivity.java      |  65 ++++++-----
 .../res/layout/main_noscroll_activity.xml     | 107 ++++++++++++++++++
 .../android/vlayout/VirtualLayoutManager.java |  15 ++-
 3 files changed, 156 insertions(+), 31 deletions(-)
 create mode 100644 examples/src/main/res/layout/main_noscroll_activity.xml

diff --git a/examples/src/main/java/com/alibaba/android/vlayout/example/VLayoutActivity.java b/examples/src/main/java/com/alibaba/android/vlayout/example/VLayoutActivity.java
index daeb7975..03bcf27f 100644
--- a/examples/src/main/java/com/alibaba/android/vlayout/example/VLayoutActivity.java
+++ b/examples/src/main/java/com/alibaba/android/vlayout/example/VLayoutActivity.java
@@ -24,27 +24,6 @@
 
 package com.alibaba.android.vlayout.example;
 
-import com.alibaba.android.vlayout.DelegateAdapter;
-import com.alibaba.android.vlayout.LayoutHelper;
-import com.alibaba.android.vlayout.RecyclablePagerAdapter;
-import com.alibaba.android.vlayout.VirtualLayoutManager;
-import com.alibaba.android.vlayout.VirtualLayoutManager.LayoutParams;
-import com.alibaba.android.vlayout.extend.LayoutManagerCanScrollListener;
-import com.alibaba.android.vlayout.extend.PerformanceMonitor;
-import com.alibaba.android.vlayout.extend.ViewLifeCycleListener;
-import com.alibaba.android.vlayout.layout.ColumnLayoutHelper;
-import com.alibaba.android.vlayout.layout.FixLayoutHelper;
-import com.alibaba.android.vlayout.layout.FloatLayoutHelper;
-import com.alibaba.android.vlayout.layout.GridLayoutHelper;
-import com.alibaba.android.vlayout.layout.LinearLayoutHelper;
-import com.alibaba.android.vlayout.layout.OnePlusNLayoutHelper;
-import com.alibaba.android.vlayout.layout.RangeGridLayoutHelper;
-import com.alibaba.android.vlayout.layout.RangeGridLayoutHelper.GridRangeStyle;
-import com.alibaba.android.vlayout.layout.ScrollFixLayoutHelper;
-import com.alibaba.android.vlayout.layout.SingleLayoutHelper;
-import com.alibaba.android.vlayout.layout.StaggeredGridLayoutHelper;
-import com.alibaba.android.vlayout.layout.StickyLayoutHelper;
-
 import android.app.Activity;
 import android.content.Context;
 import android.graphics.Color;
@@ -59,7 +38,6 @@
 import android.support.v7.widget.RecyclerView;
 import android.text.TextUtils;
 import android.util.Log;
-import android.util.Pair;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
@@ -67,7 +45,27 @@
 import android.widget.EditText;
 import android.widget.TextView;
 
-import java.util.ArrayList;
+import com.alibaba.android.vlayout.DelegateAdapter;
+import com.alibaba.android.vlayout.LayoutHelper;
+import com.alibaba.android.vlayout.RecyclablePagerAdapter;
+import com.alibaba.android.vlayout.VirtualLayoutManager;
+import com.alibaba.android.vlayout.VirtualLayoutManager.LayoutParams;
+import com.alibaba.android.vlayout.extend.LayoutManagerCanScrollListener;
+import com.alibaba.android.vlayout.extend.PerformanceMonitor;
+import com.alibaba.android.vlayout.extend.ViewLifeCycleListener;
+import com.alibaba.android.vlayout.layout.ColumnLayoutHelper;
+import com.alibaba.android.vlayout.layout.FixLayoutHelper;
+import com.alibaba.android.vlayout.layout.FloatLayoutHelper;
+import com.alibaba.android.vlayout.layout.GridLayoutHelper;
+import com.alibaba.android.vlayout.layout.LinearLayoutHelper;
+import com.alibaba.android.vlayout.layout.OnePlusNLayoutHelper;
+import com.alibaba.android.vlayout.layout.RangeGridLayoutHelper;
+import com.alibaba.android.vlayout.layout.RangeGridLayoutHelper.GridRangeStyle;
+import com.alibaba.android.vlayout.layout.ScrollFixLayoutHelper;
+import com.alibaba.android.vlayout.layout.SingleLayoutHelper;
+import com.alibaba.android.vlayout.layout.StaggeredGridLayoutHelper;
+import com.alibaba.android.vlayout.layout.StickyLayoutHelper;
+
 import java.util.LinkedList;
 import java.util.List;
 
@@ -76,15 +74,17 @@
  */
 public class VLayoutActivity extends Activity {
 
+    private static final boolean TEST_NO_SCROLLING = false;
+
     private static final boolean BANNER_LAYOUT = true;
 
-    private static final boolean FIX_LAYOUT = true;
+    private static final boolean FIX_LAYOUT = true && !TEST_NO_SCROLLING;
 
     private static final boolean LINEAR_LAYOUT = true;
 
     private static final boolean SINGLE_LAYOUT = true;
 
-    private static final boolean FLOAT_LAYOUT = true;
+    private static final boolean FLOAT_LAYOUT = true && !TEST_NO_SCROLLING;
 
     private static final boolean ONEN_LAYOUT = true;
 
@@ -92,7 +92,7 @@ public class VLayoutActivity extends Activity {
 
     private static final boolean GRID_LAYOUT = true;
 
-    private static final boolean STICKY_LAYOUT = true;
+    private static final boolean STICKY_LAYOUT = true && !TEST_NO_SCROLLING;
 
     private static final boolean STAGGER_LAYOUT = true;
 
@@ -110,10 +110,14 @@ public class VLayoutActivity extends Activity {
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
-        setContentView(R.layout.main_activity);
+        if (TEST_NO_SCROLLING) {
+            setContentView(R.layout.main_noscroll_activity);
+        } else {
+            setContentView(R.layout.main_activity);
+        }
 
         mSwipeRefreshLayout = (SwipeRefreshLayout) findViewById(R.id.swipe_container);
-        ;
+
         mFirstText = (TextView) findViewById(R.id.first);
         mLastText = (TextView) findViewById(R.id.last);
         mCountText = (TextView) findViewById(R.id.count);
@@ -122,6 +126,11 @@ protected void onCreate(Bundle savedInstanceState) {
         final RecyclerView recyclerView = (RecyclerView) findViewById(R.id.main_view);
 
         final VirtualLayoutManager layoutManager = new VirtualLayoutManager(this);
+
+        if (TEST_NO_SCROLLING) {
+            layoutManager.setNoScrolling(true);
+        }
+
         layoutManager.setPerformanceMonitor(new PerformanceMonitor() {
 
             long start;
diff --git a/examples/src/main/res/layout/main_noscroll_activity.xml b/examples/src/main/res/layout/main_noscroll_activity.xml
new file mode 100644
index 00000000..8d54025e
--- /dev/null
+++ b/examples/src/main/res/layout/main_noscroll_activity.xml
@@ -0,0 +1,107 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  ~ MIT License
+  ~
+  ~ Copyright (c) 2016 Alibaba Group
+  ~
+  ~ Permission is hereby granted, free of charge, to any person obtaining a copy
+  ~ of this software and associated documentation files (the "Software"), to deal
+  ~ in the Software without restriction, including without limitation the rights
+  ~ to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+  ~ copies of the Software, and to permit persons to whom the Software is
+  ~ furnished to do so, subject to the following conditions:
+  ~
+  ~ The above copyright notice and this permission notice shall be included in all
+  ~ copies or substantial portions of the Software.
+  ~
+  ~ THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+  ~ IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+  ~ FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+  ~ AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+  ~ LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+  ~ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+  ~ SOFTWARE.
+  -->
+
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent">
+
+    <android.support.v4.widget.SwipeRefreshLayout
+            android:id="@+id/swipe_container"
+            android:layout_width="match_parent"
+            android:layout_height="match_parent">
+
+        <android.support.v4.widget.NestedScrollView
+            android:layout_width="match_parent"
+            android:layout_height="match_parent">
+
+            <android.support.v7.widget.RecyclerView
+                android:id="@+id/main_view"
+                android:layout_width="match_parent"
+                android:layout_height="match_parent"
+                android:background="#aaaaaa"
+                android:clipToPadding="true"
+                android:paddingLeft="0dp"
+                android:paddingRight="0dp"
+                android:requiresFadingEdge="none"
+                android:scrollbars="vertical" />
+        </android.support.v4.widget.NestedScrollView>
+    </android.support.v4.widget.SwipeRefreshLayout>
+
+
+    <LinearLayout
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:layout_gravity="bottom|center"
+        android:orientation="vertical">
+
+        <LinearLayout
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content">
+
+            <TextView
+                android:id="@+id/first"
+                style="@style/IndicateText"/>
+
+            <TextView
+                android:id="@+id/last"
+                style="@style/IndicateText"/>
+
+            <TextView
+                android:id="@+id/count"
+                style="@style/IndicateText"/>
+        </LinearLayout>
+
+        <LinearLayout
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:background="#33000000">
+
+            <TextView
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:text="Jump Position"/>
+
+            <EditText
+                android:id="@+id/position"
+                android:layout_width="100dp"
+                android:layout_height="wrap_content"
+                android:numeric="integer"/>
+
+            <Button
+                android:id="@+id/jump"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:text="Jump"/>
+
+            <TextView
+                android:id="@+id/total_offset"
+                style="@style/IndicateText"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:background="@android:color/transparent"
+                android:paddingLeft="20dp"/>
+        </LinearLayout>
+    </LinearLayout>
+</FrameLayout>
\ No newline at end of file
diff --git a/vlayout/src/main/java/com/alibaba/android/vlayout/VirtualLayoutManager.java b/vlayout/src/main/java/com/alibaba/android/vlayout/VirtualLayoutManager.java
index 257d960c..42da390d 100644
--- a/vlayout/src/main/java/com/alibaba/android/vlayout/VirtualLayoutManager.java
+++ b/vlayout/src/main/java/com/alibaba/android/vlayout/VirtualLayoutManager.java
@@ -158,12 +158,21 @@ public void setPerformanceMonitor(PerformanceMonitor performanceMonitor) {
     }
 
     public void setNoScrolling(boolean noScrolling) {
+        setAutoMeasureEnabled(!noScrolling);
         this.mNoScrolling = noScrolling;
         mSpaceMeasured = false;
         mMeasuredFullSpace = 0;
         mSpaceMeasuring = false;
     }
 
+    @Override
+    public boolean isAutoMeasureEnabled() {
+        if (mNoScrolling) {
+            return false;
+        }
+        return super.isAutoMeasureEnabled();
+    }
+
     public void setCanScrollVertically(boolean canScrollVertically) {
         this.mCanScrollVertically = canScrollVertically;
     }
@@ -1588,7 +1597,7 @@ public LayoutHelper findLayoutHelperByPosition(int position) {
 
 
     // when set no scrolling, the max size should have limit
-    private static final int MAX_NO_SCROLLING_SIZE = Integer.MAX_VALUE >> 4;
+    private static final int MAX_NO_SCROLLING_SIZE = View.MEASURED_SIZE_MASK >> 2;
 
     private boolean mSpaceMeasured = false;
 
@@ -1647,9 +1656,9 @@ public void onMeasure(RecyclerView.Recycler recycler, RecyclerView.State state,
 
 
         if (getOrientation() == VERTICAL) {
-            super.onMeasure(recycler, state, widthSpec, View.MeasureSpec.makeMeasureSpec(measuredSize, View.MeasureSpec.AT_MOST));
+            super.onMeasure(recycler, state, widthSpec, View.MeasureSpec.makeMeasureSpec(measuredSize, View.MeasureSpec.EXACTLY));
         } else {
-            super.onMeasure(recycler, state, View.MeasureSpec.makeMeasureSpec(measuredSize, View.MeasureSpec.AT_MOST), heightSpec);
+            super.onMeasure(recycler, state, View.MeasureSpec.makeMeasureSpec(measuredSize, View.MeasureSpec.EXACTLY), heightSpec);
         }
     }
 }