summaryrefslogtreecommitdiffstats
path: root/chrome_frame/test/test_server.cc
Commit message (Collapse)AuthorAgeFilesLines
* Refactor TCPListenSocket.pliard@chromium.org2012-05-161-14/+20
| | | | | | | | | | | | | | | This is part of Chrome for Android upstreaming. This CL adds a common base class, StreamListenSocket, providing a default implementation inherited by TCPListenSocket and the upcoming UnixDomainSocket. That lets us share the common code used by both TCPListenSocket and UnixDomainSocket. This also removes the recently introduced ListenSocket class which is unnecessary now we have StreamListenSocket. TEST=net_unittests Review URL: https://chromiumcodereview.appspot.com/10161005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137387 0039d316-1c4b-4281-b951-d872f2087c98
* Upstream changes making ListenSocket an abstract class.pliard@chromium.org2012-04-231-2/+4
| | | | | | | | | | | | | | | | | | | This is part of Chrome for Android upstreaming. This CL makes ListenSocket an abstract class instead of a concrete class implementing a TCP Listen Socket. This abstraction will be used later to make HttpServer seamlessly operate on TCP sockets and Unix domain sockets (will be upstreamed in a separate CL). The TCP Listen socket implementation, previously in listen_socket.{cc,h}, is now in tcp_listen_socket.{cc,h}. TEST=net_unittests Review URL: http://codereview.chromium.org/10108015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133480 0039d316-1c4b-4281-b951-d872f2087c98
* Update uses of TimeDelta in chrome_frame/*. Also update a bit of TimeDelta ↵tedvessenes@gmail.com2012-04-231-2/+2
| | | | | | | | | | | | | | | use in webkit npapi code. Adding robertshield for his thoughts on this. R=ananta@chromium.org BUG=108171 Review URL: http://codereview.chromium.org/10019015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133395 0039d316-1c4b-4281-b951-d872f2087c98
* Close accepted server sockets early and often.grt@chromium.org2012-04-051-1/+1
| | | | | | | | | | | In one case, sockets weren't being closed until the listen socket was destroyed (later than optimal), and in another they weren't being closed at all process-wide. BUG=96449 TEST=hope that chrome_frame_tests.exe --gtest_filter=ChromeFrameTestWithWebServer.* becomes less flaky. Review URL: https://chromiumcodereview.appspot.com/10007006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130927 0039d316-1c4b-4281-b951-d872f2087c98
* Fix test crashes caused by toxic use of ListenSocket.grt@chromium.org2012-03-281-6/+22
| | | | | | | | | | | | ListenSocket doesn't like it if delegates delete instances from within DidFoo() callbacks. Instead of releasing references to sockets from within ConfigurableConnection::SendWithOptions (which is invoked indirectly by way of HTTPTestServer::DidRead), post a task to do so. Handle the fact that there may be connections in an HTTPTestServer instance for which the underlying socket_ is NULL by harvesting them in HTTPTestServer::FindConnection(). As a result of all of this, it's possible that DidRead or DidClose may be called with a ListenSocket* that is no longer present in the server's connection list. This isn't an error condition, so simply handle it gracefully in Did{Read,Close}. BUG=120458 TEST=nothing special. chrome_frame_tests.exe should stop crashing so much. Review URL: http://codereview.chromium.org/9863048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129319 0039d316-1c4b-4281-b951-d872f2087c98
* More non-debug logging in Chrome Frame tests.grt@chromium.org2012-03-241-3/+3
| | | | | | | | | | | | | There are only good reasons to not use DLOG and DVLOG in test code. Also, avoid a potential race by setting up ChromeFrameTestWithWebServer expectations before launching the browser. BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/9838058 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@128638 0039d316-1c4b-4281-b951-d872f2087c98
* Log WebBrowser event method names rather than DISPID integer values for more ↵grt@chromium.org2012-01-231-1/+3
| | | | | | | | | | | comprehensible logs, and log the method and path received by the test web server when it handles a request. Such logging is no longer limited to debug builds. BUG=none TEST=run any GCF test with --v=1 and watch the events fly by Review URL: http://codereview.chromium.org/9272004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118694 0039d316-1c4b-4281-b951-d872f2087c98
* Standardize StringToInt{,64} interface.tedvessenes@gmail.com2011-12-191-2/+3
| | | | | | | | | | | | | | | | | | | | | | | These changes address issue #106655. All variants of StringToInt have been converted to use the StringPiece class. One instance of conversion, in chrome/browser/history/text_database.cc, required copying an underlying string. This is because the string type in question could use 8 or 16 bit characters depending on the OS type, and because StringPiece is not implemented as a template, the code cannot specify whether to create a StringPiece or StringPiece16. This should be remedied in a future CL. R=erikwright@chromium.org BUG=106655 TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114929 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114944 Review URL: http://codereview.chromium.org/8921006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114993 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 114944 - Standardize StringToInt{,64} interface.battre@chromium.org2011-12-181-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | Revert due to compile breakage on ChromeOS. These changes address issue #106655. All variants of StringToInt have been converted to use the StringPiece class. One instance of conversion, in chrome/browser/history/text_database.cc, required copying an underlying string. This is because the string type in question could use 8 or 16 bit characters depending on the OS type, and because StringPiece is not implemented as a template, the code cannot specify whether to create a StringPiece or StringPiece16. This should be remedied in a future CL. R=erikwright@chromium.org BUG=106655 TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114929 Review URL: http://codereview.chromium.org/8921006 TBR=tedvessenes@gmail.com Review URL: http://codereview.chromium.org/8990002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114945 0039d316-1c4b-4281-b951-d872f2087c98
* Standardize StringToInt{,64} interface.tedvessenes@gmail.com2011-12-181-2/+3
| | | | | | | | | | | | | | | | | | | | | These changes address issue #106655. All variants of StringToInt have been converted to use the StringPiece class. One instance of conversion, in chrome/browser/history/text_database.cc, required copying an underlying string. This is because the string type in question could use 8 or 16 bit characters depending on the OS type, and because StringPiece is not implemented as a template, the code cannot specify whether to create a StringPiece or StringPiece16. This should be remedied in a future CL. R=erikwright@chromium.org BUG=106655 TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114929 Review URL: http://codereview.chromium.org/8921006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114944 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 114929 - Standardize StringToInt{,64} interface.rsleevi@chromium.org2011-12-171-3/+2
| | | | | | | | | | | | | | | | | | | | | | These changes address issue #106655. All variants of StringToInt have been converted to use the StringPiece class. One instance of conversion, in chrome/browser/history/text_database.cc, required copying an underlying string. This is because the string type in question could use 8 or 16 bit characters depending on the OS type, and because StringPiece is not implemented as a template, the code cannot specify whether to create a StringPiece or StringPiece16. This should be remedied in a future CL. R=erikwright@chromium.org BUG=106655 TEST= Review URL: http://codereview.chromium.org/8921006 TBR=tedvessenes@gmail.com Review URL: http://codereview.chromium.org/8984007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114930 0039d316-1c4b-4281-b951-d872f2087c98
* Standardize StringToInt{,64} interface.tedvessenes@gmail.com2011-12-171-2/+3
| | | | | | | | | | | | | | | | | | | These changes address issue #106655. All variants of StringToInt have been converted to use the StringPiece class. One instance of conversion, in chrome/browser/history/text_database.cc, required copying an underlying string. This is because the string type in question could use 8 or 16 bit characters depending on the OS type, and because StringPiece is not implemented as a template, the code cannot specify whether to create a StringPiece or StringPiece16. This should be remedied in a future CL. R=erikwright@chromium.org BUG=106655 TEST= Review URL: http://codereview.chromium.org/8921006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114929 0039d316-1c4b-4281-b951-d872f2087c98
* base::Bind: Convert chrome_frame/.jhawkins@chromium.org2011-11-181-5/+7
| | | | | | | | | | | BUG=none TEST=none R=csilv@chromium.org Review URL: http://codereview.chromium.org/8555001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110744 0039d316-1c4b-4281-b951-d872f2087c98
* Moved the ListenSocket class (found in net/base/listen_socket.h) to the net ↵rkn@chromium.org2011-06-231-13/+15
| | | | | | | | | | | namespace. BUG=87032 TEST=None Review URL: http://codereview.chromium.org/7190041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90257 0039d316-1c4b-4281-b951-d872f2087c98
* Remove Chrome Frame's dependency on xul_runner.tommi@chromium.org2011-04-111-2/+1
| | | | | | Review URL: http://codereview.chromium.org/6825026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81171 0039d316-1c4b-4281-b951-d872f2087c98
* Attempt to fix a chrome frame tests crash consistently seen on the IE9 ↵ananta@chromium.org2011-01-121-0/+2
| | | | | | | | | | | | | | | | | | builder. I was able to reproduce it once and it appears to be occuring when the ListenSocket attempts to pass an OnObjectSignaled notification to a delegate which is the web server which has already been destroyed Attempted fix is to destroy the ListenSocket in the destructor of the web server. BUG=none TEST=Existing chrome frame tests. TBR=amit Review URL: http://codereview.chromium.org/6155009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71125 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a build break due to r63514.erikwright@chromium.org2010-10-221-2/+2
| | | | | | | | BUG=52601 TEST=chrome_frame_tests / FullTabDownloadTest.CF_DownloadFileFromPost TBR=stoyan@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63521 0039d316-1c4b-4281-b951-d872f2087c98
* Update code that previously constructed strings from string iterators only ↵erikwright@chromium.org2010-10-221-1/+3
| | | | | | | | | | to use StringToInt. These usages now pass the iterators directly to the new StringToInt overloads. BUG=None TEST=All Review URL: http://codereview.chromium.org/3968001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63515 0039d316-1c4b-4281-b951-d872f2087c98
* Convert LOG(INFO) to VLOG(1) - chrome_frame/.pkasting@chromium.org2010-10-191-3/+3
| | | | | | | | | | Plus a couple misc. cleanups. BUG=none TEST=none Review URL: http://codereview.chromium.org/3850002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63095 0039d316-1c4b-4281-b951-d872f2087c98
* chrome_frame: Append base:: in the StringPrintf calls.tfarina@chromium.org2010-10-011-19/+22
| | | | | | | | | | | (Note: this is a TODO in string_util.h) BUG=None TEST=trybots Review URL: http://codereview.chromium.org/3594002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61166 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 1)thestig@chromium.org2010-08-051-1/+4
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3071012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55034 0039d316-1c4b-4281-b951-d872f2087c98
* Remove number conversion functions from string_util. These moved to ↵brettw@chromium.org2010-08-031-3/+6
| | | | | | | | | | string_number_conversions. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3054036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54747 0039d316-1c4b-4281-b951-d872f2087c98
* [chrome_frame] Refactor/merge IE no interference tests with other mock event ↵kkania@chromium.org2010-07-121-5/+33
| | | | | | | | | | sink tests. Have the test mock contain, not inherit the IE event sink. Use the new test server to verify the right requests are being sent. BUG=none TEST=none Review URL: http://codereview.chromium.org/2822016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52137 0039d316-1c4b-4281-b951-d872f2087c98
* Brushed up listen socket:pfeldman@chromium.org2010-07-051-4/+9
| | | | | | | | | | | - Upstreamed support for partial results from devtools' version - Made DidRead receive data and length (in order to support websockets data) - Fixed all the clients. Added net/server with http socket implementation that supports websockets. Will remove net/tools fetch client and server later. Review URL: http://codereview.chromium.org/2868036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51635 0039d316-1c4b-4281-b951-d872f2087c98
* Yet another test web server. Supports speed throttle and dump of the traffic.stoyan@chromium.org2010-06-091-0/+114
| | | | | | | | | | Allows binding on 0.0.0.0 to allow connections from a VM (for example). Simple GMock class and Actions. Review URL: http://codereview.chromium.org/2620006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49266 0039d316-1c4b-4281-b951-d872f2087c98
* Unit test for the multiple request issues we've been seeing.tommi@chromium.org2010-02-251-1/+10
| | | | | | | | | | | The test is disabled until we fix the actual problem. TEST=n/a BUG=33332 Review URL: http://codereview.chromium.org/661017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39954 0039d316-1c4b-4281-b951-d872f2087c98
* A new unit test for the latest issue with switching renderers.tommi@chromium.org2010-02-061-15/+13
| | | | | | | | | | | | | | This affects both the httpequiv and IInternetProtocol mechanisms we've been using. The problem is going from mshtml to a CF page via POST. In this case the original POST request is issued once, but then a subsequent GET is issued for the same URL. On top of that, the referrer is missing. TEST=This test is disabled for now. BUG=none Review URL: http://codereview.chromium.org/580001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38282 0039d316-1c4b-4281-b951-d872f2087c98
* Initial import of the Chrome Frame codebase. Integration in chrome.gyp ↵slightlyoff@chromium.org2009-09-241-0/+211
coming in a separate CL. BUG=None TEST=None Review URL: http://codereview.chromium.org/218019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27042 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-06 01:23:09 +0000