2011-06-01 Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>
authorCaio Marcelo de Oliveira Filho <cmarcelo@webkit.org>
Wed, 1 Jun 2011 16:58:35 +0000 (16:58 +0000)
committerAdemar de Souza Reis Jr <ademar.reis@openbossa.org>
Wed, 1 Jun 2011 18:28:43 +0000 (15:28 -0300)
commit0c7056f091bd373122c66e11ecb1ebff248e7f76
treee0aaf5161193070292c5bf939d100a63b8371f83
parent52da9d6e54502d9207e795e56a861dd9228a6c52
2011-06-01  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>

        Reviewed by Kenneth Rohde Christiansen.

        [Qt]Fix tst_QWebFrame::setUrlToInvalid() autotest after r84762
        https://bugs.webkit.org/show_bug.cgi?id=59345

        KURL and QUrl disagree whether certain URLs are valid or not. The regression here
        was caused by the fact that now KURL accepts "http:/example.com" (note only one
        slash) and our test case used a strange edge case that's transformed into a
        "one-slash" URL that now KURL can handle.

        QtWebKit approach in this case is to do a best effort and accept the QUrl if KURL
        can understand it. So I've updated the test to use a more meaningful example and
        show that an invalid URL gets converted to a valid URL if possible.

        * tests/qwebframe/tst_qwebframe.cpp:
        (tst_QWebFrame::setUrlToInvalid):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@87818 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebKit/qt/ChangeLog
Source/WebKit/qt/tests/qwebframe/tst_qwebframe.cpp