summaryrefslogtreecommitdiffstats
path: root/chrome_frame/test/net
Commit message (Collapse)AuthorAgeFilesLines
* chrome_frame: Get rid of all occurrences of the deprecated ToWStringHack ↵tfarina@chromium.org2010-06-081-2/+2
| | | | | | | | | | | function. BUG=24874 TEST=trybots Review URL: http://codereview.chromium.org/2632006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49133 0039d316-1c4b-4281-b951-d872f2087c98
* Prepare the default profile and turn off some services before initializing ↵tommi@chromium.org2010-05-101-3/+4
| | | | | | | | | | | | extensions. TEST=This is an attempt to reduce flakiness of Chrome Frame net tests. BUG=none TBR=ananta Review URL: http://codereview.chromium.org/2051004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46807 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up suppressing error dialogs for buildbots.phajdan.jr@chromium.org2010-05-041-1/+3
| | | | | | | | | | | | - also suppress dialogs caused by __debugbreak on Windows - avoid bad interactions with gtest (they cause odd problems, really) TEST=none BUG=29997 Review URL: http://codereview.chromium.org/1867001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46343 0039d316-1c4b-4281-b951-d872f2087c98
* Disable the following net tests for ChromeFrame as they rely on ↵ananta@chromium.org2010-04-161-0/+5
| | | | | | | | | | | | | | | functionality provided by Chrome's network stack like the ability to override a proxy for a URL request, etc. 1. URLRequestTestHTTP.ProxyTunnelRedirectTest 2.URLRequestTestHTTP.UnexpectedServerAuthTest. TBR=amit Review URL: http://codereview.chromium.org/1667003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44828 0039d316-1c4b-4281-b951-d872f2087c98
* The ChromeFrame net tests would not work on IE6 and IE7. This is because the ↵ananta@chromium.org2010-04-072-16/+2
| | | | | | | | | | | | | | | profile path used varies across these browsers. This changed in the recent fixes for privacy where on IE6 and IE7 the profile is in temporary internet files. The ChromeFrame net tests assumed the profile would always be in the default path which caused the tests to not work. Added a helper function in chrome_Frame_test_utils.cc/.h to retrieve the correct profile path based on IE version and use this. This should get the net tests up and running on the FYI IE6 builder. Review URL: http://codereview.chromium.org/1553015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43856 0039d316-1c4b-4281-b951-d872f2087c98
* Chrome Frame tests for keyboard acceleratorsamit@chromium.org2010-03-091-4/+4
| | | | | | | | | | | | | | | | Added tests for commonly used accelerator keys Ctrl+N - New window Ctrl+W - Close Window Ctrl+R - Reload page Also added test for saving current web page using: Alt-F, A BUG=34675 TEST=new tests added Review URL: http://codereview.chromium.org/689002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41055 0039d316-1c4b-4281-b951-d872f2087c98
* Next part of removing the dependency of chrome/common on chrome/browserphajdan.jr@chromium.org2010-03-061-1/+1
| | | | | | | | TEST=none BUG=none Review URL: http://codereview.chromium.org/669184 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40834 0039d316-1c4b-4281-b951-d872f2087c98
* More context menu testsamit@chromium.org2010-03-043-214/+100
| | | | | | | | | | | | | Refactored code to monitor windows so that it could be used with our mock to watch for dialogs and chrome windows. Added new tests for 'Page Info', 'Dev Tools' and 'Save As' menu options. BUG=34673 TESTS=newly added Review URL: http://codereview.chromium.org/667005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40654 0039d316-1c4b-4281-b951-d872f2087c98
* Move pref_{member,service} and important_file_writer from chrome/commonphajdan.jr@chromium.org2010-02-191-1/+1
| | | | | | | | | | | | | to chrome/browser. This is a part of an effort to remove bad dependency of chrome/common on chrome/browser. TEST=unit_tests and ui_tests, just moving code BUG=none Review URL: http://codereview.chromium.org/621004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39428 0039d316-1c4b-4281-b951-d872f2087c98
* First batch of context menu testsamit@chromium.org2010-02-121-6/+6
| | | | | | | | | | | | | | | | Refactored various methods to send keyboard and mouse input. Fixed the context menu focus issue on IE7. Improved existing tests to make them less flaky and added 3 new tests for context menu items. BUG=34673 TEST=new tests added Review URL: http://codereview.chromium.org/604014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38905 0039d316-1c4b-4281-b951-d872f2087c98
* Disabling the following net tests for ChromeFrame as this functionality does ↵ananta@chromium.org2010-02-101-1/+5
| | | | | | | | | | | | | | | | not exist there. 1. URLRequestTest.DoNotSendCookies 2. URLRequestTest.DoNotSendCookies_ViaPolicy_Async 3. URLRequestTest.CancelTest_During_OnGetCookiesBlocked 4. URLRequestTest.CancelTest_During_OnSetCookieBlocked TBR=amit Review URL: http://codereview.chromium.org/601020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38585 0039d316-1c4b-4281-b951-d872f2087c98
* Adding the URLRequestTest.CookiePolicy_ForceSession test to the list of ↵ananta@chromium.org2010-02-051-0/+1
| | | | | | | | | | | | disabled tests as it does not apply to the host network stack. TBR=amit Review URL: http://codereview.chromium.org/572040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38230 0039d316-1c4b-4281-b951-d872f2087c98
* Disabling the following url request tests for ChromeFrame as these policies ↵ananta@chromium.org2010-02-041-0/+6
| | | | | | | | | | | | | | | | don't apply to the host network stack as it should be honoring the host browser's cookie policy. 1.URLRequestTest.DoNotSaveCookies_ViaPolicy 2.URLRequestTest.DoNotSendCookies_ViaPolicy 3.URLRequestTest.DoNotSaveCookies_ViaPolicy_Async TBR=amit Review URL: http://codereview.chromium.org/569034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38058 0039d316-1c4b-4281-b951-d872f2087c98
* In IE8 new windows opened within ChromeFrame via window.open calls at times ↵ananta@chromium.org2010-01-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bypass the host network stack. In this case we don't get control over the navigation as window.open calls expect to carry the opener relationship over to the new window. This basically means that navigation occurs in Chrome and the new tab/window created by the host attaches to the newly created ExternalTabContainer instance. In IE8 the new tab opens in a new IE process, which basically uses a new automation channel to talk to Chrome. The host network stack implementation routes network requests issued by registered render views to the host browser over the automation channel. There is a timing window between the new window getting created and issuing network requests and the channel being established with the new iexplore instance. As a result network requests issued by the new window don't use the host network stack. Fix is to register the render view and process as a pending view when we get notified about the new TabContents in the original ExternalTabContainers implementation of TabContentsDelegate::AddNewContents. Any network requests issued for this view would result in the corresponding URLRequestAutomationJob instances getting created as pending as well. When the host browser connects to the new ExternalTabContainer instance, we pass over the new automation channel and tab handle to the URLRequestAutomationJob instances and resume them. This fixes bug http://code.google.com/p/chromium/issues/detail?id=33516 Bug=33516 Test=Login to gmail in IE8 in ChromeFrame. Open up a new conversation and click on New Window which opens up a tear off window. This should not bring up the login prompt again. Will think about a good approach to write a unit test for this behavior and send out a separate CL for that. Review URL: http://codereview.chromium.org/554134 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37585 0039d316-1c4b-4281-b951-d872f2087c98
* Cleaning up dangling changes: remove dead code in fake_external_tab.cc.robertshield@chromium.org2010-01-271-51/+0
| | | | | | Review URL: http://codereview.chromium.org/553118 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37269 0039d316-1c4b-4281-b951-d872f2087c98
* Fix chrome frame net tests that were failing on some machines. Checking in ↵tommi@chromium.org2010-01-081-1/+7
| | | | | | | | | | | straight away as per Robert's request. TEST=run the net tests for chrome frame. BUG=none TBR=robertshield Review URL: http://codereview.chromium.org/527034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35794 0039d316-1c4b-4281-b951-d872f2087c98
* Disabling the URLRequestTestHTTP.GetTest_NoCache test for ChromeFrame as ↵ananta@chromium.org2009-12-301-0/+4
| | | | | | | | | | | | this has been consistently failing on the chrome frame builder. TBR=tommi Review URL: http://codereview.chromium.org/515071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35395 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes to get the chrome frame network tests up and running on the chrome ↵ananta@chromium.org2009-12-021-2/+10
| | | | | | | | | | | | | | | | | frame builder. Fixes as below:- 1. Instantiate the SystemMonitor object as it is needed to instantiate the profile. 2. Adds the single-process command line to the chrome_frame_net_tests executable to ensure that the webkit thread gets initialized correctly. 3. Register additional prefs in the profile to get around some DCHECKS which fire in debug mode TBR=tommi Review URL: http://codereview.chromium.org/464001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33570 0039d316-1c4b-4281-b951-d872f2087c98
* [GTTF] Reduce header dependencies in chrome.phajdan.jr@chromium.org2009-11-302-0/+3
| | | | | | | | | TEST=none BUG=none Review URL: http://codereview.chromium.org/454004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33324 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 33301 and 33305 due to unit_tests breakage.maruel@chromium.org2009-11-302-3/+0
| | | | | | | | TBR=pawel Review URL: http://codereview.chromium.org/453007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33309 0039d316-1c4b-4281-b951-d872f2087c98
* [GTTF] Reduce header dependencies in chrome.phajdan.jr@chromium.org2009-11-302-0/+3
| | | | | | | | | TEST=none BUG=none Review URL: http://codereview.chromium.org/454004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33301 0039d316-1c4b-4281-b951-d872f2087c98
* Make SystemMonitor not a Singleton and move it out of basephajdan.jr@chromium.org2009-11-261-1/+1
| | | | | | | | | | | | | SystemMonitor makes an assumption that through its lifetime a MessageLoop exists and stays the same. It is difficult and error-prone to satisfy that when it is a Singleton. It has caused problems in the past. Additionally, extract HighResoltionTimerManager out of time_win.cc, eliminating yet another Singleton. TEST=none BUG=none Review URL: http://codereview.chromium.org/431008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33214 0039d316-1c4b-4281-b951-d872f2087c98
* Load chrome.dll as a resource dll so that resources can be loaded by the ↵tommi@chromium.org2009-11-252-2/+11
| | | | | | | | | | | browser. TEST=fixes net tests for chrome frame BUG=none Review URL: http://codereview.chromium.org/437050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33094 0039d316-1c4b-4281-b951-d872f2087c98
* Tommi, please review everything. John please review the changes to ↵ananta@chromium.org2009-11-213-14/+26
| | | | | | | | | | | | | | | | | | plugin_service.cc/.h The test automation provider registers itself as a protocol factory for http/https requests. This is to ensure that intercepts set by the url request network tests work correctly. I was seeing these tests fail consistently on my setup as their intercept function would never get called. The other change is to add a simple mechanism based on a boolean flag to disable browser side plugins like gears which also intercept network requests and expect to be called on the IO thread. The chrome frame network tests run in a relatively simple environment where the network tests run in a separate thread (not the IO thread) which causes a number of DCHECKS to fire in debug build test runs. The flag used to determine whether browser plugins are loaded defaults to true. Review URL: http://codereview.chromium.org/414017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32739 0039d316-1c4b-4281-b951-d872f2087c98
* Basic wiring to enable downloads for CF's host browser network stack. A ↵tommi@chromium.org2009-11-072-3/+9
| | | | | | | | notable change here is that url automation job id's no longer exist and instead a request id is used. There's a 1 to 1 relation between a job and a request so this is more convenient. Review URL: http://codereview.chromium.org/355036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31363 0039d316-1c4b-4281-b951-d872f2087c98
* Remove themes/default.dll and merge the resources into chrome.dll.tc@google.com2009-11-021-1/+0
| | | | | | | | | | | This gives us one less file to load on startup. This does mean that some tests need to explicitly include theme_resources.rc. BUG=24035 Review URL: http://codereview.chromium.org/348033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30755 0039d316-1c4b-4281-b951-d872f2087c98
* Initialize the chrome thread object at the start of the ↵ananta@chromium.org2009-10-301-4/+4
| | | | | | | | | | | | | | | | CFUrlRequestUnittestRunner constructor. This causes a CHECK to fire in the extensions service initialization which happens when we initialize the profile. The CHECK was added in revision 30550. Given the spate of these test failures we should really speed up the process of moving the chrome frame builder to the main waterfall. TBR=tommi Review URL: http://codereview.chromium.org/346019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30556 0039d316-1c4b-4281-b951-d872f2087c98
* The newly added AboutChromeFrame test for IE full tab mode was failing on ↵ananta@chromium.org2009-10-301-2/+2
| | | | | | | | | | | | | | | | | | | the chrome frame builder which is an XP machine with IE8 (It was not supposed to return on IE8 :) The reason being that the VK_UP keyboard message we were passing in via SendInput needs the extended flag to be true. I added the extended flag as an argument to the SendVirtualKey helper function. On a related note, I removed the GetIEVersion check from the utility function which launches IE as a COM server as the IE version checking function checks the version of the current module and compares it against IE versions which is hilarious. In any case these tests work fine. TBR=amit Review URL: http://codereview.chromium.org/339076 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30549 0039d316-1c4b-4281-b951-d872f2087c98
* Instantiate a dummy notification service instance in chrome frame on the threadananta@chromium.org2009-10-291-0/+3
| | | | | | | | | | | where the network tests run. Some of the url request tests rely on having a valid notification service pointer in the TLS on the current thread. TBR=tommi Review URL: http://codereview.chromium.org/344025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30502 0039d316-1c4b-4281-b951-d872f2087c98
* With the change in the constructor for the ChromeThread class, we are seeing ↵ananta@chromium.org2009-10-281-1/+2
| | | | | | | | | | | | | | | | | | | a DCHECK fire in the Chrome frame network tests for whether the current thread is the UI thread. We need to invoke the correct constructor. The other fix is in our com_util::IDispatchImpl class where we end up crashing if we fail to find the type information for the passed in interface. Strangely this is failing for the IChromeFrame interface which is an IDispatch derivative and seems to be correctly registered. It also fails with the ATL IDispatchImpl classes though. However we should not crash here :) I also removed the NOTREACHED as it keeps firing on debug test runs. Will take a look at this closely later. TBR=tommi Review URL: http://codereview.chromium.org/343013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30333 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the Chrome Frame preprocessor define in chrome_constants.cc and deal ↵robertshield@chromium.org2009-10-282-8/+9
| | | | | | | | | | | | with resulting fallout. Also, remove several instances of Chrome Frame using wstrings instead of FilePaths. The main goal of this patch is to move towards ensuring that Chrome Frame and Google Chrome share binary-identical exes and dlls except for setup.exe and mini_installer.exe. Review URL: http://codereview.chromium.org/338025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30290 0039d316-1c4b-4281-b951-d872f2087c98
* Fix build error.ananta@chromium.org2009-10-271-1/+1
| | | | | | | | TBR=amit Review URL: http://codereview.chromium.org/340015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30266 0039d316-1c4b-4281-b951-d872f2087c98
* If a HTTP post to a server returns any redirect code other than 307, then ↵ananta@chromium.org2009-10-272-8/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | browsers don't preserve the request method, i.e. they convert the POST request to GET. For 307 redirects browsers preserve redirects. This CL fixes the following issues :- 1. As per the above description, we reset the method which ensures that we don't generate the post related headers. The Post302RedirectGet net test does test this very case. However it works correctly as Chrome follows the redirect and reissues the GET request. In this case this does not occur as the only calls which are invoked on the bind status callback after the redirect are GetBindInfo and BeginningTransaction where we incorrectly return the post related information. Ideally we would want to turn off follow redirects in Urlmon or Chrome. I tried the latter which has a number of issues. 2. In debug mode the chrome_frame_net_tests cause a DCHECK to be fired which indicates that the test is not being run on the UI thread. 3. As the Urlmon requests are now destroyed asynchronously having a DCHECK after the Stop call on the Urlmon request object in the CleanupAsyncRequests function is incorrect. Removed this DCHECK Fixes bug http://code.google.com/p/chromium/issues/detail?id=25643 Bug=25643 Review URL: http://codereview.chromium.org/333043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30261 0039d316-1c4b-4281-b951-d872f2087c98
* Fix another build break that didn't show up on the try bots.jam@chromium.org2009-10-271-1/+1
| | | | | | | | TBR=darin BUG=25354 Review URL: http://codereview.chromium.org/329035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30167 0039d316-1c4b-4281-b951-d872f2087c98
* Remove deprecated CommandLine(std::wstring) ctor.evan@chromium.org2009-10-261-2/+1
| | | | | | | | | | | Add a ctor for creating a CommandLine for carrying arguments; convert all the users to either that or the FilePath version. BUG=24672 Review URL: http://codereview.chromium.org/329017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30117 0039d316-1c4b-4281-b951-d872f2087c98
* Fix an include.brettw@chromium.org2009-10-091-1/+1
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28585 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes a compile error due to an call to an incorrect function to get the length.ananta@chromium.org2009-10-051-6/+6
| | | | | | | | TBR=tommi Review URL: http://codereview.chromium.org/246094 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28052 0039d316-1c4b-4281-b951-d872f2087c98
* Initial import of the Chrome Frame codebase. Integration in chrome.gyp ↵slightlyoff@chromium.org2009-09-2410-0/+1105
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