76176 2012-01-12 07:25:17 -0800 Web Inspector: [Chomium] Resources loaded with 304 status code have receiving time of 15000 days in network panel. 2012-01-13 03:23:32 -0800 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 vsevik simonjam apavlov bweinstein japhet joepeck keishi loislo pfeldman pmuellr rik simonjam timothy vsevik webkit.review.bot yurys oldest_to_newest 534821 0 vsevik 2012-01-12 07:25:17 -0800 All resources with 304 status code have receiving time set to 15000. This can be easily reproduced by loading some site (e.g. cnn.com) and pressing f5 to reload. This was caused by http://trac.webkit.org/changeset/102961 (https://bugs.webkit.org/show_bug.cgi?id=58354). Switching from currentTime() to monotonicallyIncreasingTime() in SubresourceLoader (see link below) fixes it, but I am not sure it won't break things on other platforms. http://codesearch.google.com/codesearch#OAMlx_jo-ck/src/third_party/WebKit/Source/WebCore/loader/SubresourceLoader.cpp&exact_package=chromium&q=subresourceloader&type=cs&l=164 simonjam@, could you please have a look on that? 535134 1 simonjam 2012-01-12 13:58:23 -0800 Yep, that's the right fix. I'll upload a patch shortly. 535245 2 122328 simonjam 2012-01-12 15:56:20 -0800 Created attachment 122328 Patch 535252 3 simonjam 2012-01-12 16:01:11 -0800 Nate, it looks like you fixed revalidation shortly before I switched everything to monotonic time, and I missed your change. This should fix it to use monotonic time like the rest of resource loading. Would you mind reviewing it? 535350 4 122328 webkit.review.bot 2012-01-12 17:57:12 -0800 Comment on attachment 122328 Patch Clearing flags on attachment: 122328 Committed r104887: <http://trac.webkit.org/changeset/104887> 535351 5 webkit.review.bot 2012-01-12 17:57:16 -0800 All reviewed patches have been landed. Closing bug. 535561 6 vsevik 2012-01-13 03:23:32 -0800 FYI: I filed http://code.google.com/p/chromium/issues/detail?id=110154 to merge it to Chromium M17. 122328 2012-01-12 15:56:20 -0800 2012-01-12 17:57:12 -0800 Patch bug-76176-20120112155618.patch text/plain 1771 simonjam U3VidmVyc2lvbiBSZXZpc2lvbjogMTA0NDYyCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZTk3NTQ4NGYwY2M1ZTk1 MjZmNTEwYmFkOWEyMDhhZjg2M2ZmZjliZC4uZDQ3YjcyYWM1OTcyZGU0ZDlkY2YyOTM4NDg1MjU4 Y2QwYWY3OGFlOSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEyLTAxLTEyICBKYW1l cyBTaW1vbnNlbiAgPHNpbW9uamFtQGNocm9taXVtLm9yZz4KKworICAgICAgICBXZWIgSW5zcGVj dG9yOiBbQ2hvbWl1bV0gUmVzb3VyY2VzIGxvYWRlZCB3aXRoIDMwNCBzdGF0dXMgY29kZSBoYXZl IHJlY2VpdmluZyB0aW1lIG9mIDE1MDAwIGRheXMgaW4gbmV0d29yayBwYW5lbC4KKyAgICAgICAg aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTc2MTc2CisKKyAgICAgICAg UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTm8gbmV3IHRlc3RzLiBDYW4n dCB0cmlnZ2VyIHRoaXMgd2l0aCBpbnNwZWN0b3IgdGVzdHMuIFdpbGwgYWRkIG9uZSB3aGVuIHRo ZSBSZXNvdXJjZSBUaW1pbmcgQVBJIGlzIGluLgorCisgICAgICAgICogbG9hZGVyL1N1YnJlc291 cmNlTG9hZGVyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlN1YnJlc291cmNlTG9hZGVyOjpkaWRS ZWNlaXZlUmVzcG9uc2UpOiBVc2UgbW9ub3RvbmljIHRpbWUuCisKIDIwMTItMDEtMDkgIFNoZXJp ZmYgQm90ICA8d2Via2l0LnJldmlldy5ib3RAZ21haWwuY29tPgogCiAgICAgICAgIFVucmV2aWV3 ZWQsIHJvbGxpbmcgb3V0IHIxMDQ0MTguCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9sb2Fk ZXIvU3VicmVzb3VyY2VMb2FkZXIuY3BwIGIvU291cmNlL1dlYkNvcmUvbG9hZGVyL1N1YnJlc291 cmNlTG9hZGVyLmNwcAppbmRleCBiOWExZDdlM2ViMjMzMGU3YzUxNDlmNzAyMzZmNjc5MzhiNWYy ZDg2Li5kZjVhODk5NWE2OGI5N2QwZjY0YjJmMjBmMWIzOWNmNzFhMWJhMjE5IDEwMDY0NAotLS0g YS9Tb3VyY2UvV2ViQ29yZS9sb2FkZXIvU3VicmVzb3VyY2VMb2FkZXIuY3BwCisrKyBiL1NvdXJj ZS9XZWJDb3JlL2xvYWRlci9TdWJyZXNvdXJjZUxvYWRlci5jcHAKQEAgLTE2MSwxMCArMTYxLDEw IEBAIHZvaWQgU3VicmVzb3VyY2VMb2FkZXI6OmRpZFJlY2VpdmVSZXNwb25zZShjb25zdCBSZXNv dXJjZVJlc3BvbnNlJiByZXNwb25zZSkKICAgICAgICAgICAgIG1lbW9yeUNhY2hlKCktPnJldmFs aWRhdGlvblN1Y2NlZWRlZChtX3Jlc291cmNlLCByZXNwb25zZSk7CiAgICAgICAgICAgICBpZiAo IXJlYWNoZWRUZXJtaW5hbFN0YXRlKCkpIHsKICAgICAgICAgICAgICAgICBSZXNvdXJjZUxvYWRl cjo6ZGlkUmVjZWl2ZVJlc3BvbnNlKHJlc3BvbnNlKTsKLSAgICAgICAgICAgICAgICBkaWRGaW5p c2hMb2FkaW5nKGN1cnJlbnRUaW1lKCkpOworICAgICAgICAgICAgICAgIGRpZEZpbmlzaExvYWRp bmcobW9ub3RvbmljYWxseUluY3JlYXNpbmdUaW1lKCkpOwogICAgICAgICAgICAgfQogICAgICAg ICAgICAgcmV0dXJuOwotICAgICAgICB9IAorICAgICAgICB9CiAgICAgICAgIC8vIERpZCBub3Qg Z2V0IDMwNCByZXNwb25zZSwgY29udGludWUgYXMgYSByZWd1bGFyIHJlc291cmNlIGxvYWQuCiAg ICAgICAgIG1lbW9yeUNhY2hlKCktPnJldmFsaWRhdGlvbkZhaWxlZChtX3Jlc291cmNlKTsKICAg ICB9Cg==