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

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

Issue 2365313002: Reland: Ntp: restore scroll position. (Closed)
Patch Set: Avoid storing NO_POSITION 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 fddc26ea35d472a32a9440fef3e929389722851d..68daf6f136821ff21c9b9d5416a7e2f4bb895987 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 @@ public class NewTabPageView extends FrameLayout
* @param snippetsBridge The optional bridge, that can be used to interact with the snippets.
*/
public void initialize(NewTabPageManager manager, boolean searchProviderHasLogo,
- SnippetsBridge snippetsBridge) {
+ SnippetsBridge snippetsBridge, int scrollPosition) {
mManager = manager;
mUiConfig = new UiConfig(this);
ViewStub stub = (ViewStub) findViewById(R.id.new_tab_page_layout_stub);
@@ -352,6 +352,7 @@ public class NewTabPageView extends FrameLayout
mNewTabPageAdapter =
new NewTabPageAdapter(mManager, mNewTabPageLayout, snippetsBridge, mUiConfig);
mRecyclerView.setAdapter(mNewTabPageAdapter);
+ mRecyclerView.scrollToPosition(scrollPosition);
// Set up swipe-to-dismiss
ItemTouchHelper helper =
@@ -1178,4 +1179,11 @@ public class NewTabPageView extends FrameLayout
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