RESOLVED FIXED 77874
[EFL] Drop support for the Curl network backend.
https://bugs.webkit.org/show_bug.cgi?id=77874
Summary [EFL] Drop support for the Curl network backend.
Raphael Kubo da Costa (:rakuco)
Reported 2012-02-06 07:55:04 PST
[EFL] Drop support for the Curl network backend.
Attachments
Patch (37.90 KB, patch)
2012-02-06 07:58 PST, Raphael Kubo da Costa (:rakuco)
no flags
Patch for landing (38.29 KB, patch)
2012-02-09 19:02 PST, Raphael Kubo da Costa (:rakuco)
no flags
Raphael Kubo da Costa (:rakuco)
Comment 1 2012-02-06 07:58:13 PST
Gyuyoung Kim
Comment 2 2012-02-06 08:30:55 PST
Gyuyoung Kim
Comment 3 2012-02-06 18:24:46 PST
I wanna decide to remove this libcurl support after profiling libcurl. The result will be updated soon.
Eric Seidel (no email)
Comment 4 2012-02-07 16:10:14 PST
Comment on attachment 125647 [details] Patch I'm happy to rubber-stamp this, except it seems to break the efl-ews.
Eric Seidel (no email)
Comment 5 2012-02-07 16:10:46 PST
Please make sure to follow-up by removing the actual curl support files if EFL was the last client of curl.
Leandro Pereira
Comment 6 2012-02-08 10:31:44 PST
(In reply to comment #5) > Please make sure to follow-up by removing the actual curl support files if EFL > was the last client of curl. The WinCairo port seems to use cURL. The other network backend used by the Windows port is CFNetwork, which still seems to be proprietary.
Raphael Kubo da Costa (:rakuco)
Comment 7 2012-02-09 19:02:10 PST
Created attachment 126431 [details] Patch for landing
Gyuyoung Kim
Comment 8 2012-02-09 21:44:03 PST
If many guys agree on removing curl port from WebCore, I also agree with this removal. But, I'm still wanna decide this removal after libcurl performance profiling in EFL port. I was told that the performance profiling result will be reported by next week.
Sungman Kim
Comment 9 2012-02-13 03:43:36 PST
I did comparing loading test with libsoup and libcurl last week. Total test site number is 96 and our test result is below. Average : libsoup 4.93sec libcurl 5.51sec Libcurl is better than libsoup just in case of 6 sites. Libcurl's performance is not better. These are our test results. Thank you.
Raphael Kubo da Costa (:rakuco)
Comment 10 2012-02-13 05:11:20 PST
(In reply to comment #9) > I did comparing loading test with libsoup and libcurl last week. > Total test site number is 96 and > our test result is below. > Average : > libsoup 4.93sec > libcurl 5.51sec > Libcurl is better than libsoup just in case of 6 sites. > Libcurl's performance is not better. > These are our test results. > Thank you. OK, thanks. For posterity, it would be good to know more about the methodology of these tests (ie. what exactly was tested and how).
Raphael Kubo da Costa (:rakuco)
Comment 11 2012-02-13 05:31:08 PST
Sungman Kim
Comment 12 2012-02-13 17:01:46 PST
OK. We added test codes for page logging loading time including some exception handling. If we start browser app. then it read the url lists for test and memorize loading start time each url loading start. When the loading is finished for one site, loading time is calculated and automatically start loading next url. For using these methods, we tested about 96 urls each 6 times. Each Libcurl and Libsoup devices are started loading synchronously. (actually a little term is exist) Thank you.
Note You need to log in before you can comment on or make changes to this bug.