Remove cookieRequestHeaderFieldValue() and setCookie() call from WebSocketHandshake

Remove cookieRequestHeaderFieldValue() and setCookie() call from WebSocketHandshake

These calls were for non-Chromium port of WebKit to get/set cookies on
WebSocket opening handshake. In Blink, they're no longer necessary.
Chromium intercepts handshakes in the browser process and add/remove
cookies. Any cookie header added by Blink is ignored. Set-cookies are
stripped before forwarded to Blink.

BUG=none

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170560

[yhirano, 2014-03-26 07:08:32]

https://codereview.chromium.org/209823002/diff/20001/Source/modules/websocket...
File Source/modules/websockets/MainThreadWebSocketChannel.cpp (left):

https://codereview.chromium.org/209823002/diff/20001/Source/modules/websocket...
Source/modules/websockets/MainThreadWebSocketChannel.cpp:41: #include
"core/loader/CookieJar.h"
Is it necessary?

https://codereview.chromium.org/209823002/diff/20001/Source/modules/websocket...
File Source/modules/websockets/WebSocketHandshake.cpp (right):

https://codereview.chromium.org/209823002/diff/20001/Source/modules/websocket...
Source/modules/websockets/WebSocketHandshake.cpp:212: KURL url =
httpURLForAuthenticationAndCookies();
url is not used.

https://codereview.chromium.org/209823002/diff/20001/Source/modules/websocket...
Source/modules/websockets/WebSocketHandshake.cpp:260: String cookie =
cookieRequestHeaderFieldValue(m_document, url);
Can we remove it as well?

[tyoshino (SeeGerritForStatus), 2014-04-01 11:11:22]

https://codereview.chromium.org/209823002/diff/20001/Source/modules/websocket...
File Source/modules/websockets/MainThreadWebSocketChannel.cpp (left):

https://codereview.chromium.org/209823002/diff/20001/Source/modules/websocket...
Source/modules/websockets/MainThreadWebSocketChannel.cpp:41: #include
"core/loader/CookieJar.h"
On 2014/03/26 07:08:32, yhirano wrote:
> Is it necessary?

Removed.

https://codereview.chromium.org/209823002/diff/20001/Source/modules/websocket...
File Source/modules/websockets/WebSocketHandshake.cpp (right):

https://codereview.chromium.org/209823002/diff/20001/Source/modules/websocket...
Source/modules/websockets/WebSocketHandshake.cpp:212: KURL url =
httpURLForAuthenticationAndCookies();
On 2014/03/26 07:08:32, yhirano wrote:
> url is not used.

Removed.

https://codereview.chromium.org/209823002/diff/20001/Source/modules/websocket...
Source/modules/websockets/WebSocketHandshake.cpp:260: String cookie =
cookieRequestHeaderFieldValue(m_document, url);
On 2014/03/26 07:08:32, yhirano wrote:
> Can we remove it as well?

This is for inspector. It's still useful to get and append cookie information to
|request| here.

[yhirano, 2014-04-01 11:21:16]

lgtm

[tyoshino (SeeGerritForStatus), 2014-04-01 11:59:38]

The CQ bit was checked by tyoshino@chromium.org

[commit-bot: I haz the power, 2014-04-01 11:59:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tyoshino@chromium.org/209823002/60001

[commit-bot: I haz the power, 2014-04-01 13:13:06]

Change committed as 170560