[webkit-reviews] review denied: [Bug 131044] REGRESSION(r166395): It made all media tests crash on EFL : [Attachment 239541] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 9 12:25:51 PDT 2014


Benjamin Poulain <benjamin at webkit.org> has denied Adrien Destugues
<pulkomandy at gmail.com>'s request for review:
Bug 131044: REGRESSION(r166395): It made all media tests crash on EFL
https://bugs.webkit.org/show_bug.cgi?id=131044

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

------- Additional Comments from Benjamin Poulain <benjamin at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=239541&action=review


> Source/WebCore/ChangeLog:9
> +	   No new tests: fixes existing tests.
> +	   The pseudo id cannot be set before the elements get a type.

The patch is not necessarily wrong, but the changelog is unsatisfactory.

Here you should explain exactly why the crash occur and why this is the right
fix for it.


More information about the webkit-reviews mailing list