RESOLVED FIXED Bug 84705
[EFL] Enable Web Timing 
https://bugs.webkit.org/show_bug.cgi?id=84705
Summary [EFL] Enable Web Timing
Chris Dumez
Reported 2012-04-24 04:05:15 PDT
Web Timing should be enabled for the EFL port.
Attachments
Patch (3.81 KB, patch)
2012-04-24 04:15 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2012-04-24 04:15:30 PDT
Created attachment 138533 [details] Patch
Tony Gentilcore
Comment 2 2012-04-24 04:34:27 PDT
Comment on attachment 138533 [details] Patch In order for navigation timing to completely work, each port needs to provide some detailed network times via the platform/network/ResourceLoadTiming.h API. Without it, some parts of the API won't be correct. I'm happy to give an r+ either way, but just wanted to make sure that was already hooked up or else it is understood that some information won't be correct.
Chris Dumez
Comment 3 2012-04-24 04:39:17 PDT
I believe that work was already done in Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp as part of enabling Web Timing for GTK port. The EFL port is using libsoup as well. Am I missing something?
Chris Dumez
Comment 4 2012-04-24 04:40:26 PDT
I'm referring to this patch: https://bugs.webkit.org/attachment.cgi?id=134524&action=prettypatch
Tony Gentilcore
Comment 5 2012-04-24 05:11:19 PDT
Comment on attachment 138533 [details] Patch Thanks for confirming. I just wasn't aware that libsoup was used by EFL. Excited to see this turned on.
WebKit Review Bot
Comment 6 2012-04-24 06:48:43 PDT
Comment on attachment 138533 [details] Patch Clearing flags on attachment: 138533 Committed r115046: <http://trac.webkit.org/changeset/115046>
WebKit Review Bot
Comment 7 2012-04-24 06:48:48 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.