[Webkit-unassigned] [Bug 119322] New: javascript: urls should always be parsed synchronously
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jul 30 22:02:11 PDT 2013
https://bugs.webkit.org/show_bug.cgi?id=119322
Summary: javascript: urls should always be parsed synchronously
Product: WebKit
Version: 528+ (Nightly build)
Platform: Unspecified
OS/Version: Unspecified
Status: NEW
Keywords: BlinkMergeCandidate
Severity: Normal
Priority: P2
Component: HTML DOM
AssignedTo: webkit-unassigned at lists.webkit.org
ReportedBy: rniwa at webkit.org
CC: eric at webkit.org, ap at webkit.org, koivisto at iki.fi,
benjamin at webkit.org, akling at apple.com
Consider merging https://chromium.googlesource.com/chromium/blink/+/cfbec542d3a70df79e9238f15740f7f37ef5eda3
Removing the minimumLayoutDelay() threshold as part of:
https://codereview.chromium.org/20140002/
revealed that we weren't always parsing javascript: urls
synchronously. This change is a preventative fix to allow
me to move forward with this change.
In order to trigger this in current webkit, we'd need to parse
a javascript: url which included a script which took more than .5
seconds to execute and then had a second script tag.
We'd (incorrectly) yield between the two script tags while parsing
the html result of the javascript: url.
I also renamed the isLayoutTimer active method on Document to
better reflect what it's actually used for.
I also added a FIXME to WebViewImpl.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list