Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3945)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetHeaderViewHolder.java

Issue 2245013003: Remove "list item" from Android NTP class names. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: x Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetHeaderViewHolder.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetHeaderViewHolder.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetHeaderViewHolder.java
deleted file mode 100644
index 143755839dbf08e49daaeba8579facab54a09955..0000000000000000000000000000000000000000
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetHeaderViewHolder.java
+++ /dev/null
@@ -1,92 +0,0 @@
-// 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.snippets;
-
-import android.support.v7.widget.RecyclerView;
-import android.view.LayoutInflater;
-import android.widget.TextView;
-
-import org.chromium.chrome.R;
-import org.chromium.chrome.browser.ntp.UiConfig;
-import org.chromium.chrome.browser.ntp.cards.MarginResizer;
-import org.chromium.chrome.browser.ntp.cards.NewTabPageListItem;
-import org.chromium.chrome.browser.ntp.cards.NewTabPageRecyclerView;
-import org.chromium.chrome.browser.ntp.cards.NewTabPageViewHolder;
-import org.chromium.chrome.browser.util.MathUtils;
-
-/**
- * A class that represents the view for a single card snippet.
- */
-public class SnippetHeaderViewHolder extends NewTabPageViewHolder {
- private static final double SCROLL_HEADER_HEIGHT_PERCENTAGE = 0.7;
-
- private final int mMaxSnippetHeaderHeight;
- private final int mMaxPeekPadding;
- private final TextView mHeaderTextView;
- private final NewTabPageRecyclerView mRecyclerView;
-
- private SnippetHeaderListItem mHeaderListItem;
-
- /** Can the header transition. */
- private boolean mCanTransition = false;
-
- public SnippetHeaderViewHolder(final NewTabPageRecyclerView recyclerView, UiConfig config) {
- super(LayoutInflater.from(recyclerView.getContext())
- .inflate(R.layout.new_tab_page_snippets_header, recyclerView, false));
- mMaxSnippetHeaderHeight = itemView.getResources().getDimensionPixelSize(
- R.dimen.snippets_article_header_height);
-
- mMaxPeekPadding = itemView.getResources().getDimensionPixelSize(
- R.dimen.snippets_padding_and_peeking_card_height);
-
- mHeaderTextView = (TextView) itemView.findViewById(R.id.suggestions_section_header);
- mRecyclerView = recyclerView;
- MarginResizer.createWithViewAdapter(itemView, config);
- }
-
- @Override
- public void onBindViewHolder(NewTabPageListItem header) {
- mHeaderListItem = (SnippetHeaderListItem) header;
- updateDisplay();
- }
-
- /**
- * Set whether the header is in a state where the height can vary from show to hidden.
- */
- public void setCanTransition(boolean canTransition) {
- mCanTransition = canTransition;
- }
-
- /**
- * @return The header height we want to set.
- */
- private int getHeaderHeight() {
- if (!mHeaderListItem.isVisible()) return 0;
-
- // If the header cannot transition but is visible - set the height to the maximum so
- // it always displays
- if (!mCanTransition) return mMaxSnippetHeaderHeight;
-
- // Check if snippet header top is within range to start showing. Set the header height,
- // this is a percentage of how much is scrolled. The balance of the scroll will be used
- // to display the peeking card.
- int amountScrolled = (mRecyclerView.getHeight() - mMaxPeekPadding) - itemView.getTop();
- return MathUtils.clamp((int) (amountScrolled * SCROLL_HEADER_HEIGHT_PERCENTAGE),
- 0, mMaxSnippetHeaderHeight);
- }
-
- /**
- * Update the view for the fade in/out and heading height.
- */
- public void updateDisplay() {
- mHeaderTextView.setText(mHeaderListItem.getHeaderText());
- RecyclerView.LayoutParams params = (RecyclerView.LayoutParams) itemView.getLayoutParams();
- int headerHeight = getHeaderHeight();
-
- itemView.setAlpha((float) headerHeight / mMaxSnippetHeaderHeight);
- params.height = headerHeight;
- itemView.requestLayout();
- }
-}

Powered by Google App Engine
This is Rietveld 408576698