Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1152)

Unified Diff: third_party/WebKit/Source/modules/websockets/DOMWebSocket.cpp

Issue 1652983005: Remove Enumeration Histograms from the Blink Platform API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master_blink_histograms_5a
Patch Set: Fix misplaced bracket on android Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/websockets/DOMWebSocket.cpp
diff --git a/third_party/WebKit/Source/modules/websockets/DOMWebSocket.cpp b/third_party/WebKit/Source/modules/websockets/DOMWebSocket.cpp
index d1cb83a2f00ad8fc9739c8f1f3cd2b76be3c62df..f3ed5cca3510ab1ee35768dda5875382e35b6e04 100644
--- a/third_party/WebKit/Source/modules/websockets/DOMWebSocket.cpp
+++ b/third_party/WebKit/Source/modules/websockets/DOMWebSocket.cpp
@@ -49,6 +49,7 @@
#include "core/inspector/ConsoleMessage.h"
#include "core/inspector/ScriptCallStack.h"
#include "modules/websockets/CloseEvent.h"
+#include "platform/Histogram.h"
#include "platform/Logging.h"
#include "platform/blob/BlobData.h"
#include "platform/heap/Handle.h"
@@ -392,7 +393,9 @@ void DOMWebSocket::send(const String& message, ExceptionState& exceptionState)
updateBufferedAmountAfterClose(encodedMessage.length());
return;
}
- Platform::current()->histogramEnumeration("WebCore.WebSocket.SendType", WebSocketSendTypeString, WebSocketSendTypeMax);
+
+ recordSendTypeHistogram(WebSocketSendTypeString);
+
ASSERT(m_channel);
m_bufferedAmount += encodedMessage.length();
m_channel->send(encodedMessage);
@@ -410,7 +413,8 @@ void DOMWebSocket::send(DOMArrayBuffer* binaryData, ExceptionState& exceptionSta
updateBufferedAmountAfterClose(binaryData->byteLength());
return;
}
- Platform::current()->histogramEnumeration("WebCore.WebSocket.SendType", WebSocketSendTypeArrayBuffer, WebSocketSendTypeMax);
+ recordSendTypeHistogram(WebSocketSendTypeArrayBuffer);
+
ASSERT(m_channel);
m_bufferedAmount += binaryData->byteLength();
m_channel->send(*binaryData, 0, binaryData->byteLength());
@@ -428,7 +432,8 @@ void DOMWebSocket::send(DOMArrayBufferView* arrayBufferView, ExceptionState& exc
updateBufferedAmountAfterClose(arrayBufferView->byteLength());
return;
}
- Platform::current()->histogramEnumeration("WebCore.WebSocket.SendType", WebSocketSendTypeArrayBufferView, WebSocketSendTypeMax);
+ recordSendTypeHistogram(WebSocketSendTypeArrayBufferView);
+
ASSERT(m_channel);
m_bufferedAmount += arrayBufferView->byteLength();
m_channel->send(*arrayBufferView->buffer(), arrayBufferView->byteOffset(), arrayBufferView->byteLength());
@@ -446,7 +451,8 @@ void DOMWebSocket::send(Blob* binaryData, ExceptionState& exceptionState)
updateBufferedAmountAfterClose(binaryData->size());
return;
}
- Platform::current()->histogramEnumeration("WebCore.WebSocket.SendType", WebSocketSendTypeBlob, WebSocketSendTypeMax);
+ recordSendTypeHistogram(WebSocketSendTypeBlob);
+
unsigned long long size = binaryData->size();
m_bufferedAmount += size;
ASSERT(m_channel);
@@ -625,7 +631,8 @@ void DOMWebSocket::didReceiveTextMessage(const String& msg)
WTF_LOG(Network, "WebSocket %p didReceiveTextMessage() Text message '%s'", this, msg.utf8().data());
if (m_state != OPEN)
return;
- Platform::current()->histogramEnumeration("WebCore.WebSocket.ReceiveType", WebSocketReceiveTypeString, WebSocketReceiveTypeMax);
+ recordReceiveTypeHistogram(WebSocketReceiveTypeString);
+
m_eventQueue->dispatch(MessageEvent::create(msg, SecurityOrigin::create(m_url)->toString()));
}
@@ -640,14 +647,14 @@ void DOMWebSocket::didReceiveBinaryMessage(PassOwnPtr<Vector<char>> binaryData)
OwnPtr<BlobData> blobData = BlobData::create();
blobData->appendData(rawData.release(), 0, BlobDataItem::toEndOfFile);
Blob* blob = Blob::create(BlobDataHandle::create(blobData.release(), size));
- Platform::current()->histogramEnumeration("WebCore.WebSocket.ReceiveType", WebSocketReceiveTypeBlob, WebSocketReceiveTypeMax);
+ recordReceiveTypeHistogram(WebSocketReceiveTypeBlob);
m_eventQueue->dispatch(MessageEvent::create(blob, SecurityOrigin::create(m_url)->toString()));
break;
}
case BinaryTypeArrayBuffer:
RefPtr<DOMArrayBuffer> arrayBuffer = DOMArrayBuffer::create(binaryData->data(), binaryData->size());
- Platform::current()->histogramEnumeration("WebCore.WebSocket.ReceiveType", WebSocketReceiveTypeArrayBuffer, WebSocketReceiveTypeMax);
+ recordReceiveTypeHistogram(WebSocketReceiveTypeArrayBuffer);
m_eventQueue->dispatch(MessageEvent::create(arrayBuffer.release(), SecurityOrigin::create(m_url)->toString()));
break;
}
@@ -692,6 +699,18 @@ void DOMWebSocket::didClose(ClosingHandshakeCompletionStatus closingHandshakeCom
releaseChannel();
}
+void DOMWebSocket::recordSendTypeHistogram(WebSocketSendType type)
+{
+ DEFINE_THREAD_SAFE_STATIC_LOCAL(EnumerationHistogram, sendTypeHistogram, new EnumerationHistogram("WebCore.WebSocket.SendType", WebSocketSendTypeMax));
+ sendTypeHistogram.count(type);
+}
+
+void DOMWebSocket::recordReceiveTypeHistogram(WebSocketReceiveType type)
+{
+ DEFINE_THREAD_SAFE_STATIC_LOCAL(EnumerationHistogram, receiveTypeHistogram, new EnumerationHistogram("WebCore.WebSocket.ReceiveType", WebSocketReceiveTypeMax));
+ receiveTypeHistogram.count(type);
+}
+
DEFINE_TRACE(DOMWebSocket)
{
visitor->trace(m_channel);

Powered by Google App Engine
This is Rietveld 408576698