-
sergio@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=118395 Reviewed by Martin Robinson. Source/WebCore: WebTiming information was not correctly provided to WebCore because the gotHeadersCallback was incorrectly resetting the original ResourceResponse (which had the ResourceLoadTiming object) instead of simply updating their contents using the SoupMessage. No new test required as this feature is already covered by the existing webtiming tests. In any case this change includes a fix for the http/tests/misc/webtiming-ssl.php test which was not failing even if it should because it was not correct. * platform/network/soup/ResourceHandleSoup.cpp: (WebCore::gotHeadersCallback): (WebCore::restartedCallback): restartedCallback should be available only if WEB_TIMING is defined. (WebCore::createSoupMessageForHandleAndRequest): Ditto. LayoutTests: Fixed the webiming-ssl.php test that was incorrectly considering a FAIL as the expected result. It was probably just a legacy decision which came from the times where DRT was responsible of reporting WebTiming information. Also added a new test that fails due to wkb.ug/103927. It was not detected before because there were no timing information in the response. * http/tests/misc/resources/webtiming-ssl.html: * http/tests/misc/webtiming-ssl-expected.txt: * platform/gtk/TestExpectations: added http/tests/w3c/webperf/submission/Google/resource-timing/html/test_resource_attribute_order.html. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@154727 268f45cc-cd09-0410-ab3c-d52691b4dbfc
8e5df79d