1370281 - UIPasteboard causes long delay

Status: RESOLVED FIXED

(Firefox for iOS :: General, defect, P1)

Description

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

This seems to be caused by clipboard sharing between Mac and iPhone.

Comment 1

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Both devices need to be signed in to the same iCloud account. For me this happens between macOS 10.12.5 and iOS 10.3.2.

Comment 2

[Justin D'Arcangelo [:justindarc]]

Catalin, can you verify if this is happening in general? In particular, is there a many-second long delay in opening a new tab when a URL is copied from the paired-Mac's clipboard?

Comment 3

[Catalin Suciu [:csuciu]]

(In reply to Justin D'Arcangelo [:justindarc] from comment #2)
> Catalin, can you verify if this is happening in general?
I don't see any delays when using universal clipboard between iOS 10.3.2 (6S) <==> macOS 10.12.4 (MacBook Pro)

> In particular, is there a many-second long delay in opening a new tab when a URL is copied from the paired-Mac's clipboard?

I've copied links from Firefox for macOS and paste them in Firefox for iOS with no delays. Same for vice versa.

Comment 4

[Farhan Patel [:farhan]]

I've seen this happen for me a lot. Usually easier to replicate if you copy something on your mac right before you try going from the TabTray to the Tab view.

Comment 5

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Is it possible that this only happens on debug builds?

I would like to suggest the following:

If we cannot reproduce this on a recent TestFlight build:

- then we ship with the feature.

If this _does_ happen on a recent TestFlight build, we either:

- disable the feature completely and re-introduce it in 8.1 with a more structural fix
- change the pref to default this feature to off

Comment 6

[James Hugman [:jhugman] [@jhugman]]

There are a number of issues around this, but all stem from the fact that there is network activity connected to checking the clipboard, and it may be many seconds before the contents of the clipboard is available.

Comment 7

[James Hugman [:jhugman] [@jhugman]]

Attachment: Link to Github pull-request: https://github.com/mozilla-mobile/firefox-ios/pull/3067

Comment 8

[James Hugman [:jhugman] [@jhugman]]

Merged in to master.

Comment 9

[James Hugman [:jhugman] [@jhugman]]

Uplifted to v9.x

Comment 10

[James Hugman [:jhugman] [@jhugman]]

Uplifted to v9.x. 6f16c92d8