1434695 - Send tab share sheet takes too long to load list of sync'd browsers

Status: RESOLVED FIXED

(Firefox for iOS :: Sync, enhancement, P1)

Description

[Matthew Gaudet (he/him) [:mgaudet]]

When attempting to send a tab from another application via the "Send Tab" share sheet icon, I find the amount of time it takes to load the list of devices takes nearly as long as it would take to type in most URLs by hand, causing huge friction. 

My most recent attempt to send a tab from cold start (hadn't had firefox open in the last couple hours, but was using a few other applications), it took almost 12 seconds for my two synced devices to show up.

Comment 1

[Ryan Kelly [:rfkelly]]

Thanks for the report!  How many devices to you have attached to your account?

Comment 2

[Matthew Gaudet (he/him) [:mgaudet]]

Three: 2 MacOSX laptops, and 1 iOS device.

Comment 3

[Justin D'Arcangelo [:justindarc]]

This can possibly get addressed during the ongoing "Pushbox" work which may not happen until v12.x.

Comment 4

[Justin D'Arcangelo [:justindarc]]

Attachment: GitHub Pull Request

Posting :rnewman's PR here for my review.

Comment 5

[Richard Newman [:rnewman]]

https://github.com/mozilla-mobile/firefox-ios/commit/afce7dc695a862cb0be02824b4f491ce10fb0952

Comment 6

[Matthew Gaudet (he/him) [:mgaudet]]

Did this make it into version 11.0? 

I am still experiencing a lot of lag with send-tab. From inside firefox, 6 seconds to show the share sheet, and another 5 seconds for the tab send to actually occur and the sheet to be dismissed.

Comment 7

[Justin D'Arcangelo [:justindarc]]

(In reply to Matthew Gaudet (he/him) [:mgaudet] (UTC-5) from comment #6)
> Did this make it into version 11.0? 
> 
> I am still experiencing a lot of lag with send-tab. From inside firefox, 6
> seconds to show the share sheet, and another 5 seconds for the tab send to
> actually occur and the sheet to be dismissed.

No it didn't. It might make it for v11.1 though.

Comment 8

[Richard Newman [:rnewman]]

> No it didn't. It might make it for v11.1 though.

I see this change in current TestFlight (11.0 (10189)).