[webkit-reviews] review granted: [Bug 93351] [Qt] Remove duplicated platform test expectations : [Attachment 157653] Patch

[bugzilla-daemon at webkit.org]

Csaba Osztrogonac <ossy at webkit.org> has granted  review:
Bug 93351: [Qt] Remove duplicated platform test expectations
https://bugs.webkit.org/show_bug.cgi?id=93351

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

------- Additional Comments from Csaba Osztrogonac <ossy at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=157653&action=review


> LayoutTests/ChangeLog:50
> +	   * platform/qt-5.0/svg/W3C-SVG-1.1-SE/color-prop-05-t-expected.txt:
Removed.
> +	   *
platform/qt-5.0/svg/W3C-SVG-1.1-SE/filters-image-03-f-expected.txt: Removed.
> +	   *
platform/qt-5.0/svg/W3C-SVG-1.1-SE/pservers-pattern-03-f-expected.txt: Removed.

> +	   * platform/qt-5.0/svg/W3C-SVG-1.1-SE/struct-use-14-f-expected.txt:
Removed.
> +	   * platform/qt-5.0/svg/W3C-SVG-1.1-SE/svgdom-over-01-f-expected.txt:
Removed.
> +	   * platform/qt-5.0/svg/W3C-SVG-1.1-SE/types-dom-05-b-expected.txt:
Removed.
> +	   * platform/qt-5.0/svg/foreignObject/text-tref-02-b-expected.txt:
Removed.

These tests fail after removing these results, because there are orphaned 
expected file in platform/qt. I'll remove them too when I land your patch.

Otherwise LGTM, but I'll run a WK1 and a WK2 test before landing.
(Tools/Scripts/svn-apply applied the patch for me ... It seems
there is a bug somewhere in webkit-patch code ... )