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

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

Issue 2363483002: Revert of Ntp: restore scroll position. (Closed)
Patch Set: Created 4 years, 3 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/NewTabPageView.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
index 68daf6f136821ff21c9b9d5416a7e2f4bb895987..fddc26ea35d472a32a9440fef3e929389722851d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
@@ -296,7 +296,7 @@
* @param snippetsBridge The optional bridge, that can be used to interact with the snippets.
*/
public void initialize(NewTabPageManager manager, boolean searchProviderHasLogo,
- SnippetsBridge snippetsBridge, int scrollPosition) {
+ SnippetsBridge snippetsBridge) {
mManager = manager;
mUiConfig = new UiConfig(this);
ViewStub stub = (ViewStub) findViewById(R.id.new_tab_page_layout_stub);
@@ -352,7 +352,6 @@
mNewTabPageAdapter =
new NewTabPageAdapter(mManager, mNewTabPageLayout, snippetsBridge, mUiConfig);
mRecyclerView.setAdapter(mNewTabPageAdapter);
- mRecyclerView.scrollToPosition(scrollPosition);
// Set up swipe-to-dismiss
ItemTouchHelper helper =
@@ -1179,11 +1178,4 @@
return mScrollView.getScrollY();
}
}
-
- /**
- * @return The adapter position the user has scrolled to.
- */
- public int getScrollPosition() {
- return mRecyclerView.getScrollPosition();
- }
}

Powered by Google App Engine
This is Rietveld 408576698