133469 2014-06-03 01:02:16 -0700 [SOUP] ResourceLoadTiming.responseStart is always 0 2014-06-03 05:04:39 -0700 1 1 1 Unclassified WebKit Platform 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED Gtk P2 Normal --- 1 cgarcia webkit-unassigned berto cdumez commit-queue danw gustavo mrobinson svillar oldest_to_newest 1013063 0 cgarcia 2014-06-03 01:02:16 -0700 We are not recording the responseStart in the soup backend. Tests don't detect that because they time reported to the js API is always relative to the navigation time (navigation start + responseStart), and test using local files check that requestStart <= responseStart (and they are equal) and test using HTTP only check responseStart is not 0 (it can never be 0 unless navigationStart is 0). 1013064 1 232417 cgarcia 2014-06-03 01:04:22 -0700 Created attachment 232417 Patch 1013066 2 232417 svillar 2014-06-03 02:05:51 -0700 Comment on attachment 232417 Patch Doesn't it allow us to pass any extra test? 1013067 3 cgarcia 2014-06-03 02:12:28 -0700 (In reply to comment #2) > (From update of attachment 232417 [details]) > Doesn't it allow us to pass any extra test? "Tests don't detect that because they time reported to the js API is always relative to the navigation time (navigation start + responseStart), and test using local files check that requestStart <= responseStart (and they are equal) and test using HTTP only check responseStart is not 0 (it can never be 0 unless navigationStart is 0)." :-) 1013083 4 cgarcia 2014-06-03 05:04:39 -0700 Committed r169560: <http://trac.webkit.org/changeset/169560> 232417 2014-06-03 01:04:22 -0700 2014-06-03 02:05:50 -0700 Patch wk-soup-web-timing.diff text/plain 1448 cgarcia ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCBmMWY1NzdkLi45Mzc1YTZkIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTMg QEAKKzIwMTQtMDYtMDMgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t PgorCisgICAgICAgIFtTT1VQXSBSZXNvdXJjZUxvYWRUaW1pbmcucmVzcG9uc2VTdGFydCBpcyBh bHdheXMgMAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9 MTMzNDY5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg KiBwbGF0Zm9ybS9uZXR3b3JrL3NvdXAvUmVzb3VyY2VIYW5kbGVTb3VwLmNwcDoKKyAgICAgICAg KFdlYkNvcmU6OnNlbmRSZXF1ZXN0Q2FsbGJhY2spOiBSZWNvcmQgdGhlIHJlc3BvbnNlU3RhcnQg dGltZS4KKwogMjAxNC0wNS0yOSAgQ2FybG9zIEdhcmNpYSBDYW1wb3MgIDxjZ2FyY2lhQGlnYWxp YS5jb20+CiAKICAgICAgICAgW0dUS10gQWRkIHN1cHBvcnQgZm9yIEhpRFBJIGljb25zCmRpZmYg LS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9uZXR3b3JrL3NvdXAvUmVzb3VyY2VIYW5k bGVTb3VwLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdvcmsvc291cC9SZXNvdXJj ZUhhbmRsZVNvdXAuY3BwCmluZGV4IGJiZTNiNDkuLjg5MDY0NzkgMTAwNjQ0Ci0tLSBhL1NvdXJj ZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdvcmsvc291cC9SZXNvdXJjZUhhbmRsZVNvdXAuY3BwCisr KyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdvcmsvc291cC9SZXNvdXJjZUhhbmRsZVNv dXAuY3BwCkBAIC02OTUsNiArNjk1LDEwIEBAIHN0YXRpYyB2b2lkIHNlbmRSZXF1ZXN0Q2FsbGJh Y2soR09iamVjdCosIEdBc3luY1Jlc3VsdCogcmVzdWx0LCBncG9pbnRlciBkYXRhKQogICAgICAg ICBkLT5tX3Jlc3BvbnNlLnNldEV4cGVjdGVkQ29udGVudExlbmd0aChzb3VwX3JlcXVlc3RfZ2V0 X2NvbnRlbnRfbGVuZ3RoKGQtPm1fc291cFJlcXVlc3QuZ2V0KCkpKTsKICAgICB9CiAKKyNpZiBF TkFCTEUoV0VCX1RJTUlORykKKyAgICBkLT5tX3Jlc3BvbnNlLnJlc291cmNlTG9hZFRpbWluZygp LnJlc3BvbnNlU3RhcnQgPSBtaWxpc2Vjb25kc1NpbmNlUmVxdWVzdChoYW5kbGUtPm1fcmVxdWVz dFRpbWUpOworI2VuZGlmCisKICAgICBpZiAoc291cE1lc3NhZ2UgJiYgZC0+bV9yZXNwb25zZS5p c011bHRpcGFydCgpKQogICAgICAgICBkLT5tX211bHRpcGFydElucHV0U3RyZWFtID0gYWRvcHRH UmVmKHNvdXBfbXVsdGlwYXJ0X2lucHV0X3N0cmVhbV9uZXcoc291cE1lc3NhZ2UsIGlucHV0U3Ry ZWFtLmdldCgpKSk7CiAgICAgZWxzZQo=