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

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

Issue 1694863003: Refactor the offline page storage to include client namespace and id. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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 53951d59b73b55631135f0389d999eb62b2d1b4f..d63a732b566714cd5806906a94a6bbb70813008e 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
@@ -229,7 +229,7 @@ public final class OfflinePageBridge {
* exist.
*/
public OfflinePageItem getPageByBookmarkId(BookmarkId bookmarkId) {
- return nativeGetPageByBookmarkId(mNativeOfflinePageBridge, bookmarkId.getId());
+ return nativeGetPageByOfflineId(mNativeOfflinePageBridge, bookmarkId.getId());
}
/**
@@ -276,7 +276,8 @@ public final class OfflinePageBridge {
};
recordFreeSpaceHistograms(
"OfflinePages.SavePage.FreeSpacePercentage", "OfflinePages.SavePage.FreeSpaceMB");
- nativeSavePage(mNativeOfflinePageBridge, callbackWrapper, webContents, bookmarkId.getId());
+ nativeSavePage(mNativeOfflinePageBridge, callbackWrapper, webContents, bookmarkId.getId(),
+ "bookmark", Long.toString(bookmarkId.getId()));
}
/**
@@ -407,8 +408,8 @@ public final class OfflinePageBridge {
}
@CalledByNative
- private void offlinePageDeleted(long bookmarkId) {
- BookmarkId id = new BookmarkId(bookmarkId, BookmarkType.NORMAL);
+ private void offlinePageDeleted(long offlineId) {
+ BookmarkId id = new BookmarkId(offlineId, BookmarkType.NORMAL);
for (OfflinePageModelObserver observer : mObservers) {
observer.offlinePageDeleted(id);
}
@@ -416,19 +417,19 @@ public final class OfflinePageBridge {
@CalledByNative
private static void createOfflinePageAndAddToList(List<OfflinePageItem> offlinePagesList,
- String url, long bookmarkId, String offlineUrl, long fileSize, long creationTime,
+ String url, long offlineId, String offlineUrl, long fileSize, long creationTime,
int accessCount, long lastAccessTimeMs) {
offlinePagesList.add(createOfflinePageItem(
- url, bookmarkId, offlineUrl, fileSize, creationTime, accessCount,
+ url, offlineId, offlineUrl, fileSize, creationTime, accessCount,
lastAccessTimeMs));
}
@CalledByNative
- private static OfflinePageItem createOfflinePageItem(String url, long bookmarkId,
+ private static OfflinePageItem createOfflinePageItem(String url, long offlineId,
String offlineUrl, long fileSize, long creationTime, int accessCount,
long lastAccessTimeMs) {
return new OfflinePageItem(
- url, bookmarkId, offlineUrl, fileSize, creationTime, accessCount, lastAccessTimeMs);
+ url, offlineId, offlineUrl, fileSize, creationTime, accessCount, lastAccessTimeMs);
}
private static native int nativeGetFeatureMode();
@@ -438,17 +439,17 @@ public final class OfflinePageBridge {
private native void nativeDestroy(long nativeOfflinePageBridge);
private native void nativeGetAllPages(
long nativeOfflinePageBridge, List<OfflinePageItem> offlinePages);
- private native OfflinePageItem nativeGetPageByBookmarkId(
- long nativeOfflinePageBridge, long bookmarkId);
+ private native OfflinePageItem nativeGetPageByOfflineId(
+ long nativeOfflinePageBridge, long offlineId);
private native OfflinePageItem nativeGetPageByOnlineURL(
long nativeOfflinePageBridge, String onlineURL);
private native void nativeSavePage(long nativeOfflinePageBridge, SavePageCallback callback,
- WebContents webContents, long bookmarkId);
- private native void nativeMarkPageAccessed(long nativeOfflinePageBridge, long bookmarkId);
+ WebContents webContents, long offlineId, String clientNamespace, String clientId);
+ private native void nativeMarkPageAccessed(long nativeOfflinePageBridge, long offlineId);
private native void nativeDeletePage(long nativeOfflinePageBridge,
- DeletePageCallback callback, long bookmarkId);
+ DeletePageCallback callback, long offlineId);
private native void nativeDeletePages(
- long nativeOfflinePageBridge, DeletePageCallback callback, long[] bookmarkIds);
+ long nativeOfflinePageBridge, DeletePageCallback callback, long[] offlineIds);
private native void nativeGetPagesToCleanUp(
long nativeOfflinePageBridge, List<OfflinePageItem> offlinePages);
private native void nativeCheckMetadataConsistency(long nativeOfflinePageBridge);

Powered by Google App Engine
This is Rietveld 408576698