[NTP] Remove the ScrollView version of the NTP.

chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.ntp;
 
 import android.annotation.SuppressLint;
 import android.content.Context;
 import android.content.res.Configuration;
 import android.content.res.Resources;
@@ skipping 57 matching lines @@
 
 import jp.tomorrowkey.android.gifplayer.BaseGifImage;
 
 /**
  * The native new tab page, represented by some basic data such as title and url, and an Android
  * View that displays the page.
  */
 public class NewTabPageView extends FrameLayout
         implements MostVisitedURLsObserver, OnLayoutChangeListener {
 
-    private static final int SHADOW_COLOR = 0x11000000;
     private static final long SNAP_SCROLL_DELAY_MS = 30;
     private static final String TAG = "Ntp";
 
-    /**
-     * Indicates which UI mode we are using. Should be checked when manipulating some members, as
-     * they may be unused or {@code null} depending on the mode.
-     */
-    private boolean mUseCardsUi;
-
-    // Note: Only one of these will be valid at a time, depending on if we are using the old NTP
-    // (NewTabPageScrollView) or the new NTP with cards (NewTabPageRecyclerView).
-    private NewTabPageScrollView mScrollView;
     private NewTabPageRecyclerView mRecyclerView;
 
     private NewTabPageLayout mNewTabPageLayout;
     private LogoView mSearchProviderLogoView;
     private View mSearchBoxView;
     private ImageView mVoiceSearchButton;
     private MostVisitedLayout mMostVisitedLayout;
     private View mMostVisitedPlaceholder;
     private View mNoSearchLogoSpacer;
 
@@ skipping 105 matching lines @@
      * @param searchProviderHasLogo Whether the search provider has a logo.
      * @param scrollPosition The adapter scroll position to initialize to.
      */
     public void initialize(
             NewTabPageManager manager, Tab tab, boolean searchProviderHasLogo, int scrollPosition) {
         mActivity = tab.getActivity();
         mManager = manager;
         mUiConfig = new UiConfig(this);
         ViewStub stub = (ViewStub) findViewById(R.id.new_tab_page_layout_stub);
 
-        mUseCardsUi = manager.getSuggestionsSource() != null;
-        if (mUseCardsUi) {
-            stub.setLayoutResource(R.layout.new_tab_page_recycler_view);
-            mRecyclerView = (NewTabPageRecyclerView) stub.inflate();
+        assert manager.getSuggestionsSource() != null;
 
-            // Don't attach now, the recyclerView itself will determine when to do it.
-            mNewTabPageLayout =
-                    (NewTabPageLayout) LayoutInflater.from(getContext())
-                            .inflate(R.layout.new_tab_page_layout, mRecyclerView, false);
-            mNewTabPageLayout.setUseCardsUiEnabled(mUseCardsUi);
-            mRecyclerView.setAboveTheFoldView(mNewTabPageLayout);
+        stub.setLayoutResource(R.layout.new_tab_page_recycler_view);
+        mRecyclerView = (NewTabPageRecyclerView) stub.inflate();
 
-            // Tailor the LayoutParams for the snippets UI, as the configuration in the XML is
-            // made for the ScrollView UI.
-            ViewGroup.LayoutParams params = mNewTabPageLayout.getLayoutParams();
-            params.height = ViewGroup.LayoutParams.WRAP_CONTENT;
+        // Don't attach now, the recyclerView itself will determine when to do it.
+        mNewTabPageLayout =
+                (NewTabPageLayout) LayoutInflater.from(getContext())
+                        .inflate(R.layout.new_tab_page_layout, mRecyclerView, false);
+        mRecyclerView.setAboveTheFoldView(mNewTabPageLayout);
 
-            mRecyclerView.setItemAnimator(new DefaultItemAnimator() {
-                @Override
-                public void onAnimationFinished(ViewHolder viewHolder) {
-                    super.onAnimationFinished(viewHolder);
-                    // When removing sections, because the animations are all translations, the
-                    // scroll events don't fire and we can get in the situation where the toolbar
-                    // buttons disappear.
-                    updateSearchBoxOnScroll();
-                }
-            });
-        } else {
-            stub.setLayoutResource(R.layout.new_tab_page_scroll_view);
-            mScrollView = (NewTabPageScrollView) stub.inflate();
-            mScrollView.setBackgroundColor(
-                    ApiCompatibilityUtils.getColor(getResources(), R.color.ntp_bg));
-            mScrollView.enableBottomShadow(SHADOW_COLOR);
-            mNewTabPageLayout = (NewTabPageLayout) findViewById(R.id.ntp_content);
-        }
-        mContextMenuManager = new ContextMenuManager(mActivity, mManager.getNavigationDelegate(),
-                mUseCardsUi ? mRecyclerView : mScrollView);
+        // Tailor the LayoutParams for the snippets UI, as the configuration in the XML is
+        // made for the ScrollView UI.
+        ViewGroup.LayoutParams params = mNewTabPageLayout.getLayoutParams();
+        params.height = ViewGroup.LayoutParams.WRAP_CONTENT;
+
+        mRecyclerView.setItemAnimator(new DefaultItemAnimator() {
+            @Override
+            public void onAnimationFinished(ViewHolder viewHolder) {
+                super.onAnimationFinished(viewHolder);
+                // When removing sections, because the animations are all translations, the
+                // scroll events don't fire and we can get in the situation where the toolbar
+                // buttons disappear.
+                updateSearchBoxOnScroll();
+            }
+        });
+
+        mContextMenuManager =
+                new ContextMenuManager(mActivity, mManager.getNavigationDelegate(), mRecyclerView);
         mActivity.getWindowAndroid().addContextMenuCloseListener(mContextMenuManager);
         manager.addDestructionObserver(new DestructionObserver() {
             @Override
             public void onDestroy() {
                 mActivity.getWindowAndroid().removeContextMenuCloseListener(mContextMenuManager);
             }
         });
 
         mMostVisitedDesign = new MostVisitedDesign(getContext());
         mMostVisitedLayout =
@@ skipping 10 matching lines @@
         initializeBottomToolbar();
 
         mNewTabPageLayout.addOnLayoutChangeListener(this);
         setSearchProviderHasLogo(searchProviderHasLogo);
 
         mPendingLoadTasks++;
         mManager.setMostVisitedURLsObserver(
                 this, mMostVisitedDesign.getNumberOfTiles(searchProviderHasLogo));
 
         // Set up snippets
-        if (mUseCardsUi) {
-            NewTabPageAdapter newTabPageAdapter = new NewTabPageAdapter(mManager, mNewTabPageLayout,
-                    mUiConfig, OfflinePageBridge.getForProfile(Profile.getLastUsedProfile()),
-                    mContextMenuManager);
-            mRecyclerView.setAdapter(newTabPageAdapter);
+        NewTabPageAdapter newTabPageAdapter = new NewTabPageAdapter(mManager, mNewTabPageLayout,
+                mUiConfig, OfflinePageBridge.getForProfile(Profile.getLastUsedProfile()),
+                mContextMenuManager);
+        mRecyclerView.setAdapter(newTabPageAdapter);
 
-            int scrollOffset;
-            if (CardsVariationParameters.isScrollBelowTheFoldEnabled()) {
-                scrollPosition = newTabPageAdapter.getFirstHeaderPosition();
-                scrollOffset = getResources().getDimensionPixelSize(R.dimen.ntp_search_box_height);
-            } else {
-                scrollOffset = 0;
+        int scrollOffset;
+        if (CardsVariationParameters.isScrollBelowTheFoldEnabled()) {
+            scrollPosition = newTabPageAdapter.getFirstHeaderPosition();
+            scrollOffset = getResources().getDimensionPixelSize(R.dimen.ntp_search_box_height);
+        } else {
+            scrollOffset = 0;
+        }
+        mRecyclerView.getLinearLayoutManager().scrollToPositionWithOffset(
+                scrollPosition, scrollOffset);
+
+        mRecyclerView.setUpSwipeToDismiss();
+
+        setupScrollHandling();
+
+        // When the NewTabPageAdapter's data changes we need to invalidate any previous
+        // screen captures of the NewTabPageView.
+        newTabPageAdapter.registerAdapterDataObserver(new AdapterDataObserver() {
+            @Override
+            public void onChanged() {
+                mNewTabPageRecyclerViewChanged = true;
             }
-            mRecyclerView.getLinearLayoutManager().scrollToPositionWithOffset(
-                    scrollPosition, scrollOffset);
 
-            mRecyclerView.setUpSwipeToDismiss();
+            @Override
+            public void onItemRangeChanged(int positionStart, int itemCount) {
+                onChanged();
+            }
 
-            initializeSearchBoxRecyclerViewScrollHandling();
+            @Override
+            public void onItemRangeInserted(int positionStart, int itemCount) {
+                onChanged();
+            }
 
-            // When the NewTabPageAdapter's data changes we need to invalidate any previous
-            // screen captures of the NewTabPageView.
-            newTabPageAdapter.registerAdapterDataObserver(new AdapterDataObserver() {
-                @Override
-                public void onChanged() {
-                    mNewTabPageRecyclerViewChanged = true;
-                }
+            @Override
+            public void onItemRangeRemoved(int positionStart, int itemCount) {
+                onChanged();
+            }
 
-                @Override
-                public void onItemRangeChanged(int positionStart, int itemCount) {
-                    onChanged();
-                }
+            @Override
+            public void onItemRangeMoved(int fromPosition, int toPosition, int itemCount) {
+                onChanged();
+            }
+        });
 
-                @Override
-                public void onItemRangeInserted(int positionStart, int itemCount) {
-                    onChanged();
-                }
-
-                @Override
-                public void onItemRangeRemoved(int positionStart, int itemCount) {
-                    onChanged();
-                }
-
-                @Override
-                public void onItemRangeMoved(int fromPosition, int toPosition, int itemCount) {
-                    onChanged();
-                }
-            });
-        } else {
-            initializeSearchBoxScrollHandling();
-        }
     }
 
     /**
      * Sets up the hint text and event handlers for the search box text view.
      */
     private void initializeSearchBoxTextView() {
         final TextView searchBoxTextView = (TextView) mSearchBoxView
                 .findViewById(R.id.search_box_text);
         String hintText = getResources().getString(R.string.search_or_type_url);
 
@@ skipping 84 matching lines @@
      * the position of the search box.
      *
      * @return the transition percentage
      */
     private float getToolbarTransitionPercentage() {
         // During startup the view may not be fully initialized, so we only calculate the current
         // percentage if some basic view properties (height of the containing view, position of the
         // search box) are sane.
         if (getWrapperView().getHeight() == 0) return 0f;
 
-        if (mUseCardsUi && !mRecyclerView.isFirstItemVisible()) {
+        if (!mRecyclerView.isFirstItemVisible()) {
             // getVerticalScroll is valid only for the RecyclerView if the first item is visible.
             // If the first item is not visible, we must have scrolled quite far and we know the
             // toolbar transition should be 100%. This might be the initial scroll position due to
             // the scroll restore feature, so the search box will not have been laid out yet.
             return 1f;
         }
 
         int searchBoxTop = mSearchBoxView.getTop();
         if (searchBoxTop == 0) return 0f;
 
         // For all other calculations, add the search box padding, because it defines where the
         // visible "border" of the search box is.
         searchBoxTop += mSearchBoxView.getPaddingTop();
 
-        if (!mUseCardsUi) {
-            return MathUtils.clamp(getVerticalScroll() / (float) searchBoxTop, 0f, 1f);
-        }
-
         final int scrollY = getVerticalScroll();
         final float transitionLength =
                 getResources().getDimension(R.dimen.ntp_search_box_transition_length);
         // Tab strip height is zero on phones, nonzero on tablets.
         int tabStripHeight = getResources().getDimensionPixelSize(R.dimen.tab_strip_height);
 
         // |scrollY - searchBoxTop + tabStripHeight| gives the distance the search bar is from the
         // top of the tab.
         return MathUtils.clamp((scrollY - searchBoxTop + transitionLength + tabStripHeight)
                 / transitionLength, 0f, 1f);
     }
 
     @VisibleForTesting
     public ViewGroup getWrapperView() {
-        return mUseCardsUi ? mRecyclerView : mScrollView;
+        return mRecyclerView;
     }
 
     /**
-     * Sets up scrolling when snippets are enabled. It adds scroll listeners and touch listeners to
-     * the RecyclerView.
+     * Adds listeners to scrolling to take care of snap scrolling and updating the search box on
+     * scroll.
      */
-    private void initializeSearchBoxRecyclerViewScrollHandling() {
+    private void setupScrollHandling() {
         final Runnable mSnapScrollRunnable = new Runnable() {
             @Override
             public void run() {
                 assert mPendingSnapScroll;
                 mPendingSnapScroll = false;
 
                 mRecyclerView.snapScroll(mSearchBoxView, getVerticalScroll(), getHeight());
             }
         };
 
@@ skipping 21 matching lines @@
                     mRecyclerView.postDelayed(mSnapScrollRunnable, SNAP_SCROLL_DELAY_MS);
                 } else {
                     mPendingSnapScroll = false;
                 }
                 return false;
             }
         });
     }
 
     /**
-     * Sets up scrolling when snippets are disabled. It adds scroll and touch listeners to the
-     * scroll view.
-     */
-    private void initializeSearchBoxScrollHandling() {
-        final Runnable mSnapScrollRunnable = new Runnable() {
-            @Override
-            public void run() {
-                if (!mPendingSnapScroll) return;
-                int scrollY = mScrollView.getScrollY();
-                int dividerTop = mMostVisitedLayout.getTop() - mNewTabPageLayout.getPaddingTop();
-                if (scrollY > 0 && scrollY < dividerTop) {
-                    mScrollView.smoothScrollTo(0, scrollY < (dividerTop / 2) ? 0 : dividerTop);
-                }
-                mPendingSnapScroll = false;
-            }
-        };
-        mScrollView.setOnScrollListener(new NewTabPageScrollView.OnScrollListener() {
-            @Override
-            public void onScrollChanged(int l, int t, int oldl, int oldt) {
-                if (mPendingSnapScroll) {
-                    mScrollView.removeCallbacks(mSnapScrollRunnable);
-                    mScrollView.postDelayed(mSnapScrollRunnable, SNAP_SCROLL_DELAY_MS);
-                }
-                updateSearchBoxOnScroll();
-            }
-        });
-        mScrollView.setOnTouchListener(new OnTouchListener() {
-            @Override
-            @SuppressLint("ClickableViewAccessibility")
-            public boolean onTouch(View v, MotionEvent event) {
-                mScrollView.removeCallbacks(mSnapScrollRunnable);
-
-                if (event.getActionMasked() == MotionEvent.ACTION_CANCEL
-                        || event.getActionMasked() == MotionEvent.ACTION_UP) {
-                    mPendingSnapScroll = true;
-                    mScrollView.postDelayed(mSnapScrollRunnable, SNAP_SCROLL_DELAY_MS);
-                } else {
-                    mPendingSnapScroll = false;
-                }
-                return false;
-            }
-        });
-    }
-
-    /**
      * Decrements the count of pending load tasks and notifies the manager when the page load
      * is complete.
      */
     private void loadTaskCompleted() {
         assert mPendingLoadTasks > 0;
         mPendingLoadTasks--;
         if (mPendingLoadTasks == 0) {
             if (mLoadHasCompleted) {
                 assert false;
             } else {
@@ skipping 96 matching lines @@
      * @return The percentage that the URL bar is focused during an animation.
      */
     @VisibleForTesting
     float getUrlFocusChangeAnimationPercent() {
         return mUrlFocusChangePercent;
     }
 
     private void onUrlFocusAnimationChanged() {
         if (mDisableUrlFocusChangeAnimations) return;
 
+        // Translate so that the search box is at the top, but only upwards.
         float percent = mSearchProviderHasLogo ? mUrlFocusChangePercent : 0;
+        int basePosition = getVerticalScroll() + mNewTabPageLayout.getPaddingTop();
+        int target = Math.max(basePosition,
+                    mSearchBoxView.getBottom() - mSearchBoxView.getPaddingBottom());
 
-        int basePosition = getVerticalScroll() + mNewTabPageLayout.getPaddingTop();
-        int target;
-        if (mUseCardsUi) {
-            // Cards UI: translate so that the search box is at the top, but only upwards.
-            target = Math.max(basePosition,
-                    mSearchBoxView.getBottom() - mSearchBoxView.getPaddingBottom());
-        } else {
-            // Otherwise: translate so that Most Visited is right below the omnibox.
-            target = mMostVisitedLayout.getTop();
-        }
         mNewTabPageLayout.setTranslationY(percent * (basePosition - target));
     }
 
     /**
      * Updates the opacity of the search box when scrolling.
      *
      * @param alpha opacity (alpha) value to use.
      */
     public void setSearchBoxAlpha(float alpha) {
         mSearchBoxView.setAlpha(alpha);
@@ skipping 130 matching lines @@
         int newHeight = bottom - top;
 
         if (oldHeight == newHeight && !mTileCountChanged) return;
         mTileCountChanged = false;
 
         // Re-apply the url focus change amount after a rotation to ensure the views are correctly
         // placed with their new layout configurations.
         onUrlFocusAnimationChanged();
         updateSearchBoxOnScroll();
 
-        if (mUseCardsUi) {
-            mRecyclerView.updatePeekingCardAndHeader();
-            // The positioning of elements may have been changed (since the elements expand to fill
-            // the available vertical space), so adjust the scroll.
-            mRecyclerView.snapScroll(mSearchBoxView, getVerticalScroll(), getHeight());
-        }
+        mRecyclerView.updatePeekingCardAndHeader();
+        // The positioning of elements may have been changed (since the elements expand to fill
+        // the available vertical space), so adjust the scroll.
+        mRecyclerView.snapScroll(mSearchBoxView, getVerticalScroll(), getHeight());
     }
 
     // MostVisitedURLsObserver implementation
 
     @Override
     public void onMostVisitedURLsAvailable(final String[] titles, final String[] urls,
             final String[] whitelistIconPaths, final int[] sources) {
         Set<String> urlSet = new HashSet<>(Arrays.asList(urls));
 
         // If no Most Visited items have been built yet, this is the initial load. Build the Most
@@ skipping 241 matching lines @@
         }
     }
 
     @Override
     protected void onMeasure(int widthMeasureSpec, int heightMeasureSpec) {
         if (mNewTabPageLayout != null) {
             mNewTabPageLayout.setParentViewportHeight(MeasureSpec.getSize(heightMeasureSpec));
         }
         super.onMeasure(widthMeasureSpec, heightMeasureSpec);
 
-        if (mUseCardsUi) mRecyclerView.updatePeekingCardAndHeader();
+        mRecyclerView.updatePeekingCardAndHeader();
     }
 
     @Override
     protected void onConfigurationChanged(Configuration newConfig) {
         super.onConfigurationChanged(newConfig);
 
         // When the viewport configuration changes, we want to update the display style so that the
         // observers are aware of the new available space. Another moment to do this update could
         // be through a OnLayoutChangeListener, but then we get notified of the change after the
         // layout pass, which means that the new style will only be visible after layout happens
         // again. We prefer updating here to avoid having to require that additional layout pass.
         mUiConfig.updateDisplayStyle();
 
         // Close the Context Menu as it may have moved (https://crbug.com/642688).
         mContextMenuManager.closeContextMenu();
     }
 
     private int getVerticalScroll() {
-        if (mUseCardsUi) {
-            return mRecyclerView.computeVerticalScrollOffset();
-        } else {
-            return mScrollView.getScrollY();
-        }
+        return mRecyclerView.computeVerticalScrollOffset();
     }
 
     /**
      * @return The adapter position the user has scrolled to.
      */
     public int getScrollPosition() {
-        if (mUseCardsUi) return mRecyclerView.getScrollPosition();
-        return RecyclerView.NO_POSITION;
+        return mRecyclerView.getScrollPosition();
     }
 
     /** @return the context menu manager. */
     public ContextMenuManager getContextMenuManager() {
         return mContextMenuManager;
     }
 }