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

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

Issue 1367063004: Support undoing offline page deletion (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: One more change Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser.offlinepages; 5 package org.chromium.chrome.browser.offlinepages;
6 6
7 import org.chromium.base.ObserverList; 7 import org.chromium.base.ObserverList;
8 import org.chromium.base.ThreadUtils; 8 import org.chromium.base.ThreadUtils;
9 import org.chromium.base.VisibleForTesting; 9 import org.chromium.base.VisibleForTesting;
10 import org.chromium.base.annotations.CalledByNative; 10 import org.chromium.base.annotations.CalledByNative;
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 * 57 *
58 * @param deletePageResult Result of deleting the page. Uses 58 * @param deletePageResult Result of deleting the page. Uses
59 * {@see org.chromium.components.offlinepages.DeletePageResult} enum . 59 * {@see org.chromium.components.offlinepages.DeletePageResult} enum .
60 * @see OfflinePageBridge#deletePage() 60 * @see OfflinePageBridge#deletePage()
61 */ 61 */
62 @CalledByNative("DeletePageCallback") 62 @CalledByNative("DeletePageCallback")
63 void onDeletePageDone(int deletePageResult); 63 void onDeletePageDone(int deletePageResult);
64 } 64 }
65 65
66 /** 66 /**
67 * Interface that provides listeners to be notified of changes to the offlin e page model. 67 * Base empty implementation observer class that provides listeners to be no tified of changes to
68 * the offline page model.
68 */ 69 */
69 public interface OfflinePageModelObserver { 70 public abstract static class OfflinePageModelObserver {
70 /** 71 /**
71 * Called when the native side of offline pages is loaded and now in usa ble state. 72 * Called when the native side of offline pages is loaded and now in usa ble state.
72 */ 73 */
73 void offlinePageModelLoaded(); 74 public void offlinePageModelLoaded() {}
75
76 /**
77 * Called when the native side of offline pages is changed due to adding , removing or
78 * update an offline page.
79 */
80 public void offlinePageModelChanged() {}
74 } 81 }
75 82
76 private static int getFreeSpacePercentage() { 83 private static int getFreeSpacePercentage() {
77 return (int) (1.0 * OfflinePageUtils.getFreeSpaceInBytes() 84 return (int) (1.0 * OfflinePageUtils.getFreeSpaceInBytes()
78 / OfflinePageUtils.getTotalSpaceInBytes() * 100); 85 / OfflinePageUtils.getTotalSpaceInBytes() * 100);
79 } 86 }
80 87
81 private static int getFreeSpaceMB() { 88 private static int getFreeSpaceMB() {
82 return (int) (OfflinePageUtils.getFreeSpaceInBytes() / (1024 * 1024)); 89 return (int) (OfflinePageUtils.getFreeSpaceInBytes() / (1024 * 1024));
83 } 90 }
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
237 244
238 @CalledByNative 245 @CalledByNative
239 private void offlinePageModelLoaded() { 246 private void offlinePageModelLoaded() {
240 mIsNativeOfflinePageModelLoaded = true; 247 mIsNativeOfflinePageModelLoaded = true;
241 for (OfflinePageModelObserver observer : mObservers) { 248 for (OfflinePageModelObserver observer : mObservers) {
242 observer.offlinePageModelLoaded(); 249 observer.offlinePageModelLoaded();
243 } 250 }
244 } 251 }
245 252
246 @CalledByNative 253 @CalledByNative
254 private void offlinePageModelChanged() {
255 for (OfflinePageModelObserver observer : mObservers) {
256 observer.offlinePageModelChanged();
257 }
258 }
259
260 @CalledByNative
247 private static void createOfflinePageAndAddToList(List<OfflinePageItem> offl inePagesList, 261 private static void createOfflinePageAndAddToList(List<OfflinePageItem> offl inePagesList,
248 String url, long bookmarkId, String offlineUrl, long fileSize, int a ccessCount) { 262 String url, long bookmarkId, String offlineUrl, long fileSize, int a ccessCount) {
249 offlinePagesList.add(createOfflinePageItem(url, bookmarkId, offlineUrl, fileSize, 263 offlinePagesList.add(createOfflinePageItem(url, bookmarkId, offlineUrl, fileSize,
250 accessCount)); 264 accessCount));
251 } 265 }
252 266
253 @CalledByNative 267 @CalledByNative
254 private static OfflinePageItem createOfflinePageItem( 268 private static OfflinePageItem createOfflinePageItem(
255 String url, long bookmarkId, String offlineUrl, long fileSize, int a ccessCount) { 269 String url, long bookmarkId, String offlineUrl, long fileSize, int a ccessCount) {
256 return new OfflinePageItem(url, bookmarkId, offlineUrl, fileSize, access Count); 270 return new OfflinePageItem(url, bookmarkId, offlineUrl, fileSize, access Count);
(...skipping 10 matching lines...) Expand all
267 private native void nativeSavePage(long nativeOfflinePageBridge, SavePageCal lback callback, 281 private native void nativeSavePage(long nativeOfflinePageBridge, SavePageCal lback callback,
268 WebContents webContents, long bookmarkId); 282 WebContents webContents, long bookmarkId);
269 private native void nativeMarkPageAccessed(long nativeOfflinePageBridge, lon g bookmarkId); 283 private native void nativeMarkPageAccessed(long nativeOfflinePageBridge, lon g bookmarkId);
270 private native void nativeDeletePage(long nativeOfflinePageBridge, 284 private native void nativeDeletePage(long nativeOfflinePageBridge,
271 DeletePageCallback callback, long bookmarkId); 285 DeletePageCallback callback, long bookmarkId);
272 private native void nativeDeletePages( 286 private native void nativeDeletePages(
273 long nativeOfflinePageBridge, DeletePageCallback callback, long[] bo okmarkIds); 287 long nativeOfflinePageBridge, DeletePageCallback callback, long[] bo okmarkIds);
274 private native void nativeGetPagesToCleanUp( 288 private native void nativeGetPagesToCleanUp(
275 long nativeOfflinePageBridge, List<OfflinePageItem> offlinePages); 289 long nativeOfflinePageBridge, List<OfflinePageItem> offlinePages);
276 } 290 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698