[fenix] For https://github.com/mozilla-mobile/fenix/issues/1092: Fixes ordering of custom tab menu items (https://github.com/mozilla-mobile/fenix/pull/2072)

app/src/main/java/org/mozilla/fenix/customtabs/CustomTabToolbarMenu.kt

@@ -91,17 +91,17 @@ class CustomTabToolbarMenu(
 
     private val menuItems by lazy {
         listOf(
-            SimpleBrowserMenuItem(
+            menuToolbar,
-                {
-                    val appName = context.getString(R.string.app_name)
-                    context.getString(R.string.browser_menu_powered_by, appName).toUpperCase()
-                }(),
-                ToolbarMenu.CAPTION_TEXT_SIZE,
-                DefaultThemeManager.resolveAttribute(R.attr.primaryText, context)
-            ),
 
             BrowserMenuDivider(),
 
+            BrowserMenuImageText(
+                context.getString(R.string.browser_menu_share),
+                R.drawable.mozac_ic_share
+            ) {
+                onItemTapped.invoke(ToolbarMenu.Item.Share)
+            },
+
             SimpleBrowserMenuItem(
                 {
                     val appName = context.getString(R.string.app_name)
@@ -115,14 +115,16 @@ class CustomTabToolbarMenu(
                 onItemTapped.invoke(ToolbarMenu.Item.OpenInFenix)
             },
 
-            BrowserMenuImageText(
+            BrowserMenuDivider(),
-                context.getString(R.string.browser_menu_share),
+
-                R.drawable.mozac_ic_share,
+            SimpleBrowserMenuItem(
+                {
+                    val appName = context.getString(R.string.app_name)
+                    context.getString(R.string.browser_menu_powered_by, appName).toUpperCase()
+                }(),
+                ToolbarMenu.CAPTION_TEXT_SIZE,
                 DefaultThemeManager.resolveAttribute(R.attr.primaryText, context)
-            ) {
+            )
-                onItemTapped.invoke(ToolbarMenu.Item.Share)
-            },
-            menuToolbar
         )
     }
 }

app/src/main/java/org/mozilla/fenix/customtabs/CustomTabsIntegration.kt

@@ -34,6 +34,7 @@ class CustomTabsIntegration(
         toolbar,
         sessionId,
         customTabToolbarMenu.menuBuilder,
+        START_OF_MENU_ITEMS_INDEX,
         closeListener = { activity?.finish() })
 
     override fun start() {
@@ -47,4 +48,8 @@ class CustomTabsIntegration(
     override fun onBackPressed(): Boolean {
         return feature.onBackPressed()
     }
+
+    companion object {
+        const val START_OF_MENU_ITEMS_INDEX = 2
+    }
 }