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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java

Issue 2284933002: Remove OfflineURL from offline page (Closed)
Patch Set: Fix trybot Created 4 years, 4 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/offlinepages/OfflinePageBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java
index 9e283ad498a80e2bd67aa55ef99af7ca420842ab..79b7df94cfd46c8686e054fb5278f60f21e1a485 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java
@@ -334,17 +334,6 @@ public class OfflinePageBridge {
}
/**
- * Get the offline page associated with the provided offline URL.
- *
- * @param offlineUrl URL pointing to the offline copy of the web page.
- * @param callback callback to pass back the
- * matching {@link OfflinePageItem} if found. Will pass back <code>null</code> if not.
- */
- public void getPageByOfflineUrl(String offlineUrl, Callback<OfflinePageItem> callback) {
- nativeGetPageByOfflineUrl(mNativeOfflinePageBridge, offlineUrl, callback);
- }
-
- /**
* Get the offline page associated with the provided offline ID.
*
* @param offlineId ID of the offline page.
@@ -571,18 +560,17 @@ public class OfflinePageBridge {
@CalledByNative
private static void createOfflinePageAndAddToList(List<OfflinePageItem> offlinePagesList,
- String url, long offlineId, String clientNamespace, String clientId, String offlineUrl,
- String filePath, long fileSize, long creationTime, int accessCount,
- long lastAccessTimeMs) {
+ String url, long offlineId, String clientNamespace, String clientId, String filePath,
+ long fileSize, long creationTime, int accessCount, long lastAccessTimeMs) {
offlinePagesList.add(createOfflinePageItem(url, offlineId, clientNamespace, clientId,
- offlineUrl, filePath, fileSize, creationTime, accessCount, lastAccessTimeMs));
+ filePath, fileSize, creationTime, accessCount, lastAccessTimeMs));
}
@CalledByNative
private static OfflinePageItem createOfflinePageItem(String url, long offlineId,
- String clientNamespace, String clientId, String offlineUrl, String filePath,
- long fileSize, long creationTime, int accessCount, long lastAccessTimeMs) {
- return new OfflinePageItem(url, offlineId, clientNamespace, clientId, offlineUrl, filePath,
+ String clientNamespace, String clientId, String filePath, long fileSize,
+ long creationTime, int accessCount, long lastAccessTimeMs) {
+ return new OfflinePageItem(url, offlineId, clientNamespace, clientId, filePath,
fileSize, creationTime, accessCount, lastAccessTimeMs);
}
@@ -623,8 +611,6 @@ public class OfflinePageBridge {
private native void nativeSelectPageForOnlineUrl(
long nativeOfflinePageBridge, String onlineUrl, int tabId,
Callback<OfflinePageItem> callback);
- private native void nativeGetPageByOfflineUrl(
- long nativeOfflinePageBridge, String offlineUrl, Callback<OfflinePageItem> callback);
private native void nativeSavePage(long nativeOfflinePageBridge, SavePageCallback callback,
WebContents webContents, String clientNamespace, String clientId);
private native void nativeSavePageLater(long nativeOfflinePageBridge, String url,

Powered by Google App Engine
This is Rietveld 408576698