Open Bug 2027960 Opened 4 months ago Updated 4 months ago

Drag-and-drop 5MB of text on the URL-Bar hangs teh parent-process, spending 8s in nsIURIFixup.getFixupURIInfo

Categories

(Firefox :: Address Bar, defect, P2)

defect

Tracking

()

People

(Reporter: mayankleoboy1, Unassigned)

References

Details

(Keywords: perf, Whiteboard: [sng])

Attachments

(1 file)

4.73 MB, application/x-javascript
Details
Attached file p5.js

Open a nee blank tab
Download the attached sample text to your local machine
Select all the text and drag-and-drop the text to the URL-bar

AR: Long jank that hangs the parent process
ER: Not so?

I thought that bug 2002920 would have fixed this?

Profile: https://share.firefox.dev/4sInZVL

Summary: Drag-and-drop 5MB of text on the URL-Bar spends 8s in nsIURIFixup.getFixupURIInfo → Drag-and-drop 5MB of text on the URL-Bar hangs teh parent-process, spending 8s in nsIURIFixup.getFixupURIInfo

And if you copy-paste the text in the URL-bar, the text seems to be processed twice , or atleast takes twice the time: https://share.firefox.dev/48kl5hw

Component: Search → Address Bar

(In reply to Mayank Bansal from comment #0)

I thought that bug 2002920 would have fixed this?

not all the hangs come from the same code, that bug fixed the dragover and parsing, it looks like there's other stuff that happens after the drop when a search starts.

Most of the time in the profile is spent in matchUnknownUrl when it tries to fixup the string, and it's the Regexes .test calls. I'm pretty sure we can avoid some or most of them if we can easily establish this is not a URI, or at least limit them to substrings.

Severity: -- → S3
Keywords: perf
Priority: -- → P2
Whiteboard: [sng]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: