[Offline pages] Removing offline pages from Bookmarks UI

chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkActionBar.java

Index: chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkActionBar.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkActionBar.java b/chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkActionBar.java
index 0dc98288bea282b26cf37dbd43538e53e8085b3e..f8cbbeb923751660ad7fe13974b21e784632d81b 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkActionBar.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkActionBar.java
@@ -19,7 +19,6 @@ import org.chromium.base.ApiCompatibilityUtils;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.bookmarks.BookmarkBridge.BookmarkItem;
 import org.chromium.chrome.browser.bookmarks.BookmarkBridge.BookmarkModelObserver;
-import org.chromium.chrome.browser.offlinepages.OfflinePageUtils;
 import org.chromium.chrome.browser.widget.NumberRollView;
 import org.chromium.components.bookmarks.BookmarkId;
 import org.chromium.components.bookmarks.BookmarkType;
@@ -58,16 +57,16 @@ public class BookmarkActionBar extends Toolbar implements BookmarkUIObserver,
 
         getMenu()

[Ian Wen, 2016/04/18 18:33:13]

Reformoat #58 - #69? At lease getMenu() and findItem() should be on the same
line.

[fgorski, 2016/04/18 23:39:05]

Done.

                 .findItem(R.id.search_menu_id)
-                .setTitle(OfflinePageUtils.getStringId(R.string.bookmark_action_bar_search));
+                .setTitle(R.string.bookmark_action_bar_search);
         getMenu()
                 .findItem(R.id.selection_mode_edit_menu_id)
-                .setTitle(OfflinePageUtils.getStringId(R.string.edit_bookmark));
+                .setTitle(R.string.edit_bookmark);
         getMenu()
                 .findItem(R.id.selection_mode_move_menu_id)
-                .setTitle(OfflinePageUtils.getStringId(R.string.bookmark_action_bar_move));
+                .setTitle(R.string.bookmark_action_bar_move);
         getMenu()
                 .findItem(R.id.selection_mode_delete_menu_id)
-                .setTitle(OfflinePageUtils.getStringId(R.string.bookmark_action_bar_delete));
+                .setTitle(R.string.bookmark_action_bar_delete);
     }
 
     @Override
@@ -108,7 +107,7 @@ public class BookmarkActionBar extends Toolbar implements BookmarkUIObserver,
             if (item.isFolder()) {
                 BookmarkAddEditFolderActivity.startEditFolderActivity(getContext(), item.getId());
             } else {
-                BookmarkUtils.startEditActivity(getContext(), item.getId(), null);
+                BookmarkUtils.startEditActivity(getContext(), item.getId());
             }
             return true;
         } else if (menuItem.getItemId() == R.id.selection_mode_move_menu_id) {
@@ -217,7 +216,7 @@ public class BookmarkActionBar extends Toolbar implements BookmarkUIObserver,
 
     @Override
     public void onAllBookmarksStateSet() {
-        setTitle(getTitleForAllItems());
+        setTitle(R.string.bookmark_title_bar_all_items);
         setNavigationButton(NAVIGATION_BUTTON_MENU);
         getMenu().findItem(R.id.search_menu_id).setVisible(true);
         getMenu().findItem(R.id.edit_menu_id).setVisible(false);
@@ -234,7 +233,7 @@ public class BookmarkActionBar extends Toolbar implements BookmarkUIObserver,
         if (mDelegate.getModel().getTopLevelFolderParentIDs().contains(
                 mCurrentFolder.getParentId())) {
             if (TextUtils.isEmpty(mCurrentFolder.getTitle())) {
-                setTitle(getTitleForAllItems());
+                setTitle(R.string.bookmark_title_bar_all_items);
             } else {
                 setTitle(mCurrentFolder.getTitle());
             }
@@ -246,14 +245,6 @@ public class BookmarkActionBar extends Toolbar implements BookmarkUIObserver,
     }
 
     @Override
-    public void onFilterStateSet(BookmarkFilter filter) {
-        assert filter == BookmarkFilter.OFFLINE_PAGES;
-        setTitle(R.string.bookmark_title_bar_filter_offline_pages);
-        setNavigationButton(NAVIGATION_BUTTON_MENU);
-        getMenu().findItem(R.id.edit_menu_id).setVisible(false);
-    }
-
-    @Override
     public void onSelectionStateChange(List<BookmarkId> selectedBookmarks) {
         boolean wasSelectionEnabled = mIsSelectionEnabled;
         mIsSelectionEnabled = mDelegate.isSelectionEnabled();
@@ -294,8 +285,4 @@ public class BookmarkActionBar extends Toolbar implements BookmarkUIObserver,
             mDelegate.notifyStateChange(this);
         }
     }
-
-    private int getTitleForAllItems() {
-        return OfflinePageUtils.getStringId(R.string.bookmark_title_bar_all_items);
-    }
 }