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

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

Issue 1894703002: [Offline pages] Removing offline pages from Bookmarks UI (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Marking more methods as visible for testing Created 4 years, 8 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/OfflinePageFreeUpSpaceDialog.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageFreeUpSpaceDialog.java b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageFreeUpSpaceDialog.java
deleted file mode 100644
index a611a76da37d3e127d6982fb93581c728af38c2a..0000000000000000000000000000000000000000
--- a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageFreeUpSpaceDialog.java
+++ /dev/null
@@ -1,117 +0,0 @@
-// Copyright 2015 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-package org.chromium.chrome.browser.offlinepages;
-
-import android.app.Dialog;
-import android.app.DialogFragment;
-import android.content.Context;
-import android.content.DialogInterface;
-import android.os.Bundle;
-import android.support.v7.app.AlertDialog;
-import android.text.format.Formatter;
-
-import org.chromium.base.metrics.RecordUserAction;
-import org.chromium.chrome.R;
-import org.chromium.chrome.browser.offlinepages.OfflinePageBridge.DeletePageCallback;
-import org.chromium.chrome.browser.snackbar.Snackbar;
-import org.chromium.chrome.browser.snackbar.SnackbarManager.SnackbarController;
-
-import java.util.ArrayList;
-import java.util.List;
-
-/**
- * Shows a "Free up space" dialog to clean up storage for offline pages.
- */
-public class OfflinePageFreeUpSpaceDialog
- extends DialogFragment implements DialogInterface.OnClickListener {
- private OfflinePageBridge mOfflinePageBridge;
- private List<OfflinePageItem> mOfflinePagesToDelete;
- private OfflinePageFreeUpSpaceCallback mCallback;
-
- /**
- * Creates the dialog. If the passed bridge instance needs to be destroyed after the dialog
- * is finished, it should be taken care of in the callback implementation.
- *
- * @param offlinePageBridge An object to access offline page functionality.
- * @param callback An object that will be called when the dialog finishes. Can be null.
- * @see OfflinePageFreeUpSpaceCallback
- */
- public static OfflinePageFreeUpSpaceDialog newInstance(
- OfflinePageBridge offlinePageBridge, OfflinePageFreeUpSpaceCallback callback) {
- assert offlinePageBridge != null;
- OfflinePageFreeUpSpaceDialog dialog = new OfflinePageFreeUpSpaceDialog();
- dialog.mOfflinePageBridge = offlinePageBridge;
- dialog.mCallback = callback;
- return dialog;
- }
-
- /**
- * Creates a snackbar informing user that the storage has been cleared.
- */
- public static Snackbar createStorageClearedSnackbar(Context context) {
- return Snackbar.make(context.getString(OfflinePageUtils.getStringId(
- R.string.offline_pages_storage_cleared)),
- new SnackbarController() {
- @Override
- public void onDismissNoAction(Object actionData) {}
- @Override
- public void onAction(Object actionData) {}
- }, Snackbar.TYPE_ACTION);
- }
-
- @Override
- public Dialog onCreateDialog(Bundle savedInstanceState) {
- if (savedInstanceState != null) dismiss();
-
- mOfflinePagesToDelete = mOfflinePageBridge.getPagesToCleanUp();
- AlertDialog.Builder builder =
- new AlertDialog.Builder(getActivity(), R.style.AlertDialogTheme)
- .setTitle(OfflinePageUtils.getStringId(
- R.string.offline_pages_free_up_space_title))
- .setPositiveButton(R.string.delete, this)
- .setNegativeButton(R.string.cancel, this)
- .setMessage(getString(OfflinePageUtils.getStringId(
- R.string.offline_pages_free_up_space_message),
- mOfflinePagesToDelete.size(),
- Formatter.formatFileSize(getActivity(), getTotalSize())));
- return builder.create();
- }
-
- @Override
- public void onClick(DialogInterface dialog, int id) {
- if (id == AlertDialog.BUTTON_NEGATIVE) {
- RecordUserAction.record("OfflinePages.FreeUpSpaceDialogButtonNotClicked");
- dialog.cancel();
- if (mCallback != null) mCallback.onFreeUpSpaceCancelled();
- return;
- }
-
- mOfflinePageBridge.deletePages(getOfflineIdsToDelete(), new DeletePageCallback() {
- @Override
- public void onDeletePageDone(int deletePageResult) {
- RecordUserAction.record("OfflinePages.FreeUpSpaceDialogButtonClicked");
- if (mCallback != null) mCallback.onFreeUpSpaceDone();
- }
- });
- }
-
- /** Returns a list of IDs for which the offline pages will be deleted. */
- private List<Long> getOfflineIdsToDelete() {
- List<Long> offlineIds = new ArrayList<>();
- for (OfflinePageItem offlinePage : mOfflinePagesToDelete) {
- offlineIds.add(offlinePage.getOfflineId());
- }
- return offlineIds;
- }
-
- /** Returns a total size of offline pages that will be deleted. */
- private long getTotalSize() {
- long totalSize = 0;
- for (OfflinePageItem offlinePage : mOfflinePagesToDelete) {
- totalSize += offlinePage.getFileSize();
- }
- return totalSize;
- }
-}

Powered by Google App Engine
This is Rietveld 408576698