[webkit-reviews] review denied: [Bug 103867] [Resource Timing] redirectStart & redirectEnd time need to be recorded to populate resource timing entries : [Attachment 177246] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Dec 3 12:53:22 PST 2012


Nate Chapin <japhet at chromium.org> has denied pdeng6 <pan.deng at intel.com>'s
request for review:
Bug 103867: [Resource Timing] redirectStart & redirectEnd time need to be
recorded to populate resource timing entries
https://bugs.webkit.org/show_bug.cgi?id=103867

Attachment 177246: Patch
https://bugs.webkit.org/attachment.cgi?id=177246&action=review

------- Additional Comments from Nate Chapin <japhet at chromium.org>
I don't really like leaving these values as member variables on ResourceLoader.
I'd feel much better if they were wrapped up as a struct.

Also, how are you planning on accessing this data? Since there are no callers
in this patch, it's hard for me to get a complete perspective.


More information about the webkit-reviews mailing list