summaryrefslogtreecommitdiffstats
path: root/net/websockets/websocket_job.cc
Commit message (Collapse)AuthorAgeFilesLines
* WebSocket over SPDY: handshake support for both of SPDY/2 and SPDY/3toyoshim@chromium.org2012-08-141-2/+6
| | | | | | | | | | | | | If negotiated protocol is SPDY/3 or later, well defined name of SYN_STREAM and SYN_REPLY must be colon prefixed name. BUG=42320 TEST=net_uinttests Review URL: https://chromiumcodereview.appspot.com/10843050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151453 0039d316-1c4b-4281-b951-d872f2087c98
* Remove linked_ptr from SpdyStream.rch@chromium.org2012-07-181-2/+2
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10784055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147257 0039d316-1c4b-4281-b951-d872f2087c98
* NetLogEventParameter to Callback refactoring 8.mmenke@chromium.org2012-06-141-4/+3
| | | | | | | | | | | | | Get rid of all uses of NetLogEventParameters in: net/socket_stream, net/websockets, net/base. R=eroman@chromium.org BUG=126243 Review URL: https://chromiumcodereview.appspot.com/10548028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142202 0039d316-1c4b-4281-b951-d872f2087c98
* Move NextProto enum to a new file net/socket/next_proto.hrch@chromium.org2012-03-311-2/+1
| | | | | | | | | BUG=120890 Review URL: http://codereview.chromium.org/9959033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130038 0039d316-1c4b-4281-b951-d872f2087c98
* Move all objects from the spdy:: namespace to net::rch@chromium.org2012-03-221-2/+2
| | | | | | Review URL: http://codereview.chromium.org/9815028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@128310 0039d316-1c4b-4281-b951-d872f2087c98
* Update net/base/cookie_*.h includes to net/cookies/cookie_*.h .erikwright@chromium.org2012-03-151-1/+1
| | | | | | | | | | | | Once complete, forwarding headers will be removed from net/base/ . TEST=none R=willchan Review URL: http://codereview.chromium.org/9703062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126999 0039d316-1c4b-4281-b951-d872f2087c98
* OnSSLCertificateError delegate chain back to SocketStreamDispatcherHosttoyoshim@chromium.org2012-03-091-0/+6
| | | | | | | | | | | | | SSL cert errors must be handled by SSLManager. This change provide delegate chain back to SocketStreamDispatcherHost to handle the error by SSLManager here. BUG=53836 TEST=run existing unit tests because this change is a kind of refactoring Review URL: http://codereview.chromium.org/9454011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125882 0039d316-1c4b-4281-b951-d872f2087c98
* Remove WebSocketFrameHandler.yutak@chromium.org2012-02-151-67/+44
| | | | | | | | | | | | | | | | | WebSocketFrameHandler supports only old frame format (hybi-00) and its parser functionality has never been used (i.e. used with unbuffered mode) for a long time. This change removes WebSocketFrameHandler and lets WebSocketJob handle incoming and outgoing frame data directly. BUG=none TEST=none Review URL: http://codereview.chromium.org/9112003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122074 0039d316-1c4b-4281-b951-d872f2087c98
* Initialize protocol_version to Unknown.rtenneti@chromium.org2011-12-201-1/+2
| | | | | | | | | | R=mmenke TEST=network unittests BUG=108134 Review URL: http://codereview.chromium.org/8956042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115196 0039d316-1c4b-4281-b951-d872f2087c98
* Added protocol_version negotiated with the server.rtenneti@google.com2011-12-201-1/+3
| | | | | | | | | | | | | protocol_version will be printed by the LoadTimes extension. R=willchan BUG=108134 TEST=network unittests Review URL: http://codereview.chromium.org/8914005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115145 0039d316-1c4b-4281-b951-d872f2087c98
* Replace OldCompletionCallback in SocketStream::Delegate.yutak@chromium.org2011-12-011-14/+13
| | | | | | | | | | BUG=105683 TEST=none Review URL: http://codereview.chromium.org/8747013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112414 0039d316-1c4b-4281-b951-d872f2087c98
* Convert ScopedRunnableMethodFactory in WebSocketJob.yutak@chromium.org2011-11-291-5/+8
| | | | | | | | | | BUG=105683 TEST=none Review URL: http://codereview.chromium.org/8728025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111915 0039d316-1c4b-4281-b951-d872f2087c98
* Allow linker initialization of lazy instancejoth@chromium.org2011-11-151-2/+2
| | | | | | | | | | | | | | Using the initializer list construct = {0} allows the object to be linker initialized. Modify the LazyInstance class design to make it a pod aggregate type that can be linker initialized this way. Also combines the instance and state members, in line with the Singleton<> class design. Introduces a new LAZY_INSTANCE_INITIALIZER macro specifically for using to init all lazy instances + modify all existing callsites to use it. (Old code would no longer compile) BUG=94925 TEST=existing tests pass. http://build.chromium.org/f/chromium/perf/linux-release/sizes/report.html?history=150&header=chrome-si&graph=chrome-si&rev=-1 should step downward. TBR=jam@chromium.org,rvargas@chromium.org,darin@chromium.org,ben@chromium.org,apatrick@chromium.org,akalin@chromium.org Review URL: http://codereview.chromium.org/8491043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110076 0039d316-1c4b-4281-b951-d872f2087c98
* Use AuthCredentials throughout the network stack instead of username/password.cbentzel@chromium.org2011-10-281-4/+2
| | | | | | | | This is a refactor only - no behavior change should happen. Review URL: http://codereview.chromium.org/8340026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107766 0039d316-1c4b-4281-b951-d872f2087c98
* Begin CompletionCallback switchover.willchan@chromium.org2011-10-011-2/+2
| | | | | | | | | | | | | Rename CompletionCallback to OldCompletionCallback in preparation for introducing a new CompletionCallback based on base::Callback. Also renames other CompletionCallback types like CancelableCompletionCallback and TestCompletionCallback and CompletionCallbackImpl. All using sed with s/CompletionCallback/OldCompletionCallback/g. BUG=98719 TEST=none Review URL: http://codereview.chromium.org/8070013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103650 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash bug at net::WebSocketJob::OnSentData.ukai@chromium.org2011-09-201-2/+5
| | | | | | | | | | | | | | | | | According to crash dump, I suspect it is crashed at current_buffer_->DidConsume() where current_buffer_ is NULL. If current_buffer_ is NULL, WebSocketJob doesn't think there are writing data and nothing to do, so this CL changes to just return from net::WebSocketJob::OnSentData in this case. Also checks if amount_sent is negative or zero and return if it is so. BUG=91591 TEST=none Review URL: http://codereview.chromium.org/7923004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101942 0039d316-1c4b-4281-b951-d872f2087c98
* Apply the asynchronous CookieMonster API to WebsocketJob.ycxiao@chromium.org2011-08-011-33/+51
| | | | | | | | | BUG=68657 TEST=XXXX Review URL: http://codereview.chromium.org/7278026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94937 0039d316-1c4b-4281-b951-d872f2087c98
* Use ScopedRunnableMethodFactory in WebSocketJobukai@chromium.org2011-07-221-5/+11
| | | | | | | | | | | Don't post SendPending if it is already posted. BUG=89795 TEST=none Review URL: http://codereview.chromium.org/7488007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93599 0039d316-1c4b-4281-b951-d872f2087c98
* Check socket_ is not NULL.ukai@chromium.org2011-07-221-1/+5
| | | | | | | | | | | Also, if state_ is already CLOSED, Close() does nothing. BUG=89795 TEST=none Review URL: http://codereview.chromium.org/7488002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93556 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce a new connection test which verify WebSocket and SPDY connection ↵toyoshim@chromium.org2011-07-221-0/+4
| | | | | | | | | | | sequence after throttling work. BUG=NONE TEST=net_unittest --gtest_filter=WebSocketJobTest.Throttling\* Review URL: http://codereview.chromium.org/7331007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93555 0039d316-1c4b-4281-b951-d872f2087c98
* Add WebSocket over SPDY experimental implementation.toyoshim@chromium.org2011-07-111-38/+144
| | | | | | | | | | | | | | | | (Re-land with memory leak fix) - Realize WebSocketJob's internal protocol switch to SPDY using SpdyWebSocketStream - Add simple test to verify connection over SPDY BUG=42320 TEST=net_unittests --gtest_filter=WebSocketJobTest\* Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=91997 Review URL: http://codereview.chromium.org/7185032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92017 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 91997 - Add WebSocket over SPDY experimental implementation.toyoshim@chromium.org2011-07-111-144/+38
| | | | | | | | | | | | | | | - Realize WebSocketJob's internal protocol switch to SPDY using SpdyWebSocketStream - Add simple test to verify connection over SPDY BUG=42320 TEST=net_unittests --gtest_filter=WebSocketJobTest\* Review URL: http://codereview.chromium.org/7185032 TBR=toyoshim@chromium.org Review URL: http://codereview.chromium.org/7333006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92003 0039d316-1c4b-4281-b951-d872f2087c98
* Add WebSocket over SPDY experimental implementation.toyoshim@chromium.org2011-07-111-38/+144
| | | | | | | | | | | | - Realize WebSocketJob's internal protocol switch to SPDY using SpdyWebSocketStream - Add simple test to verify connection over SPDY BUG=42320 TEST=net_unittests --gtest_filter=WebSocketJobTest\* Review URL: http://codereview.chromium.org/7185032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91997 0039d316-1c4b-4281-b951-d872f2087c98
* WebSocket over SPDY: protocol switchtoyoshim@chromium.org2011-06-161-12/+57
| | | | | | | | | | | | | This change enable to switch protocol from WebSocket to SPDY. SpdyWebsocketStream class is not included, then SPDY connection always fail now. BUG=42320 TEST=net_unittests --gtest_filter=WebSocket\* Review URL: http://codereview.chromium.org/7062043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89318 0039d316-1c4b-4281-b951-d872f2087c98
* Add invalid data send check on CONNECTING statetoyoshim@chromium.org2011-06-021-0/+4
| | | | | | | | | | | | | | Handshake data should be sent at once. WebSocketJob should reject any other subsequent data sending until the handshake is done and connection is opened. BUG=84422 TEST=net_unittest --gtest_filter=WebSocket\* Review URL: http://codereview.chromium.org/7075027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87597 0039d316-1c4b-4281-b951-d872f2087c98
* Add a websocket-over-spdy flag. This change is a part of WebSocket over SPDYtoyoshim@chromium.org2011-05-261-0/+7
| | | | | | | | | | | support in progress. BUG=42320 TEST=N/A Review URL: http://codereview.chromium.org/7047003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@86812 0039d316-1c4b-4281-b951-d872f2087c98
* Get rid of net::CookiePolicy, now that all code that uses it (except ↵jam@chromium.org2011-05-121-34/+12
| | | | | | | | | WebSocketJob, which appears to be unused and which I updated in this cl) is switched over to use ContentBrowserClient. BUG=76793 Review URL: http://codereview.chromium.org/6973011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85136 0039d316-1c4b-4281-b951-d872f2087c98
* Miscelaneous cleanups to AddressList to make it harder to mis-use.eroman@chromium.org2011-05-121-1/+1
| | | | | | | | | | | | | | - Removed all destructive non-const member functions -- these were dangerous since if you called them without first making a copy of the AddressList, it could mutate earlier copies. - Made AddressList::Data::head const, so new code added to AddressList cannot inadvertently introduce such dangerous mutations (won't compile). - Moved the non-trivial constructors and assign methods into factory methods (for added readability) - Removed the bool parameter from Copy (for added readability). Review URL: http://codereview.chromium.org/6880302 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85090 0039d316-1c4b-4281-b951-d872f2087c98
* Accept new WebSocket handshake format (hybi-04 and later).yutak@chromium.org2011-05-061-0/+2
| | | | | | | | | | | | | | | To update WebSocket protocol implementation in WebKit, WebSocketRequestHandshakeHandler and WebSocketResponseHandler need to be able to understand both old handshake (hybi-03 and prior, including hixie-76) and new handshake (hybi-04 and later). BUG=64470 TEST=net_unittests --gtest_filter="WebSocketHandshake*HandlerTest.*" Review URL: http://codereview.chromium.org/6823075 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84427 0039d316-1c4b-4281-b951-d872f2087c98
* Remove async functionality from net::CookiePolicy.willchan@chromium.org2011-04-061-19/+5
| | | | | | | | | | | | This lets us stop refcounting URLRequestHttpJob. BUG=none TEST=none Review URL: http://codereview.chromium.org/6749044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80660 0039d316-1c4b-4281-b951-d872f2087c98
* Change other usages of .size() to .empty() when applicable.erg@google.com2011-03-041-1/+1
| | | | | | | | | BUG=carnitas TEST=compiles Review URL: http://codereview.chromium.org/6609008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76962 0039d316-1c4b-4281-b951-d872f2087c98
* Make members of Singleton<T> private and only visible to the singleton type. ↵satish@chromium.org2010-12-141-3/+6
| | | | | | | | | | | | | | | This enforces that the Singleton<T> pattern can only be used within classes which want singleton-ness. As part of this CL I have also fixed up files which got missed in my previous CLs to use a GetInstance() method and use Singleton<T> from the source file. There are a small number of places where I have also switched to LazyInstance as that was more appropriate for types used in a single source file. BUG=65298 TEST=all existing tests should continue to pass. Review URL: http://codereview.chromium.org/5682008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69107 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 68932 - Make members of Singleton<T> private and only visible to the ↵satish@chromium.org2010-12-111-6/+3
| | | | | | | | | | | | | | | | | | singleton type. This enforces that the Singleton<T> pattern can only be used within classes which want singleton-ness. As part of this CL I have also fixed up files which got missed in my previous CLs to use a GetInstance() method and use Singleton<T> from the source file. There are a small number of places where I have also switched to LazyInstance as that was more appropriate for types used in a single source file. BUG=65298 TEST=all existing tests should continue to pass. Review URL: http://codereview.chromium.org/5682008 TBR=satish@chromium.org Review URL: http://codereview.chromium.org/5721005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68936 0039d316-1c4b-4281-b951-d872f2087c98
* Make members of Singleton<T> private and only visible to the singleton type. ↵satish@chromium.org2010-12-111-3/+6
| | | | | | | | | | | | | | | This enforces that the Singleton<T> pattern can only be used within classes which want singleton-ness. As part of this CL I have also fixed up files which got missed in my previous CLs to use a GetInstance() method and use Singleton<T> from the source file. There are a small number of places where I have also switched to LazyInstance as that was more appropriate for types used in a single source file. BUG=65298 TEST=all existing tests should continue to pass. Review URL: http://codereview.chromium.org/5682008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68932 0039d316-1c4b-4281-b951-d872f2087c98
* Add a new GetInstance() method for singleton classes under chrome/service ↵satish@chromium.org2010-12-091-7/+8
| | | | | | | | | | | | | | | | | | | | and /net. This is a small step towards making all singleton classes use the Singleton<T> pattern within their code and not expect the callers to know about it. This CL includes files under chrome/service and /net with related files elsewhere. Suggested files to focus for reviewers: - @sanjeevr for chrome/common and chrome/service - @ukai for net/websockets - @agl for rest of net BUG=65298 TEST=all existing tests should continue to pass. Review URL: http://codereview.chromium.org/5634005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68722 0039d316-1c4b-4281-b951-d872f2087c98
* Convert implicit scoped_refptr constructor calls to explicit ones, part 2thakis@chromium.org2010-11-021-3/+4
| | | | | | | | | | | | This CL was created automatically by this clang rewriter: http://codereview.appspot.com/2826041 I then did quite a bit of manual editing to fix style issues. BUG=28083 TEST=None Review URL: http://codereview.chromium.org/4291001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64798 0039d316-1c4b-4281-b951-d872f2087c98
* Add WebSocket handshake message in net_logukai@chromium.org2010-08-121-0/+9
| | | | | | | | | BUG=none TEST=websocket handshake message shown in chrome://net-internals/#requests Review URL: http://codereview.chromium.org/3076043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55845 0039d316-1c4b-4281-b951-d872f2087c98
* Net: Convert username and password to string16.thestig@chromium.org2010-07-291-2/+2
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3040016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54101 0039d316-1c4b-4281-b951-d872f2087c98
* Ignore OnConnected event if the job state is already CLOSED.ukai@chromium.org2010-06-291-0/+2
| | | | | | | | | | | There is a case while SocketStream is connecting, WebSocketJob is closed and SocketStream notifies it is connected. In this case, we should just ignore OnConnected event. BUG=47818 TEST=websocket/tests/url-with-credential.html Review URL: http://codereview.chromium.org/2823034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51108 0039d316-1c4b-4281-b951-d872f2087c98
* Check state in SocketStream delegate methods.ukai@chromium.org2010-06-171-0/+11
| | | | | | | | | | | I suspect flaky failure in reload-crash.html was OnSentData is called when state_ has been changed to CLOSED while sending handshake message. BUG=45720 TEST=none Review URL: http://codereview.chromium.org/2845012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50087 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor WebSocket handshake.ukai@chromium.org2010-06-031-189/+51
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/2452001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48816 0039d316-1c4b-4281-b951-d872f2087c98
* Fix regression of websocket/tests/simple-stress.htmlukai@chromium.org2010-06-011-4/+16
| | | | | | | | | | | | | | | | | | Since stress-test.html tries to send 256K message, but we'll set 32K for max_pending_send_allowed in SocketStream. Thus, WebCore::SocketStreamHandle won't send full websocket frame for such large message since WebSocketJob doesn't notify OnSentData at all. Don't parse websocket frame for sending to avoid this issue for now. Also disable buffering and parsing websocket frame for receiving to fix performance regression. BUG=32018 TEST=LayoutTests/websocket/tests/stress-test.html passes Review URL: http://codereview.chromium.org/2453002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48629 0039d316-1c4b-4281-b951-d872f2087c98
* Factor out WebSocket frame handler out of WebSocketJob.ukai@chromium.org2010-06-011-35/+139
| | | | | | | | | | | | | WebSocket frame handler might be used to count number of frames, variance of type or length of frame. It might be also used to compress/decompress message in websocket frame. BUG=none TEST=none Review URL: http://codereview.chromium.org/2078002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48615 0039d316-1c4b-4281-b951-d872f2087c98
* Fix WebSocketJob to pass /key_3/ after handshake request header.ukai@chromium.org2010-03-301-5/+9
| | | | | | | | | BUG=none TEST=WebSocketJob::SimpleHandshakeDraft76 passes Review URL: http://codereview.chromium.org/1256001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43054 0039d316-1c4b-4281-b951-d872f2087c98
* Use FilterHeaders() instead of HttpUtil::StripHeaders().ukai@chromium.org2010-03-251-5/+57
| | | | | | | | | | | | | | | | We want to pass original response headers to WebKit, but just dropping set cookie headers. HttpUtil::StripHeaders() strips malformed headers too, so handshake in websocket/tests/long-invalid-header.html will success unexpectedly. Use FilterHeaders() to preserve malformed headers in response headers. BUG=39062 TEST=handshake fails when malformed header is received. Review URL: http://codereview.chromium.org/1254001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42588 0039d316-1c4b-4281-b951-d872f2087c98
* Try to fix flaky websocket tests again.ukai@chromium.org2010-03-231-30/+29
| | | | | | | | | | | | | | | | | | | | | Some websoket layout tests became flaky from r41818. This is because it adds websocket throttling in WebSocketJob. Make sure Close() will call OnClose() even if it is waiting resolving or waiting in throttling queue, so that WebSocketJob is removed from throttling queue and wake up next WebSocketJob. r42074 was reverted because failure in SocketStreamMetrics::OnClose on Linux Builder (Views dbg). Ignore UMA record if the connection has not been established. TBR=tyoshino BUG=38397 TEST=layout tests websocket/tests passes Review URL: http://codereview.chromium.org/1134004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42320 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 42074 - Try to fix flaky websocket tests.amit@chromium.org2010-03-191-25/+30
| | | | | | | | | | | | | | | | | | | Some websoket layout tests became flaky from r41818. This is because it adds websocket throttling in WebSocketJob. Make sure Close() will call OnClose() even if it is waiting resolving or waiting in throttling queue, so that WebSocketJob is removed from throttling queue and wake up next WebSocketJob. BUG=38397 TEST=layout tests websocket/tests passes Review URL: http://codereview.chromium.org/1096001 TBR=ukai@chromium.org Review URL: http://codereview.chromium.org/1120004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42078 0039d316-1c4b-4281-b951-d872f2087c98
* Try to fix flaky websocket tests.ukai@chromium.org2010-03-191-30/+25
| | | | | | | | | | | | | | | | Some websoket layout tests became flaky from r41818. This is because it adds websocket throttling in WebSocketJob. Make sure Close() will call OnClose() even if it is waiting resolving or waiting in throttling queue, so that WebSocketJob is removed from throttling queue and wake up next WebSocketJob. BUG=38397 TEST=layout tests websocket/tests passes Review URL: http://codereview.chromium.org/1096001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42074 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor WebSocket throttling feature.ukai@chromium.org2010-03-171-0/+73
| | | | | | | | | Protocol specific handling should be done in SocketStreamJob subclasss, so websocket throttling should be handled in WebSocketJob. Review URL: http://codereview.chromium.org/669157 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41818 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash in ExtensionApiTest.FLAKY_WebSocketukai@chromium.org2010-03-011-1/+2
| | | | | | | | | | TBR=darin BUG=none TEST=browser_tests ExtensionApiTest.FLAKY_WebSocket doesn't crash Review URL: http://codereview.chromium.org/660259 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40253 0039d316-1c4b-4281-b951-d872f2087c98