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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/framehost/NavigationControllerImpl.java

Issue 369503003: Migration of code from ContentViewCore to NavigationController to avoid ContentViewCore abstraction… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 6 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: content/public/android/java/src/org/chromium/content/browser/framehost/NavigationControllerImpl.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/framehost/NavigationControllerImpl.java b/content/public/android/java/src/org/chromium/content/browser/framehost/NavigationControllerImpl.java
index 30cc7f8da5b05d798e9caa1b2258af637dfef127..cde8e2890ebbee7e660233bc7205177835d8cef1 100644
--- a/content/public/android/java/src/org/chromium/content/browser/framehost/NavigationControllerImpl.java
+++ b/content/public/android/java/src/org/chromium/content/browser/framehost/NavigationControllerImpl.java
@@ -79,8 +79,64 @@ import org.chromium.content_public.browser.NavigationController;
}
}
+ /**
+ * Loads the current navigation if there is a pending lazy load (after tab restore).
+ */
Yaron 2014/07/02 17:32:39 Please remove function comments and replace them w
AKVT 2014/07/05 10:32:17 Done.
+ public void loadIfNecessary() {
+ if (mNativeNavigationControllerAndroid != 0) {
+ nativeLoadIfNecessary(mNativeNavigationControllerAndroid);
+ }
+ }
+
+ /**
+ * Requests the current navigation to be loaded upon the next call to loadIfNecessary().
+ */
+ public void requestRestoreLoad() {
+ if (mNativeNavigationControllerAndroid != 0) {
+ nativeRequestRestoreLoad(mNativeNavigationControllerAndroid);
+ }
+ }
+
+ /**
+ * Reload the current page.
+ */
+ public void reload(boolean checkForRepost) {
+ if (mNativeNavigationControllerAndroid != 0) {
+ nativeReload(mNativeNavigationControllerAndroid, checkForRepost);
+ }
+ }
+
+ /**
+ * Reload the current page, ignoring the contents of the cache.
+ */
+ public void reloadIgnoringCache(boolean checkForRepost) {
+ if (mNativeNavigationControllerAndroid != 0) {
+ nativeReloadIgnoringCache(mNativeNavigationControllerAndroid, checkForRepost);
+ }
+ }
+
+ /**
+ * Cancel the pending reload.
+ */
+ public void cancelPendingReload() {
+ if (mNativeNavigationControllerAndroid != 0) {
+ nativeCancelPendingReload(mNativeNavigationControllerAndroid);
+ }
+ }
+
+ /**
+ * Continue the pending reload.
+ */
+ public void continuePendingReload() {
+ if (mNativeNavigationControllerAndroid != 0) {
+ nativeContinuePendingReload(mNativeNavigationControllerAndroid);
+ }
+ }
+
private native boolean nativeCanGoBack(long nativeNavigationControllerAndroid);
private native boolean nativeCanGoForward(long nativeNavigationControllerAndroid);
+ private native void nativeLoadIfNecessary(long nativeNavigationControllerAndroid);
+ private native void nativeRequestRestoreLoad(long nativeNavigationControllerAndroid);
private native boolean nativeCanGoToOffset(
long nativeNavigationControllerAndroid, int offset);
private native void nativeGoBack(long nativeNavigationControllerAndroid);
@@ -88,4 +144,10 @@ import org.chromium.content_public.browser.NavigationController;
private native void nativeGoToOffset(long nativeNavigationControllerAndroid, int offset);
private native void nativeGoToNavigationIndex(
long nativeNavigationControllerAndroid, int index);
+ private native void nativeCancelPendingReload(long nativeNavigationControllerAndroid);
+ private native void nativeContinuePendingReload(long nativeNavigationControllerAndroid);
+ private native void nativeReload(long nativeNavigationControllerAndroid,
+ boolean checkForRepost);
+ private native void nativeReloadIgnoringCache(long nativeNavigationControllerAndroid,
+ boolean checkForRepost);
}

Powered by Google App Engine
This is Rietveld 408576698