[webkit-reviews] review denied: [Bug 63440] ASSERTION FAILED in Range::Range causes a crash in QtTestBrowser : [Attachment 98692] Patch v01 for review

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jun 27 10:36:01 PDT 2011


Ryosuke Niwa <rniwa at webkit.org> has denied Aparna Nandyal
<aparna.nand at wipro.com>'s request for review:
Bug 63440: ASSERTION FAILED in Range::Range causes a crash in QtTestBrowser
https://bugs.webkit.org/show_bug.cgi?id=63440

Attachment 98692: Patch v01 for review
https://bugs.webkit.org/attachment.cgi?id=98692&action=review

------- Additional Comments from Ryosuke Niwa <rniwa at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=98692&action=review

> Source/WebCore/ChangeLog:10
> +	   Node object used is of the new document (web page) though
> +	   the display is still is showing the old webpage. Removing 
> +	   the assertion.

That's a bug!  We shouldn't be removing assertions to suppress it.


More information about the webkit-reviews mailing list