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

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

Issue 1263573004: Move NTP related classes to ntp package. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: newt's nits Created 5 years, 5 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/NewTabPagePrefs.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/NewTabPagePrefs.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPagePrefs.java
similarity index 86%
rename from chrome/android/java/src/org/chromium/chrome/browser/NewTabPagePrefs.java
rename to chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPagePrefs.java
index 21e80f8d0021bd90086f86cc4312942d01864d45..7c6886a811b3c8d97cd0eeb8c7f760fb8fca77ef 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/NewTabPagePrefs.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPagePrefs.java
@@ -2,29 +2,29 @@
// 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;
+package org.chromium.chrome.browser.ntp;
-import org.chromium.chrome.browser.ForeignSessionHelper.ForeignSession;
+import org.chromium.chrome.browser.ntp.ForeignSessionHelper.ForeignSession;
import org.chromium.chrome.browser.profiles.Profile;
/**
* This class allows Java code to read and modify preferences related to the NTP
*/
-public class NewTabPagePrefs {
+class NewTabPagePrefs {
private long mNativeNewTabPagePrefs;
/**
* Initialize this class with the given profile.
* @param profile Profile that will be used for syncing.
*/
- public NewTabPagePrefs(Profile profile) {
+ NewTabPagePrefs(Profile profile) {
mNativeNewTabPagePrefs = nativeInit(profile);
}
/**
* Clean up the C++ side of this class. After the call, this class instance shouldn't be used.
*/
- public void destroy() {
+ void destroy() {
assert mNativeNewTabPagePrefs != 0;
nativeDestroy(mNativeNewTabPagePrefs);
mNativeNewTabPagePrefs = 0;
@@ -35,7 +35,7 @@ public class NewTabPagePrefs {
* page.
* @param isCollapsed Whether we want the currently open tabs list to be collapsed.
*/
- public void setCurrentlyOpenTabsCollapsed(boolean isCollapsed) {
+ void setCurrentlyOpenTabsCollapsed(boolean isCollapsed) {
nativeSetCurrentlyOpenTabsCollapsed(mNativeNewTabPagePrefs, isCollapsed);
}
@@ -44,7 +44,7 @@ public class NewTabPagePrefs {
* @return Whether the list of currently open tabs is collapsed (vs expanded) on
* the Recent Tabs page.
*/
- public boolean getCurrentlyOpenTabsCollapsed() {
+ boolean getCurrentlyOpenTabsCollapsed() {
return nativeGetCurrentlyOpenTabsCollapsed(mNativeNewTabPagePrefs);
}
@@ -53,7 +53,7 @@ public class NewTabPagePrefs {
* page.
* @param isCollapsed Whether we want the snapshot documents list to be collapsed.
*/
- public void setSnapshotDocumentCollapsed(boolean isCollapsed) {
+ void setSnapshotDocumentCollapsed(boolean isCollapsed) {
nativeSetSnapshotDocumentCollapsed(mNativeNewTabPagePrefs, isCollapsed);
}
@@ -63,7 +63,7 @@ public class NewTabPagePrefs {
* @return Whether the list of snapshot documents is collapsed (vs expanded) on
* the Recent Tabs page.
*/
- public boolean getSnapshotDocumentCollapsed() {
+ boolean getSnapshotDocumentCollapsed() {
return nativeGetSnapshotDocumentCollapsed(mNativeNewTabPagePrefs);
}
@@ -72,7 +72,7 @@ public class NewTabPagePrefs {
* page.
* @param isCollapsed Whether we want the recently closed tabs list to be collapsed.
*/
- public void setRecentlyClosedTabsCollapsed(boolean isCollapsed) {
+ void setRecentlyClosedTabsCollapsed(boolean isCollapsed) {
nativeSetRecentlyClosedTabsCollapsed(mNativeNewTabPagePrefs, isCollapsed);
}
@@ -82,7 +82,7 @@ public class NewTabPagePrefs {
* @return Whether the list of recently closed tabs is collapsed (vs expanded) on
* the Recent Tabs page.
*/
- public boolean getRecentlyClosedTabsCollapsed() {
+ boolean getRecentlyClosedTabsCollapsed() {
return nativeGetRecentlyClosedTabsCollapsed(mNativeNewTabPagePrefs);
}
@@ -90,7 +90,7 @@ public class NewTabPagePrefs {
* Sets whether sync promo is collapsed (vs expanded) on the Recent Tabs page.
* @param isCollapsed Whether we want the sync promo to be collapsed.
*/
- public void setSyncPromoCollapsed(boolean isCollapsed) {
+ void setSyncPromoCollapsed(boolean isCollapsed) {
nativeSetSyncPromoCollapsed(mNativeNewTabPagePrefs, isCollapsed);
}
@@ -98,7 +98,7 @@ public class NewTabPagePrefs {
* Gets whether sync promo is collapsed (vs expanded) on the Recent Tabs page.
* @return Whether the sync promo is collapsed (vs expanded) on the Recent Tabs page.
*/
- public boolean getSyncPromoCollapsed() {
+ boolean getSyncPromoCollapsed() {
return nativeGetSyncPromoCollapsed(mNativeNewTabPagePrefs);
}
@@ -107,7 +107,7 @@ public class NewTabPagePrefs {
* @param session Session to set collapsed or expanded.
* @param isCollapsed Whether we want the foreign session to be collapsed.
*/
- public void setForeignSessionCollapsed(ForeignSession session, boolean isCollapsed) {
+ void setForeignSessionCollapsed(ForeignSession session, boolean isCollapsed) {
nativeSetForeignSessionCollapsed(mNativeNewTabPagePrefs, session.tag, isCollapsed);
}
@@ -116,7 +116,7 @@ public class NewTabPagePrefs {
* @param session Session to fetch collapsed state.
* @return Whether the given foreign session is collapsed (vs expanded) on the Recent Tabs page.
*/
- public boolean getForeignSessionCollapsed(ForeignSession session) {
+ boolean getForeignSessionCollapsed(ForeignSession session) {
return nativeGetForeignSessionCollapsed(mNativeNewTabPagePrefs, session.tag);
}

Powered by Google App Engine
This is Rietveld 408576698