Attachment 346805 Details for Bug 188391 – Patch

[patch] Patch

bug-188391.diff (text/plain), 5.83 KB, created by Don Olmstead on 2018-08-08 17:27:55 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Don Olmstead

Created: 2018-08-08 17:27:55 PDT

Size: 5.83 KB

patch

obsolete

>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 1a7de65ef11..e33db1bf8c6 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,13 @@
>+2018-08-08  Don Olmstead  <don.olmstead@sony.com>
>+
>+        [Curl] Surface additional NetworkLoadMetrics
>+        https://bugs.webkit.org/show_bug.cgi?id=188391
>+
>+        Reviewed by Joseph Pecoraro.
>+
>+        * platform/network/curl/CurlContext.cpp:
>+        (WebCore::CurlHandle::getNetworkLoadMetrics):
>+
> 2018-08-08  Don Olmstead  <don.olmstead@sony.com>
> 
>         [Nicosia] Add the Nicosia-specific PlatformLayer type alias
>diff --git a/Source/WebCore/platform/network/curl/CurlContext.cpp b/Source/WebCore/platform/network/curl/CurlContext.cpp
>index 166ee38f5be..2c28b62368a 100644
>--- a/Source/WebCore/platform/network/curl/CurlContext.cpp
>+++ b/Source/WebCore/platform/network/curl/CurlContext.cpp
>@@ -77,6 +77,11 @@ constexpr const char* EnvironmentVariableReader::sscanTemplate<signed>() { retur
> template<>
> constexpr const char* EnvironmentVariableReader::sscanTemplate<unsigned>() { return "%u"; }
> 
>+// ALPN Protocol ID (RFC7301) https://tools.ietf.org/html/rfc7301
>+static const ASCIILiteral httpVersion10 { "http/1.0"_s };
>+static const ASCIILiteral httpVersion11 { "http/1.1"_s };
>+static const ASCIILiteral httpVersion2 { "h2"_s };
>+
> // CurlContext -------------------------------------------------------------------
> 
> CurlContext& CurlContext::singleton()
>@@ -725,6 +730,13 @@ std::optional<NetworkLoadMetrics> CurlHandle::getNetworkLoadMetrics()
>     double connect = 0.0;
>     double appConnect = 0.0;
>     double startTransfer = 0.0;
>+    long requestHeaderSize = 0;
>+    curl_off_t requestBodySize = 0;
>+    long responseHeaderSize = 0;
>+    curl_off_t responseBodySize = 0;
>+    long version = 0;
>+    char* ip = nullptr;
>+    long port = 0;
> 
>     if (!m_handle)
>         return std::nullopt;
>@@ -745,6 +757,35 @@ std::optional<NetworkLoadMetrics> CurlHandle::getNetworkLoadMetrics()
>     if (errorCode != CURLE_OK)
>         return std::nullopt;
> 
>+    // FIXME: Gets total request size not just headers https://bugs.webkit.org/show_bug.cgi?id=188363
>+    errorCode = curl_easy_getinfo(m_handle, CURLINFO_REQUEST_SIZE, &requestHeaderSize);
>+    if (errorCode != CURLE_OK)
>+        return std::nullopt;
>+
>+    errorCode = curl_easy_getinfo(m_handle, CURLINFO_SIZE_UPLOAD_T, &requestBodySize);
>+    if (errorCode != CURLE_OK)
>+        return std::nullopt;
>+
>+    errorCode = curl_easy_getinfo(m_handle, CURLINFO_HEADER_SIZE, &responseHeaderSize);
>+    if (errorCode != CURLE_OK)
>+        return std::nullopt;
>+
>+    errorCode = curl_easy_getinfo(m_handle, CURLINFO_SIZE_DOWNLOAD_T, &responseBodySize);
>+    if (errorCode != CURLE_OK)
>+        return std::nullopt;
>+
>+    errorCode = curl_easy_getinfo(m_handle, CURLINFO_PRIMARY_IP, &ip);
>+    if (errorCode != CURLE_OK)
>+        return std::nullopt;
>+
>+    errorCode = curl_easy_getinfo(m_handle, CURLINFO_PRIMARY_PORT, &port);
>+    if (errorCode != CURLE_OK)
>+        return std::nullopt;
>+
>+    errorCode = curl_easy_getinfo(m_handle, CURLINFO_HTTP_VERSION, &version);
>+    if (errorCode != CURLE_OK)
>+        return std::nullopt;
>+
>     NetworkLoadMetrics networkLoadMetrics;
> 
>     networkLoadMetrics.domainLookupStart = Seconds(0);
>@@ -760,6 +801,24 @@ std::optional<NetworkLoadMetrics> CurlHandle::getNetworkLoadMetrics()
>     networkLoadMetrics.requestStart = networkLoadMetrics.connectEnd;
>     networkLoadMetrics.responseStart = Seconds(startTransfer);
> 
>+    networkLoadMetrics.requestHeaderBytesSent = requestHeaderSize;
>+    networkLoadMetrics.requestBodyBytesSent = requestBodySize;
>+    networkLoadMetrics.responseHeaderBytesReceived = responseHeaderSize;
>+    networkLoadMetrics.responseBodyBytesReceived = responseBodySize;
>+
>+    if (ip) {
>+        networkLoadMetrics.remoteAddress = String(ip);
>+        if (port)
>+            networkLoadMetrics.remoteAddress.append(":" + String::number(port));
>+    }
>+
>+    if (version == CURL_HTTP_VERSION_1_0)
>+        networkLoadMetrics.protocol = httpVersion10;
>+    else if (version == CURL_HTTP_VERSION_1_1)
>+        networkLoadMetrics.protocol = httpVersion11;
>+    else if (version == CURL_HTTP_VERSION_2)
>+        networkLoadMetrics.protocol = httpVersion2;
>+
>     return networkLoadMetrics;
> }
> 
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index b854c601526..846be07fbb4 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,13 @@
>+2018-08-08  Don Olmstead  <don.olmstead@sony.com>
>+
>+        [Curl] Surface additional NetworkLoadMetrics
>+        https://bugs.webkit.org/show_bug.cgi?id=188391
>+
>+        Reviewed by Joseph Pecoraro.
>+
>+        * NetworkProcess/curl/NetworkDataTaskCurl.cpp:
>+        (WebKit::NetworkDataTaskCurl::curlDidReceiveResponse):
>+
> 2018-08-08  Commit Queue  <commit-queue@webkit.org>
> 
>         Unreviewed, rolling out r234314, r234320, and r234321.
>diff --git a/Source/WebKit/NetworkProcess/curl/NetworkDataTaskCurl.cpp b/Source/WebKit/NetworkProcess/curl/NetworkDataTaskCurl.cpp
>index fbc89df0c7a..e7c487f5ee7 100644
>--- a/Source/WebKit/NetworkProcess/curl/NetworkDataTaskCurl.cpp
>+++ b/Source/WebKit/NetworkProcess/curl/NetworkDataTaskCurl.cpp
>@@ -147,13 +147,14 @@ void NetworkDataTaskCurl::curlDidSendData(CurlRequest&, unsigned long long total
>     m_client->didSendData(totalBytesSent, totalBytesExpectedToSend);
> }
> 
>-void NetworkDataTaskCurl::curlDidReceiveResponse(CurlRequest&, const CurlResponse& receivedResponse)
>+void NetworkDataTaskCurl::curlDidReceiveResponse(CurlRequest& request, const CurlResponse& receivedResponse)
> {
>     auto protectedThis = makeRef(*this);
>     if (state() == State::Canceling || state() == State::Completed || !m_client)
>         return;
> 
>     m_response = ResourceResponse(receivedResponse);
>+    m_response.setDeprecatedNetworkLoadMetrics(request.networkLoadMetrics().isolatedCopy());
> 
>     handleCookieHeaders(receivedResponse);
>

Actions: View | Formatted Diff | Diff

Attachments on bug 188391: 346737 | 346738 | 346805