WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
153002
Form submit event has a long delay before triggering when many controls on the page
https://bugs.webkit.org/show_bug.cgi?id=153002
Summary
Form submit event has a long delay before triggering when many controls on th...
Jamie Gaskins
Reported
2016-01-11 21:25:03 PST
A form's submit handler takes a long time to trigger when there are a lot of inputs on the page, even when they are not contained within that form. Here is an example:
http://jsbin.com/hasulikiqe/2/edit?html,js,console,output
It's a dummy todo-list app. With the JavaScript console open, you can see that it a second or so to trigger the submit handler (which just prints a message to the console) after entering text into the input field and pressing enter, even though none of the checkboxes are part of that form. When we remove the checkboxes, the delay is gone:
http://jsbin.com/rogipubexe/2/edit?html,js,console,output
Attachments
Add attachment
proposed patch, testcase, etc.
Chris Dumez
Comment 1
2016-01-12 12:19:53 PST
I can reproduce with tip of tree. I see most of the CPU time spent in client code (injected bundle) under: Running Time Self (ms) Symbol Name 994.0ms 24.5% 0.0 WebKit::InjectedBundlePageFormClient::willSendSubmitEvent(WebKit::WebPage*, WebCore::HTMLFormElement*, WebKit::WebFrame*, WebKit::WebFrame*, WTF::Vector<std::__1::pair<WTF::String, WTF::String>, 0ul, WTF::CrashOnOverflow, 16ul> const&)
Ryosuke Niwa
Comment 2
2016-01-12 12:35:06 PST
Maybe we're doing the work inside Safari autofill code?
Chris Dumez
Comment 3
2016-01-12 12:36:30 PST
(In reply to
comment #2
)
> Maybe we're doing the work inside Safari autofill code?
Yes, I was trying to get radar-importer to create a radar for me so I could post the profile and investigation results.
Chris Dumez
Comment 4
2016-01-12 13:03:31 PST
rdar://problem/24154762
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug