[webkit-reviews] review requested: [Bug 24122] REGRESSION: DOM Range extractContents/deleteContents bug : [Attachment 28771] proposed fix

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Mar 19 16:36:22 PDT 2009

Alexey Proskuryakov <ap at webkit.org> has asked  for review:
Bug 24122: REGRESSION: DOM Range extractContents/deleteContents bug

Attachment 28771: proposed fix

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
I couldn't find any remnants of code that could make this work, so it's
surprising that this is a regression. Perhaps this code was removed in hopes
that Document::textRemoved() would automatically take care of fixing the range?
It does not, and it is not supposed to - I've verified that a range that's
equal to the one that deleteContents is invoked on ends up being different from
it in Firefox:

var range1 = something
var range2 = the same
// Both ranges are fixed, but differently - range1 is always collapsed, but
range2 may not be.

More information about the webkit-reviews mailing list