Rename EnhancedBookmarks to Bookmarks, part 1.

chrome/android/java/src/org/chromium/chrome/browser/enhancedbookmarks/EnhancedBookmarkUtils.java

Index: chrome/android/java/src/org/chromium/chrome/browser/enhancedbookmarks/EnhancedBookmarkUtils.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/enhancedbookmarks/EnhancedBookmarkUtils.java b/chrome/android/java/src/org/chromium/chrome/browser/enhancedbookmarks/EnhancedBookmarkUtils.java
deleted file mode 100644
index 25b44421648a81f1cfcdc14640d5e539654bd365..0000000000000000000000000000000000000000
--- a/chrome/android/java/src/org/chromium/chrome/browser/enhancedbookmarks/EnhancedBookmarkUtils.java
+++ /dev/null
@@ -1,467 +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.enhancedbookmarks;
-
-import android.app.Activity;
-import android.content.Context;
-import android.content.Intent;
-import android.content.SharedPreferences;
-import android.net.Uri;
-import android.os.AsyncTask;
-import android.preference.PreferenceManager;
-import android.provider.Browser;
-import android.text.TextUtils;
-
-import org.chromium.base.ApiCompatibilityUtils;
-import org.chromium.base.VisibleForTesting;
-import org.chromium.base.metrics.RecordHistogram;
-import org.chromium.base.metrics.RecordUserAction;
-import org.chromium.chrome.R;
-import org.chromium.chrome.browser.ChromeBrowserProviderClient;
-import org.chromium.chrome.browser.IntentHandler;
-import org.chromium.chrome.browser.UrlConstants;
-import org.chromium.chrome.browser.document.ChromeLauncherActivity;
-import org.chromium.chrome.browser.enhancedbookmarks.EnhancedBookmarksModel.AddBookmarkCallback;
-import org.chromium.chrome.browser.ntp.NewTabPageUma;
-import org.chromium.chrome.browser.offlinepages.OfflinePageBridge;
-import org.chromium.chrome.browser.offlinepages.OfflinePageFreeUpSpaceCallback;
-import org.chromium.chrome.browser.offlinepages.OfflinePageFreeUpSpaceDialog;
-import org.chromium.chrome.browser.offlinepages.OfflinePageOpenStorageSettingsDialog;
-import org.chromium.chrome.browser.offlinepages.OfflinePageStorageSpacePolicy;
-import org.chromium.chrome.browser.offlinepages.OfflinePageUtils;
-import org.chromium.chrome.browser.snackbar.Snackbar;
-import org.chromium.chrome.browser.snackbar.SnackbarManager;
-import org.chromium.chrome.browser.snackbar.SnackbarManager.SnackbarController;
-import org.chromium.chrome.browser.tab.Tab;
-import org.chromium.chrome.browser.util.FeatureUtilities;
-import org.chromium.components.bookmarks.BookmarkId;
-import org.chromium.components.bookmarks.BookmarkType;
-import org.chromium.content_public.browser.WebContents;
-import org.chromium.ui.base.DeviceFormFactor;
-
-/**
- * A class holding static util functions for enhanced bookmark.
- */
-public class EnhancedBookmarkUtils {
-    private static final String PREF_LAST_USED_URL = "enhanced_bookmark_last_used_url";
-    private static final String PREF_LAST_USED_PARENT = "enhanced_bookmark_last_used_parent_folder";
-
-    /**
-     * If the tab has already been bookmarked, start {@link EnhancedBookmarkEditActivity} for the
-     * bookmark. If not, add the bookmark to bookmarkmodel, and show a snackbar notifying the user.
-     * @param idToAdd The bookmark ID if the tab has already been bookmarked.
-     * @param bookmarkModel The enhanced bookmark model.
-     * @param tab The tab to add or edit a bookmark.
-     * @param snackbarManager The SnackbarManager used to show the snackbar.
-     * @param activity Current activity.
-     */
-    public static void addOrEditBookmark(long idToAdd, EnhancedBookmarksModel bookmarkModel,
-            Tab tab, SnackbarManager snackbarManager, Activity activity) {
-        // See if the Tab's contents should be saved or not.
-        WebContents webContentsToSave = null;
-        if (!shouldSkipSavingTabOffline(tab)) webContentsToSave = tab.getWebContents();
-
-        if (idToAdd != ChromeBrowserProviderClient.INVALID_BOOKMARK_ID) {
-            startEditActivity(activity, new BookmarkId(idToAdd, BookmarkType.NORMAL),
-                    webContentsToSave);
-            return;
-        }
-
-        BookmarkId parent = getLastUsedParent(activity);
-        if (parent == null || !bookmarkModel.doesBookmarkExist(parent)) {
-            parent = bookmarkModel.getDefaultFolder();
-        }
-
-        bookmarkModel.addBookmarkAsync(parent, bookmarkModel.getChildCount(parent), tab.getTitle(),
-                tab.getUrl(), webContentsToSave,
-                createAddBookmarkCallback(bookmarkModel, snackbarManager, activity));
-    }
-
-    /**
-     * Adds a bookmark with the given title and url to the last used parent folder. Provides
-     * no visual feedback that a bookmark has been added.
-     *
-     * @param title The title of the bookmark.
-     * @param url The URL of the new bookmark.
-     */
-    public static BookmarkId addBookmarkSilently(Context context,
-            EnhancedBookmarksModel bookmarkModel, String title, String url) {
-        BookmarkId parent = getLastUsedParent(context);
-        if (parent == null || !bookmarkModel.doesBookmarkExist(parent)) {
-            parent = bookmarkModel.getDefaultFolder();
-        }
-
-        return bookmarkModel.addBookmark(parent, bookmarkModel.getChildCount(parent), title, url);
-    }
-
-    /**
-     * Saves an offline copy for the specified tab that is bookmarked. A snackbar will be shown to
-     * notify the user.
-     * @param id The bookmark ID for the tab.
-     * @param bookmarkModel The enhanced bookmark model.
-     * @param tab The bookmarked tab to save an offline copy.
-     * @param snackbarManager The SnackbarManager used to show the snackbar.
-     * @param activity Current activity.
-     */
-    public static void saveBookmarkOffline(long id, EnhancedBookmarksModel bookmarkModel,
-            Tab tab, final SnackbarManager snackbarManager, Activity activity) {
-        assert id != ChromeBrowserProviderClient.INVALID_BOOKMARK_ID;
-        BookmarkId bookmarkId = new BookmarkId(id, BookmarkType.NORMAL);
-
-        // Bail out if the ID no longer points to a valid bookmark, which might happen if the user
-        // deleted the bookmark while the page was loading.
-        if (!bookmarkModel.doesBookmarkExist(bookmarkId)) return;
-
-        // Skip saving the offline page for the bookmark if the tab
-        // cannot be saved currently (error or sad tab being shown).
-        // TODO(sansid, petewil): Snackbar triggering for error tabs should be handled.
-        //      See: http://crbug/568310 for details.
-        if (shouldSkipSavingTabOffline(tab)) return;
-
-        bookmarkModel.saveOfflinePage(bookmarkId, tab.getWebContents(),
-                createAddBookmarkCallback(bookmarkModel, snackbarManager, activity));
-    }
-
-    private static void showSnackbarForAddingBookmark(final EnhancedBookmarksModel bookmarkModel,
-            final SnackbarManager snackbarManager, final Activity activity,
-            final BookmarkId bookmarkId, final int saveResult, boolean isStorageAlmostFull) {
-        Snackbar snackbar;
-        OfflinePageBridge offlinePageBridge = bookmarkModel.getOfflinePageBridge();
-        if (offlinePageBridge == null) {
-            String folderName = bookmarkModel
-                    .getBookmarkTitle(bookmarkModel.getBookmarkById(bookmarkId).getParentId());
-            SnackbarController snackbarController = createSnackbarControllerForEditButton(
-                    bookmarkModel, activity, bookmarkId);
-            if (getLastUsedParent(activity) == null) {
-                snackbar = Snackbar.make(activity.getString(R.string.enhanced_bookmark_page_saved),
-                        snackbarController, Snackbar.TYPE_ACTION);
-            } else {
-                snackbar = Snackbar.make(folderName, snackbarController, Snackbar.TYPE_ACTION)
-                        .setTemplateText(activity.getString(
-                                R.string.enhanced_bookmark_page_saved_folder));
-            }
-            snackbar = snackbar.setSingleLine(false)
-                    .setAction(activity.getString(R.string.enhanced_bookmark_item_edit), null);
-        } else {
-            SnackbarController snackbarController = null;
-            int messageId;
-            String suffix = null;
-            int buttonId = R.string.enhanced_bookmark_item_edit;
-
-            if (saveResult == AddBookmarkCallback.SKIPPED) {
-                messageId = OfflinePageUtils.getStringId(
-                        R.string.offline_pages_as_bookmarks_page_skipped);
-            } else if (isStorageAlmostFull) {
-                messageId = OfflinePageUtils.getStringId(saveResult == AddBookmarkCallback.SAVED
-                    ? R.string.offline_pages_as_bookmarks_page_saved_storage_near_full
-                    : R.string.offline_pages_as_bookmarks_page_failed_to_save_storage_near_full);
-                // Show "Free up space" button.
-                buttonId = OfflinePageUtils.getStringId(R.string.offline_pages_free_up_space_title);
-                snackbarController = createSnackbarControllerForFreeUpSpaceButton(
-                        bookmarkModel, snackbarManager, activity);
-            } else {
-                if (saveResult == AddBookmarkCallback.SAVED) {
-                    if (getLastUsedParent(activity) == null) {
-                        messageId = OfflinePageUtils.getStringId(
-                                R.string.offline_pages_as_bookmarks_page_saved);
-                    } else {
-                        messageId = OfflinePageUtils.getStringId(
-                                R.string.offline_pages_as_bookmarks_page_saved_folder);
-                        suffix = bookmarkModel.getBookmarkTitle(
-                                bookmarkModel.getBookmarkById(bookmarkId).getParentId());
-                    }
-                } else {
-                    messageId = OfflinePageUtils.getStringId(
-                            R.string.offline_pages_as_bookmarks_page_failed_to_save);
-                }
-            }
-            if (snackbarController == null) {
-                snackbarController = createSnackbarControllerForEditButton(
-                        bookmarkModel, activity, bookmarkId);
-            }
-            snackbar = Snackbar
-                    .make(activity.getString(messageId, suffix), snackbarController,
-                            Snackbar.TYPE_ACTION)
-                    .setAction(activity.getString(buttonId), null).setSingleLine(false);
-        }
-
-        snackbarManager.showSnackbar(snackbar);
-    }
-
-    private static AddBookmarkCallback createAddBookmarkCallback(
-            final EnhancedBookmarksModel bookmarkModel, final SnackbarManager snackbarManager,
-            final Activity activity) {
-        return new AddBookmarkCallback() {
-            @Override
-            public void onBookmarkAdded(final BookmarkId bookmarkId, final int saveResult) {
-                // Shows the snackbar right away when offline pages feature is not enabled since
-                // there is no need to wait to get the storage info.
-                if (bookmarkModel.getOfflinePageBridge() == null) {
-                    showSnackbarForAddingBookmark(bookmarkModel, snackbarManager, activity,
-                                bookmarkId, saveResult, false);
-                    return;
-                }
-
-                // Gets the storage info asynchronously which is needed to produce the message for
-                // the snackbar.
-                new AsyncTask<Void, Void, Boolean>() {
-                    @Override
-                    protected Boolean doInBackground(Void... params) {
-                        return OfflinePageUtils.isStorageAlmostFull();
-                    }
-
-                    @Override
-                    protected void onPostExecute(Boolean isStorageAlmostFull) {
-                        showSnackbarForAddingBookmark(bookmarkModel, snackbarManager, activity,
-                                bookmarkId, saveResult, isStorageAlmostFull);
-                    }
-                }.execute();
-            }
-        };
-    }
-
-    /**
-     * Creates a snackbar controller for a case where "Edit" button is shown to edit the newly
-     * created bookmark.
-     */
-    private static SnackbarController createSnackbarControllerForEditButton(
-            final EnhancedBookmarksModel bookmarkModel, final Activity activity,
-            final BookmarkId bookmarkId) {
-        return new SnackbarController() {
-
-            @Override
-            public void onDismissNoAction(Object actionData) {
-                RecordUserAction.record("EnhancedBookmarks.EditAfterCreateButtonNotClicked");
-                // This method will be called only if the snackbar is dismissed by timeout.
-                bookmarkModel.destroy();
-            }
-
-            @Override
-            public void onAction(Object actionData) {
-                RecordUserAction.record("EnhancedBookmarks.EditAfterCreateButtonClicked");
-                startEditActivity(activity, bookmarkId, null);
-                bookmarkModel.destroy();
-            }
-        };
-    }
-
-    /**
-     * Creates a snackbar controller for a case where "Free up space" button is shown to clean up
-     * space taken by the offline pages.
-     */
-    private static SnackbarController createSnackbarControllerForFreeUpSpaceButton(
-            final EnhancedBookmarksModel bookmarkModel, final SnackbarManager snackbarManager,
-            final Activity activity) {
-        return new SnackbarController() {
-            @Override
-            public void onDismissNoAction(Object actionData) {
-                // This method will be called only if the snackbar is dismissed by timeout.
-                RecordUserAction.record(
-                        "OfflinePages.SaveStatusSnackbar.FreeUpSpaceButtonNotClicked");
-                bookmarkModel.destroy();
-            }
-
-            @Override
-            public void onAction(Object actionData) {
-                RecordUserAction.record("OfflinePages.SaveStatusSnackbar.FreeUpSpaceButtonClicked");
-                OfflinePageStorageSpacePolicy policy =
-                        new OfflinePageStorageSpacePolicy(bookmarkModel.getOfflinePageBridge());
-                if (policy.hasPagesToCleanUp()) {
-                    OfflinePageFreeUpSpaceCallback callback = new OfflinePageFreeUpSpaceCallback() {
-                        @Override
-                        public void onFreeUpSpaceDone() {
-                            snackbarManager.showSnackbar(
-                                    OfflinePageFreeUpSpaceDialog.createStorageClearedSnackbar(
-                                            activity));
-                            bookmarkModel.destroy();
-                        }
-                        @Override
-                        public void onFreeUpSpaceCancelled() {
-                            bookmarkModel.destroy();
-                        }
-                    };
-                    OfflinePageFreeUpSpaceDialog dialog = OfflinePageFreeUpSpaceDialog.newInstance(
-                            bookmarkModel.getOfflinePageBridge(), callback);
-                    dialog.show(activity.getFragmentManager(), null);
-                } else {
-                    OfflinePageOpenStorageSettingsDialog.showDialog(activity);
-                }
-            }
-        };
-    }
-
-    /**
-     * Gets whether bookmark manager should load offline page initially.
-     */
-    private static boolean shouldShowOfflinePageAtFirst(EnhancedBookmarksModel model,
-            Context context) {
-        OfflinePageBridge bridge = model.getOfflinePageBridge();
-        if (bridge == null || bridge.getAllPages().isEmpty()
-                || OfflinePageUtils.isConnected(context)) {
-            return false;
-        }
-        return true;
-    }
-
-    /**
-     * Shows enhanced bookmark main UI.
-     */
-    public static void showBookmarkManager(Activity activity) {
-        String url = getFirstUrlToLoad(activity);
-
-        if (DeviceFormFactor.isTablet(activity)) {
-            openUrl(activity, url);
-        } else {
-            Intent intent = new Intent(activity, EnhancedBookmarkActivity.class);
-            intent.setData(Uri.parse(url));
-            activity.startActivity(intent);
-        }
-    }
-
-    /**
-     * The initial url the bookmark manager shows depends on offline page status and some
-     * experiments we run.
-     */
-    private static String getFirstUrlToLoad(Activity activity) {
-        EnhancedBookmarksModel model = new EnhancedBookmarksModel();
-        try {
-            if (shouldShowOfflinePageAtFirst(model, activity)) {
-                return EnhancedBookmarkUIState.createFilterUrl(EnhancedBookmarkFilter.OFFLINE_PAGES,
-                        false).toString();
-            }
-            String lastUsedUrl = getLastUsedUrl(activity);
-            if (!TextUtils.isEmpty(lastUsedUrl)) return lastUsedUrl;
-            return UrlConstants.BOOKMARKS_URL;
-        } finally {
-            model.destroy();
-        }
-    }
-
-    /**
-     * Saves the last used url to preference. The saved url will be later queried by
-     * {@link #getLastUsedUrl(Context)}
-     */
-    static void setLastUsedUrl(Context context, String url) {
-        PreferenceManager.getDefaultSharedPreferences(context).edit()
-                .putString(PREF_LAST_USED_URL, url).apply();
-    }
-
-    /**
-     * Fetches url representing the user's state last time they close the bookmark manager.
-     */
-    @VisibleForTesting
-    static String getLastUsedUrl(Context context) {
-        return PreferenceManager.getDefaultSharedPreferences(context).getString(
-                PREF_LAST_USED_URL, UrlConstants.BOOKMARKS_URL);
-    }
-
-    /**
-     * Save the last used {@link BookmarkId} as a folder to put new bookmarks to.
-     */
-    static void setLastUsedParent(Context context, BookmarkId bookmarkId) {
-        PreferenceManager.getDefaultSharedPreferences(context).edit()
-                .putString(PREF_LAST_USED_PARENT, bookmarkId.toString()).apply();
-    }
-
-    /**
-     * @return The parent {@link BookmarkId} that the user used the last time or null if the user
-     *         has never selected a parent folder to use.
-     */
-    static BookmarkId getLastUsedParent(Context context) {
-        SharedPreferences preferences = PreferenceManager.getDefaultSharedPreferences(context);
-        if (!preferences.contains(PREF_LAST_USED_PARENT)) return null;
-
-        return BookmarkId.getBookmarkIdFromString(
-                preferences.getString(PREF_LAST_USED_PARENT, null));
-    }
-
-    /**
-     * Starts an {@link EnhancedBookmarkEditActivity} for the given {@link BookmarkId}.
-     * If the given {@link WebContents} is null, an option to visit the page is shown
-     * as opposed to showing an option to directly save the page
-     * (only if offline pages are enabled).
-     */
-    public static void startEditActivity(
-            Context context, BookmarkId bookmarkId, WebContents webContents) {
-        Intent intent = new Intent(context, EnhancedBookmarkEditActivity.class);
-        intent.putExtra(EnhancedBookmarkEditActivity.INTENT_BOOKMARK_ID, bookmarkId.toString());
-        if (webContents != null) {
-            intent.putExtra(EnhancedBookmarkEditActivity.INTENT_WEB_CONTENTS, webContents);
-        }
-        if (context instanceof EnhancedBookmarkActivity) {
-            ((EnhancedBookmarkActivity) context).startActivityForResult(
-                    intent, EnhancedBookmarkActivity.EDIT_BOOKMARK_REQUEST_CODE);
-        } else {
-            context.startActivity(intent);
-        }
-    }
-
-    /**
-     * Opens a bookmark depending on connection status and reports UMA.
-     * @param model Enhanced bookmarks model to manage the bookmark.
-     * @param activity Activity requesting to open the bookmark.
-     * @param bookmarkId ID of the bookmark to be opened.
-     * @param launchLocation Location from which the bookmark is being opened.
-     * @return Whether the bookmark was successfully opened.
-     */
-    public static boolean openBookmark(EnhancedBookmarksModel model, Activity activity,
-            BookmarkId bookmarkId, int launchLocation) {
-        if (model.getBookmarkById(bookmarkId) == null) return false;
-
-        String url = model.getLaunchUrlAndMarkAccessed(activity, bookmarkId);
-
-        // TODO(jianli): Notify the user about the failure.
-        if (TextUtils.isEmpty(url)) return false;
-
-        NewTabPageUma.recordAction(NewTabPageUma.ACTION_OPENED_BOOKMARK);
-        if (url.startsWith("file:")) {
-            RecordHistogram.recordEnumeratedHistogram(
-                    "OfflinePages.LaunchLocation", launchLocation, BookmarkLaunchLocation.COUNT);
-        } else {
-            RecordHistogram.recordEnumeratedHistogram(
-                    "Stars.LaunchLocation", launchLocation, BookmarkLaunchLocation.COUNT);
-        }
-
-        openUrl(activity, url);
-        return true;
-    }
-
-    private static void openUrl(Activity activity, String url) {
-        Intent intent = new Intent(Intent.ACTION_VIEW, Uri.parse(url));
-        intent.setClassName(activity.getApplicationContext().getPackageName(),
-                ChromeLauncherActivity.class.getName());
-        intent.putExtra(Browser.EXTRA_APPLICATION_ID,
-                activity.getApplicationContext().getPackageName());
-        intent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
-        IntentHandler.startActivityForTrustedIntent(intent, activity);
-    }
-
-    /**
-     * Updates the title of chrome shown in recent tasks. It only takes effect in document mode.
-     */
-    public static void setTaskDescriptionInDocumentMode(Activity activity, String description) {
-        if (FeatureUtilities.isDocumentMode(activity)) {
-            // Setting icon to be null and color to be 0 will means "take no effect".
-            ApiCompatibilityUtils.setTaskDescription(activity, description, null, 0);
-        }
-    }
-
-    /**
-     * Closes the EnhancedBookmark Activity on Phone. Does nothing on tablet.
-     */
-    public static void finishActivityOnPhone(Context context) {
-        if (context instanceof EnhancedBookmarkActivity) {
-            ((Activity) context).finish();
-        }
-    }
-
-    /**
-     * Indicates whether we should skip saving the given tab as an offline page.
-     * A tab shouldn't be saved offline if it shows an error page or a sad tab page.
-     */
-    private static boolean shouldSkipSavingTabOffline(Tab tab) {
-        return tab.isShowingErrorPage() || tab.isShowingSadTab();
-    }
-}