Bug 39946 - REGRESSION (r58665): Infinite recursion in Position::getInlineBoxAndOffset()
Summary: REGRESSION (r58665): Infinite recursion in Position::getInlineBoxAndOffset()
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P1 Normal
Assignee: Nobody
URL:
Keywords: HasReduction, InRadar, Regression
Depends on:
Blocks:
 
Reported: 2010-05-31 02:42 PDT by Tony Chang
Modified: 2010-05-31 19:16 PDT (History)
2 users (show)

See Also:


Attachments
test case (256 bytes, text/html)
2010-05-31 02:42 PDT, Tony Chang
no flags Details
Patch (4.35 KB, patch)
2010-05-31 18:26 PDT, Tony Chang
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tony Chang 2010-05-31 02:42:03 PDT
Created attachment 57445 [details]
test case

This is a regression caused by a similar fix in http://trac.webkit.org/changeset/58665

Test case attached.

I'll investigate tomorrow.
Comment 1 Tony Chang 2010-05-31 02:43:05 PDT
Originally reported in crbug.com/44051
Comment 2 mitz 2010-05-31 09:04:09 PDT
<rdar://problem/8045485>
Comment 3 Tony Chang 2010-05-31 18:26:12 PDT
Created attachment 57506 [details]
Patch
Comment 4 mitz 2010-05-31 18:29:46 PDT
Comment on attachment 57506 [details]
Patch

Thanks for the quick fix!

> +        r58665 added an infinite recurion check, but didn't take into consideration recursion between two

Typo: recurion.
Comment 5 Tony Chang 2010-05-31 19:16:31 PDT
Committed r60459: <http://trac.webkit.org/changeset/60459>