[webkit-reviews] review granted: [Bug 16764] DOMRange doesn't correctly re-size when inserting items (Acid3) : [Attachment 18299] Proposed patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Andrew Wellington
<proton at wiretapped.net>'s request for review:
Bug 16764: DOMRange doesn't correctly re-size when inserting items (Acid3)
http://bugs.webkit.org/show_bug.cgi?id=16764

Attachment 18299: Proposed patch
http://bugs.webkit.org/attachment.cgi?id=18299&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
Usually a count would be named with a noun. The word "move" is more like a
verb, so "offsetMove" is not a great variable name. Maybe offsetDelta or
endOffsetAdjustment?

+    if (m_startContainer == m_endContainer)
+	 m_endOffset += offsetMove;

There's no if statement needed here -- you already ensured that the variable
will be 0 in the case where the two are not equal.

r=me