[webkit-reviews] review granted: [Bug 63908] chrome.dll!WebCore::firstPositionInNode ReadAV at NULL (b1c286808dfae0c5027c71627f71b962) : [Attachment 101491] Proposed patch.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 20 12:23:34 PDT 2011


Eric Seidel <eric at webkit.org> has granted Kulanthaivel Palanichamy
<kulanthaivel at codeaurora.org>'s request for review:
Bug 63908: chrome.dll!WebCore::firstPositionInNode ReadAV at NULL
(b1c286808dfae0c5027c71627f71b962)
https://bugs.webkit.org/show_bug.cgi?id=63908

Attachment 101491: Proposed patch.
https://bugs.webkit.org/attachment.cgi?id=101491&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
I wonder if the !documentElement() part of the check is necessary.  Does
firstPositionInNode() handle NULL?


More information about the webkit-reviews mailing list