[Webkit-unassigned] [Bug 72016] [Chromium] [WebSocket] export WebSocketChannel interface for plugins

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Nov 15 05:44:20 PST 2011


https://bugs.webkit.org/show_bug.cgi?id=72016





--- Comment #8 from Yuta Kitamura <yutak at chromium.org>  2011-11-15 05:44:20 PST ---
(From update of attachment 114904)
View in context: https://bugs.webkit.org/attachment.cgi?id=114904&action=review

>>> Source/WebCore/websockets/WebSocketChannel.h:72
>>> +    virtual bool send(const char* data, int length);
>> 
>> This overload might be confusing because it isn't clear whether this version sends a text message or a binary message; maybe we should rename the first overload to "sendText" and the other three to "sendBinary".
> 
> I see, but I'd like to split these refactoring into another change.
> Because that change will affect other WebSocket related files.
> It make the purpose of this change vague.

OK.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list