Bug 125832 - Eliminate timerAlignmentinterval() recursion
Summary: Eliminate timerAlignmentinterval() recursion
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BlinkMergeCandidate
Depends on:
Blocks:
 
Reported: 2013-12-16 22:09 PST by Ryosuke Niwa
Modified: 2022-08-09 16:59 PDT (History)
5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2013-12-16 22:09:01 PST 
Consider merging https://chromium.googlesource.com/chromium/blink/+/acc34ea7332c9254511e069181750a7c097790b4
if the hang also reproduces in WebKit

The recursion breaks the call stack. We shouldn't do that.
Comment 1 Ryosuke Niwa 2013-12-16 22:09:14 PST 
Also see https://code.google.com/p/chromium/issues/detail?id=311317
Comment 2 Ahmad Saleem 2022-08-09 16:59:13 PDT 
From Comment 0, the following code exist today:

https://github.com/WebKit/WebKit/blob/d03ed349d77c73455b7ba94aa908b7dc0a5eddc8/Source/WebCore/dom/Document.cpp#L3364

While I don't have access to bug mentioned in Comment 01.

rniwa@webkit.org - Is this needed? Thanks!