Bug 1321981 - [RTL] ViewPager Support - part 3. Skip set current item if adapter is empty, r=sebastian draft
authormaliu <max@mxli.us>
Tue, 21 Mar 2017 14:58:03 +0800
changeset 502110 aedb97580a91df2bc7b6955f3433d4c9903b1abb
parent 502109 4c2dde631a7bc3093deaaf3c78b02c0890b442cc
child 502111 0e9fa92270bbcb8e726d528ca5f644f61324c0d5
push id50180
push userbmo:max@mxli.us
push dateTue, 21 Mar 2017 08:29:36 +0000
reviewerssebastian
bugs1321981
milestone55.0a1
Bug 1321981 - [RTL] ViewPager Support - part 3. Skip set current item if adapter is empty, r=sebastian MozReview-Commit-ID: L82yxhdG5bd
mobile/android/thirdparty/com/booking/rtlviewpager/RtlViewPager.java
--- a/mobile/android/thirdparty/com/booking/rtlviewpager/RtlViewPager.java
+++ b/mobile/android/thirdparty/com/booking/rtlviewpager/RtlViewPager.java
@@ -124,17 +124,17 @@ public class RtlViewPager extends ViewPa
         unregisterRtlDataSetObserver();
 
         final boolean rtlReady = adapter != null && isRtl();
         if (rtlReady) {
             adapter = new ReverseAdapter(adapter);
             registerRtlDataSetObserver(adapter);
         }
         super.setAdapter(adapter);
-        if (rtlReady) {
+        if (rtlReady && adapter.getCount() > 0) {
             setCurrentItemWithoutNotification(0);
         }
     }
 
     private void setCurrentItemWithoutNotification(int index) {
         suppressOnPageChangeListeners = true;
         setCurrentItem(index, false);
         suppressOnPageChangeListeners = false;