QRect autotest misuses QEXPECT_FAIL

In tbase/tests/auto/corelib/tools/qrect/tst_qrect.cpp, the normalized() test function contains the following line:

    QEXPECT_FAIL("ZeroHeight1", "due to broken QRect definition (not possible to change)", Continue);

QEXPECT_FAIL is provided for the purpose of documenting how a known bug affects test results and enabling the tests to continue to work until the known bug is fixed. The case above seems more like something that should be documented in class documentation, as it will never be fixed – in which case the test should be corrected to match the expected (but not necessarily optimal) behaviour.