Commit 5fb00aa4 authored by Yoichi Osato's avatar Yoichi Osato Committed by Commit Bot

[WebSocket][nit] Remove an invalid TODO

That's because we've decided not to unify op code and data on a datapipe.

Change-Id: I1d9d83d10108552c6ba04b240ee71c2298d7663e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903226Reviewed-by: default avatarAdam Rice <ricea@chromium.org>
Reviewed-by: default avatarYutaka Hirano <yhirano@chromium.org>
Commit-Queue: Yoichi Osato <yoichio@chromium.org>
Cr-Commit-Position: refs/heads/master@{#713709}
parent 3a4a099f
......@@ -205,7 +205,6 @@ void WebSocket::WebSocketEventHandler::OnDataFrame(
DVLOG(3) << "WebSocketEventHandler::OnDataFrame @"
<< reinterpret_cast<void*>(this) << " fin=" << fin
<< " type=" << type << " data is " << payload.size() << " bytes";
// TODO(yoichio): Merge OnDataFrame mojo channel into data pipe.
impl_->client_->OnDataFrame(fin, OpCodeToMessageType(type), payload.size());
if (payload.size() > 0) {
impl_->pending_data_frames_.push(payload);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment