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

Unified Diff: blimp/client/core/android/java/src/org/chromium/blimp/core/BlimpNavigationControllerImpl.java

Issue 2058263002: Tied up BlimpNavigationController to NavigationFeature (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@blimp_core
Patch Set: Addressed code review comments Created 4 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: blimp/client/core/android/java/src/org/chromium/blimp/core/BlimpNavigationControllerImpl.java
diff --git a/blimp/client/core/android/java/src/org/chromium/blimp/core/BlimpNavigationControllerImpl.java b/blimp/client/core/android/java/src/org/chromium/blimp/core/BlimpNavigationControllerImpl.java
index 7dc5264ae79671fdaf29dfc5daf47ea7234a1833..93d293557365cd1c135f3b3cd3e9fd284cae18f8 100644
--- a/blimp/client/core/android/java/src/org/chromium/blimp/core/BlimpNavigationControllerImpl.java
+++ b/blimp/client/core/android/java/src/org/chromium/blimp/core/BlimpNavigationControllerImpl.java
@@ -49,6 +49,41 @@ public class BlimpNavigationControllerImpl implements BlimpNavigationController
return nativeGetURL(mNativeBlimpNavigationControllerImplAndroid);
}
+ @Override
+ public boolean canGoBack() {
+ if (mNativeBlimpNavigationControllerImplAndroid == 0) return false;
+ return nativeCanGoBack(mNativeBlimpNavigationControllerImplAndroid);
+ }
+
+ @Override
+ public boolean canGoForward() {
+ if (mNativeBlimpNavigationControllerImplAndroid == 0) return false;
+ return nativeCanGoForward(mNativeBlimpNavigationControllerImplAndroid);
+ }
+
+ @Override
+ public void goBack() {
+ if (mNativeBlimpNavigationControllerImplAndroid == 0) return;
+ nativeGoBack(mNativeBlimpNavigationControllerImplAndroid);
+ }
+
+ @Override
+ public void goForward() {
+ if (mNativeBlimpNavigationControllerImplAndroid == 0) return;
+ nativeGoForward(mNativeBlimpNavigationControllerImplAndroid);
+ }
+
+ @Override
+ public void reload(boolean checkForRepost) {
+ if (mNativeBlimpNavigationControllerImplAndroid == 0) return;
+ nativeReload(mNativeBlimpNavigationControllerImplAndroid);
+ }
+
+ private native void nativeGoBack(long nativeBlimpNavigationControllerImplAndroid);
+ private native void nativeGoForward(long nativeBlimpNavigationControllerImplAndroid);
+ private native void nativeReload(long nativeBlimpNavigationControllerImplAndroid);
+ private native boolean nativeCanGoBack(long nativeBlimpNavigationControllerImplAndroid);
+ private native boolean nativeCanGoForward(long nativeBlimpNavigationControllerImplAndroid);
private native void nativeLoadURL(long nativeBlimpNavigationControllerImplAndroid, String url);
private native String nativeGetURL(long nativeBlimpNavigationControllerImplAndroid);
}

Powered by Google App Engine
This is Rietveld 408576698