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

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

Issue 2284933002: Remove OfflineURL from offline page (Closed)
Patch Set: Add comment per feedback Created 4 years, 2 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/OfflinePageItem.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageItem.java b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageItem.java
index 4dd6e139915db271b014b55bc5b305369c65075d..75af288d137691e57a600507884d5adf27deffb2 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageItem.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageItem.java
@@ -13,7 +13,6 @@ public class OfflinePageItem {
private final String mUrl;
private final long mOfflineId;
private final ClientId mClientId;
- private final String mOfflineUrl;
private final String mFilePath;
private final long mFileSize;
private final long mCreationTimeMs;
@@ -21,12 +20,11 @@ public class OfflinePageItem {
private final long mLastAccessTimeMs;
public OfflinePageItem(String url, long offlineId, String clientNamespace, String clientId,
- String offlineUrl, String filePath, long fileSize, long creationTimeMs, int accessCount,
+ String filePath, long fileSize, long creationTimeMs, int accessCount,
long lastAccessTimeMs) {
mUrl = url;
mOfflineId = offlineId;
mClientId = new ClientId(clientNamespace, clientId);
- mOfflineUrl = offlineUrl;
mFilePath = filePath;
mFileSize = fileSize;
mCreationTimeMs = creationTimeMs;
@@ -52,12 +50,6 @@ public class OfflinePageItem {
return mClientId;
}
- /** @return Path to the offline copy of the page. */
- @VisibleForTesting
- public String getOfflineUrl() {
- return mOfflineUrl;
- }
-
/** @return File Path to the offline copy of the page. */
@VisibleForTesting
public String getFilePath() {

Powered by Google App Engine
This is Rietveld 408576698