From 40cda1d75862f0909bf3c2aa1360e92a20f7671b Mon Sep 17 00:00:00 2001
From: Pawel Obarzanek <pawelobarzanek@gmail.com>
Date: Tue, 22 Oct 2019 23:34:05 +0200
Subject: [PATCH] =?UTF-8?q?Closes=20#2435=20Change=20edit=20icon=20to=20sh?=
 =?UTF-8?q?are=20icon=20on=20the=20book=E2=80=A6=20(#6137)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Closes mozilla-mobile#2435 Change edit icon to share icon on the bookmark toolbar

* Closes mozilla-mobile#2435 Changed raw string to string resources

* Pass bookmark arguments to share fragment

* Closes mozilla-mobile#2435 add orEmpty
---
 .../fenix/library/bookmarks/BookmarkFragment.kt       | 11 +++++++----
 app/src/main/res/menu/bookmarks_select_multi.xml      |  6 +++---
 2 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt
index 8bda529a3..435112979 100644
--- a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt
+++ b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt
@@ -46,6 +46,7 @@ import org.mozilla.fenix.ext.minus
 import org.mozilla.fenix.ext.nav
 import org.mozilla.fenix.ext.urlToTrimmedHost
 import org.mozilla.fenix.library.LibraryPageFragment
+import org.mozilla.fenix.share.ShareTab
 import org.mozilla.fenix.utils.allowUndo
 
 @Suppress("TooManyFunctions", "LargeClass")
@@ -164,7 +165,7 @@ class BookmarkFragment : LibraryPageFragment<BookmarkNode>(), BackHandler {
                     inflater.inflate(R.menu.bookmarks_select_multi, menu)
                 }
 
-                menu.findItem(R.id.edit_bookmark_multi_select)?.isVisible = mode.selectedItems.size == 1
+                menu.findItem(R.id.share_bookmark_multi_select)?.isVisible = mode.selectedItems.size == 1
             }
         }
     }
@@ -197,11 +198,13 @@ class BookmarkFragment : LibraryPageFragment<BookmarkNode>(), BackHandler {
                 metrics?.track(Event.OpenedBookmarksInPrivateTabs)
                 true
             }
-            R.id.edit_bookmark_multi_select -> {
+            R.id.share_bookmark_multi_select -> {
                 val bookmark = bookmarkStore.state.mode.selectedItems.first()
                 navigate(
-                    BookmarkFragmentDirections.actionBookmarkFragmentToBookmarkEditFragment(
-                        bookmark.guid
+                    BookmarkFragmentDirections.actionBookmarkFragmentToShareFragment(
+                        url = bookmark.url,
+                        title = bookmark.title,
+                        tabs = arrayOf(ShareTab(bookmark.url.orEmpty(), bookmark.title.orEmpty()))
                     )
                 )
                 true
diff --git a/app/src/main/res/menu/bookmarks_select_multi.xml b/app/src/main/res/menu/bookmarks_select_multi.xml
index 2901f2dd2..27a79b122 100644
--- a/app/src/main/res/menu/bookmarks_select_multi.xml
+++ b/app/src/main/res/menu/bookmarks_select_multi.xml
@@ -5,10 +5,10 @@
 <menu xmlns:android="http://schemas.android.com/apk/res/android"
         xmlns:app="http://schemas.android.com/apk/res-auto">
     <item
-        android:id="@+id/edit_bookmark_multi_select"
-        android:icon="@drawable/ic_edit"
+        android:id="@+id/share_bookmark_multi_select"
+        android:icon="@drawable/ic_hollow_share"
         app:iconTint="@color/white_color"
-        android:title="@string/bookmark_edit"
+        android:title="@string/browser_menu_share"
         app:showAsAction="ifRoom" />
     <item
         android:id="@+id/open_bookmarks_in_new_tabs_multi_select"