summaryrefslogtreecommitdiffstats
path: root/chrome/test/ui/npapi_uitest.cc
Commit message (Collapse)AuthorAgeFilesLines
* Implement NPAPI HTTP Redirect handling. This basically supports the NPAPI ↵ananta@chromium.org2010-11-231-0/+8
| | | | | | | | | | | | | | | | | | | URL redirect notifications which are sent out from the browser while processing URL requests issued by the plugin via the NPN_GetURLNotify and NPN_PostURLNotify APIs. For more info please see https://wiki.mozilla.org/NPAPI:HTTPRedirectHandling As part of this CL we also block cross origin 307 POST url redirects. Test=Covered by NPAPI UI test GetURLRedirectNotification Bug=63030,63698 Review URL: http://codereview.chromium.org/5228007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67029 0039d316-1c4b-4281-b951-d872f2087c98
* Remove some obsolete FLAKY_ markers, and add some missing bug referencesstuartmorgan@chromium.org2010-11-091-2/+1
| | | | | | | | | BUG=29020,35221,62143,62595 TEST=N/A Review URL: http://codereview.chromium.org/4759001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65589 0039d316-1c4b-4281-b951-d872f2087c98
* Update a bunch of FLAKY_ tests to match current realitystuartmorgan@chromium.org2010-10-281-5/+2
| | | | | | | | | | | Mostly this is removing FLAKY_ where the bug was fixed but the label never removed, but also switches some FLAKY_ to platform-specific FAILS_. BUG=28372,32048,32070,35341,45561,48544,48562,52858 TEST=N/A Review URL: http://codereview.chromium.org/4087009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64264 0039d316-1c4b-4281-b951-d872f2087c98
* Add UI test for click-to-play.bauerb@chromium.org2010-10-251-0/+19
| | | | | | | | | BUG=57277 TEST=NPAPIVisiblePluginTester.ClickToPlay:ClickToPlayPluginTest.* Review URL: http://codereview.chromium.org/3539002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63722 0039d316-1c4b-4281-b951-d872f2087c98
* GTTF: Remove flaky mark from the following tests:phajdan.jr@chromium.org2010-09-281-2/+1
| | | | | | | | | | | | | | | | | | | | | | - AccountScreenTest.TestBasic (bug 49004) - NPAPIVisiblePluginTester.VerifyNPObjectLifetimeTest (bug 34997) - ChromeFrameTestWithWebServer.WidgetModeFF_CFInstanceSingleton (bug 31532, still one test to be fixed in that bug) - SavePageBrowserTest.FileNameFromPageTitle (bug 52292) - FilePathWatcherTest.DisappearingDirectory (bug 54822) - FilePathWatcherTest.DestroyWithPendingNotification (bug 54822) - FilePathWatcherTest.WatchDirectory (bug 54822) - FilePathWatcherTest.Unregister (bug 54822) - FilePathWatcherTest.DeleteDuringNotify (bug 54822) - WindowAppleScriptTest.WindowOrder (bug 53584) TBR=mal BUG=49004, 34997, 31532, 52292, 54822, 53584 TEST=none Review URL: http://codereview.chromium.org/3471020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60776 0039d316-1c4b-4281-b951-d872f2087c98
* Renabling the NPAPIVisiblePluginTester.OpenPopupWindowWithPlugin test. This ↵ananta@chromium.org2010-09-131-2/+7
| | | | | | | | | | | | | | test was marked as DISABLED as it caused crashes. I could not repro the failures and looking at the code did not reveal anything suspicious at this point. I am enabling this test and marking it as FLAKY for now. Will monitor the builders to see if the crash recurs. Bug=17645 Review URL: http://codereview.chromium.org/3376002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59279 0039d316-1c4b-4281-b951-d872f2087c98
* Fix for the missing plugin message displayed at times on sites like youtube ↵ananta@chromium.org2010-08-251-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | while playing playlists, etc. The message is displayed when we fail to create a channel for the plugin process. The problem occurs because of a race condition between the plugin channel information being deleted from the channel map and the actual channel being deleted. This race occurs if the plugin is in the context of a sync call to the renderer which results in the plugin instance being deleted and the new one getting created in the same context. This results in the plugin channel information getting deleted from the map. The actual channel gets deleted only when the plugin sync call unwinds which occurs after the browser sends in a request to create the channel for the new plugin instance. Fix is to create the channel name with an incrementing number in the plugin process, if the mode is server. The channel map is still keyed off the process id and renderer id as before. Fixes bug http://code.google.com/p/chromium/issues/detail?id=43595 Bug=43595 Test=Covered by new ui test SelfDeleteCreatePluginInNPNEvaluate. The other change is a fix in the test plugin to delete a windows timer correctly and to add support for a new plugin test case. Review URL: http://codereview.chromium.org/3142039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57372 0039d316-1c4b-4281-b951-d872f2087c98
* Take 2: Copy the npapi pepper test plugin during the build so wetony@chromium.org2010-08-071-11/+11
| | | | | | | | | | | | | | | | don't have to do it when running the test. Remove all the code in the test for copying files since it's no longer needed. The difference from before is we only copy over the pepper test plugin on windows because it's the only platform with tests for it. On Linux, unloading the plugin was causing random crashes in tcmalloc. Review URL: http://codereview.chromium.org/3015064 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55357 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Copy the npapi pepper test plugin during the build so we don't"tony@chromium.org2010-08-061-11/+11
| | | | | | | | This reverts commit r55235. Review URL: http://codereview.chromium.org/3090020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55287 0039d316-1c4b-4281-b951-d872f2087c98
* Copy the npapi pepper test plugin during the build so we don'ttony@chromium.org2010-08-061-11/+11
| | | | | | | | | | have to do it when running the test. Remove all the code in the test for copying files since it's no longer needed. Review URL: http://codereview.chromium.org/3048048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55235 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 3)thestig@chromium.org2010-08-061-1/+0
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3037044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55192 0039d316-1c4b-4281-b951-d872f2087c98
* Move the number conversions from string_util to a new file.brettw@chromium.org2010-07-301-1/+2
| | | | | | | | | | | | | Use the base namespace in the new file. Update callers. I removed all wstring variants and also the string->number ones that ignore the return value. That encourages people to write code and forget about error handling. TEST=included unit tests BUG=none Review URL: http://codereview.chromium.org/3056029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54355 0039d316-1c4b-4281-b951-d872f2087c98
* Move tests for the layout test plugin to the right fixture.jochen@chromium.org2010-07-161-55/+1
| | | | | | | | | BUG=30702 TEST=none Review URL: http://codereview.chromium.org/3020005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52669 0039d316-1c4b-4281-b951-d872f2087c98
* Re-flakify TEST_F(NPAPITester, FLAKY_ManyPlugins)jrg@chromium.org2010-06-011-1/+3
| | | | | | | | | | | | | | Fails under valgrind http://build.chromium.org/buildbot/memory/builders/Chromium%20Mac%20UI%20(valgrind)(4)/builds/2607/steps/valgrind%20test:%20ui/logs/stdio BUG=http://crbug.com/28372 BUG=http://crbug.com/45561 TBR=jungshik@chromium.org Review URL: http://codereview.chromium.org/2482001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48645 0039d316-1c4b-4281-b951-d872f2087c98
* Make SelfDeletePluginInvokeAlert an interactive_ui test and reenable. Itmpcomplete@chromium.org2010-05-281-20/+0
| | | | | | | | | | requires the screen to be unlocked to run. BUG=44958 Review URL: http://codereview.chromium.org/2321005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48523 0039d316-1c4b-4281-b951-d872f2087c98
* Mark NPAPITester.ManyPlugins as no longer flaky.phajdan.jr@chromium.org2010-05-271-2/+1
| | | | | | | | | | | | | Has not failed for 14 days, last failure was May 12. TBR=jam TEST=ui_tests BUG=28372 Review URL: http://codereview.chromium.org/2240007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48402 0039d316-1c4b-4281-b951-d872f2087c98
* Enable SelfDeletePluginInvokeInSynchronousPaint as FLAKYstuartmorgan@chromium.org2010-05-251-1/+3
| | | | | | | | | | | Converts SelfDeletePluginInvokeInSynchronousPaint, which was disabled for flakiness, to the newer FLAKY_, and adds a bug annotation. Also adds a bug annotation for SelfDeletePluginInvokeAlert. BUG=44960 TEST=N/A Review URL: http://codereview.chromium.org/2095030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48151 0039d316-1c4b-4281-b951-d872f2087c98
* Remove FLAKY marker from some Mac plugin testsstuartmorgan@chromium.org2010-05-191-14/+2
| | | | | | | | | | | Neither of these tests appear to have failed recently on the 10.6 bots. BUG=42340, 42086 TEST=Tests pass on 10.6 bots. Review URL: http://codereview.chromium.org/2086016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47729 0039d316-1c4b-4281-b951-d872f2087c98
* Shrink the window used for the NPN_ConvertPoint teststuartmorgan@chromium.org2010-05-191-3/+2
| | | | | | | | | | | | | The test should now fit on an 800x600 display, which is apparently what the 10.6 bots have. Removes the FLAKY marker since it should now pass on 10.6 bots. BUG=36670 TEST=PluginConvertPointTest should pass on 10.6 bots Review URL: http://codereview.chromium.org/2100008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47720 0039d316-1c4b-4281-b951-d872f2087c98
* Mark NPAPIVisiblePluginTester.VerifyNPObjectLifetime as FLAKY.phajdan.jr@chromium.org2010-04-291-1/+2
| | | | | | | | | | TBR=jam TEST=none BUG=34997 Review URL: http://codereview.chromium.org/1687021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45949 0039d316-1c4b-4281-b951-d872f2087c98
* Marks NPAPITester.Arguments as flaky on Mac.bulach@chromium.org2010-04-221-1/+7
| | | | | | | | TBR=stuartmorgan@chromium.org Review URL: http://codereview.chromium.org/1763005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45340 0039d316-1c4b-4281-b951-d872f2087c98
* [Mac] Mark a test flaky since it fails most of the time on 10.6thomasvl@chromium.org2010-04-201-1/+7
| | | | | | | | | TBR=stuartmorgan BUG=42086 TEST=none Review URL: http://codereview.chromium.org/1527039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45033 0039d316-1c4b-4281-b951-d872f2087c98
* [GTTF] Tests cleanup:phajdan.jr@chromium.org2010-04-081-31/+31
| | | | | | | | | | | | | | - use timeouts correctly - use more automation methods to make test code more clear - use more reliable ways of waiting - remove unnecessary automation calls TEST=none BUG=none Review URL: http://codereview.chromium.org/1553016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43932 0039d316-1c4b-4281-b951-d872f2087c98
* Merge UiTest::GetTest* and ui_test_utils::GetTest*. Convert these utility ↵thestig@chromium.org2010-03-311-50/+70
| | | | | | | | | | functions to use FilePaths. BUG=none TEST=none Review URL: http://codereview.chromium.org/1567009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43190 0039d316-1c4b-4281-b951-d872f2087c98
* Added a test for pepper3d. It ensures that we can successfully load a pepper ↵alokp@chromium.org2010-03-261-4/+4
| | | | | | | | 3d plugin and render. Review URL: http://codereview.chromium.org/1073003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42793 0039d316-1c4b-4281-b951-d872f2087c98
* [GTTF] Reduce number of automation methods ending with WithTimeout.phajdan.jr@chromium.org2010-03-221-3/+3
| | | | | | | | | | | | | | | | | | | | | They are just too easy to misuse. People started inventing their own hardcoded timeouts all over the place. Also, the is_timeout return parameter was not checked consistently. Additionally, some calls actually had no timeout at all, making hangs possible. This change also removes useless DLOG statements. We should get the required info from the ASSERT/EXPECT macros in test code, which is quite well checked by WARN_UNUSED_RESULT. TEST=all ui-based BUG=none Review URL: http://codereview.chromium.org/1076005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42213 0039d316-1c4b-4281-b951-d872f2087c98
* [GTTF] Make automation framework more solid by making sure thatphajdan.jr@chromium.org2010-03-161-4/+6
| | | | | | | | | | | | | | | | | | | important return values are not ignored. The macro used here, WARN_UNUSED_RESULT, takes effect only for GCC, but that's enough since we have bots for all platforms. Adding these warnings already detected some cases where a return value was ignored. In some of them the test was trying to get the information about success "some other way", in some it could sometimes confuse test failure with test success. TEST=ui_tests BUG=none Review URL: http://codereview.chromium.org/948002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41696 0039d316-1c4b-4281-b951-d872f2087c98
* Reduce the flakiness of things using NPAPITester, includingphajdan.jr@chromium.org2010-03-091-59/+36
| | | | | | | | | | | | | | NPAPITester.ManyPlugins. Make sure that the copied plugin gets deleted, and do it after the browser has exited. TEST=less flaky BUG=28372 Review URL: http://codereview.chromium.org/668255 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41015 0039d316-1c4b-4281-b951-d872f2087c98
* adding bug link i forgotthomasvl@chromium.org2010-02-241-0/+1
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39897 0039d316-1c4b-4281-b951-d872f2087c98
* Mark a mac test as flaky that seems to be failing on 10.6 but not 10.5thomasvl@chromium.org2010-02-241-1/+1
| | | | | | | | | TBR=stuartmorgain@chromium.org BUG=36670 TEST=stops some red on the waterfall Review URL: http://codereview.chromium.org/657084 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39894 0039d316-1c4b-4281-b951-d872f2087c98
* Don't NPAPIVisiblePluginTester.VerifyNPObjectLifetime as FLAKY so that we're ↵jam@chromium.org2010-02-091-2/+1
| | | | | | | | | | | forced to figure out what's broken. TBR=phajdan.jr TEST=none BUG=34997 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38460 0039d316-1c4b-4281-b951-d872f2087c98
* Finish implementing NPN_ConvertPoint, and add a unit test for itstuartmorgan@chromium.org2010-02-091-0/+21
| | | | | | | | | BUG=29457,31767 TEST=self-testing Review URL: http://codereview.chromium.org/580019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38417 0039d316-1c4b-4281-b951-d872f2087c98
* Mark NPAPIVisiblePluginTester.VerifyNPObjectLifetime as FLAKY.phajdan.jr@chromium.org2010-02-081-1/+2
| | | | | | | | | | TBR=jam TEST=none BUG=34997 Review URL: http://codereview.chromium.org/583009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38359 0039d316-1c4b-4281-b951-d872f2087c98
* Mark the following tests FLAKY on Windows.tkent@chromium.org2010-01-121-1/+2
| | | | | | | | | | | | | | - NPAPITester.NoHangIfInitCrashes - MetricsServiceTest.CrashRenderers - ResourceDispatcherTest.CrossSiteAfterCrash - RendererCrashTest.Crash BUG=32048 TEST=none TBR=jar, nsylvain Review URL: http://codereview.chromium.org/548009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36004 0039d316-1c4b-4281-b951-d872f2087c98
* Enable windowless-based npapi_test_plugin tests on the Mac.stuartmorgan@chromium.org2009-12-181-2/+2
| | | | | | | | | BUG=30488 TEST=N/A Review URL: http://codereview.chromium.org/501074 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34959 0039d316-1c4b-4281-b951-d872f2087c98
* Marking NPAPITester.SelfDeletePluginInvoke as flaky.mirandac@chromium.org2009-12-171-1/+2
| | | | | | | | BUG= 30702 TEST= none Review URL: http://codereview.chromium.org/506056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34838 0039d316-1c4b-4281-b951-d872f2087c98
* Enable many of the NPAPI UI tests on Macstuartmorgan@chromium.org2009-12-161-6/+9
| | | | | | | | | BUG=none TEST=N/A Review URL: http://codereview.chromium.org/503029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34755 0039d316-1c4b-4281-b951-d872f2087c98
* Don't set referrers on outgoing plugin requests if the load_manually flag is ↵ananta@chromium.org2009-12-031-0/+15
| | | | | | | | | | | | | | | | | | | set. This emulates the behavior of other browsers and fixes http://code.google.com/p/chromium/issues/detail?id=28800 I added a UI test to validate that the plugin source URL is set on outgoing GetURL requests issued by the plugin. To validate that the document URL is set as the referrer on the initial URL request would take some more work. Will try and add that in a future CL. I also changed the WebPluginImpl::RouteToFrame function to set the referrer on similar lines. Bug=28800 Test=Covered by UI test Review URL: http://codereview.chromium.org/459003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33673 0039d316-1c4b-4281-b951-d872f2087c98
* Mark GetJavaScriptURL2 as flaky.willchan@chromium.org2009-11-301-1/+2
| | | | | | | | BUG=http://crbug.com/29020 Review URL: http://codereview.chromium.org/450015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33340 0039d316-1c4b-4281-b951-d872f2087c98
* Mark NPAPITester.ManyPlugins as flaky, it keeps flickering.tim@chromium.org2009-11-201-1/+2
| | | | | | | | | | TBR=phajdan.jr BUG=28372 Review URL: http://codereview.chromium.org/421008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32639 0039d316-1c4b-4281-b951-d872f2087c98
* If an NP_* function is called on an out of process plugin, save enough info ↵japhet@chromium.org2009-11-181-0/+12
| | | | | | | | to send an NPN_SetException back to the correct renderer if necessary. BUG=26764 TEST=none Review URL: http://codereview.chromium.org/375005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32419 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure that NPN_PluginThreadAsyncCall callbacks are not invoked after ↵apatrick@google.com2009-10-301-0/+7
| | | | | | | | | | | NPP_Destroy. TEST=none BUG=none Review URL: http://codereview.chromium.org/338050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30628 0039d316-1c4b-4281-b951-d872f2087c98
* Implemented NPN_ScheduleTimer and NPN_UnscheduleTimer.apatrick@google.com2009-10-261-0/+7
| | | | | | | | | TEST=none BUG=18020 Review URL: http://codereview.chromium.org/329013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30105 0039d316-1c4b-4281-b951-d872f2087c98
* Add a regression test for the PluginChannel::CleanUp. My earlier ↵jam@chromium.org2009-10-241-0/+16
| | | | | | | | | | | speculative fix was correct. I tracked this down to my earlier change to not leak NPObjects on channel shutdown (which now happens a lot more often because of sudden termination for tab close). Since we now call an NPObject's deallocate function, it's possible that it leads to an NPObjectProxy being deleted, in which case we would delete an object in the listener's array while we're looping across it. BUG=25439 Review URL: http://codereview.chromium.org/332013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29982 0039d316-1c4b-4281-b951-d872f2087c98
* Add a regression test for the renderer hang when a plugin's NP_Initialize ↵jam@chromium.org2009-10-231-0/+25
| | | | | | | | | function crashes. BUG=25104 Review URL: http://codereview.chromium.org/337004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29923 0039d316-1c4b-4281-b951-d872f2087c98
* Use FilePath in URLRequestMocks.evan@chromium.org2009-10-141-1/+2
| | | | | | | | | BUG=24672 TEST=compiles Review URL: http://codereview.chromium.org/271066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29013 0039d316-1c4b-4281-b951-d872f2087c98
* Mark SelfDeletePluginInvokeInSynchronousPaint as disabled since it seems to ↵johnnyg@chromium.org2009-10-081-1/+2
| | | | | | | | | | | | be flaky. BUG=none TEST=none TBR=jam Review URL: http://codereview.chromium.org/261020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28379 0039d316-1c4b-4281-b951-d872f2087c98
* Fix scripting during NPP_Destroy. Note that if the plugin is making a call ↵jam@chromium.org2009-10-061-35/+50
| | | | | | | | | | to the renderer so this instance is in the callstack, destruction will have to be asynchronous and so scripting still won't work. This change also fixes use of PluginChannel after it's deleted (if this was the last instance). BUG=23713, 23706 TEST=added ui test Review URL: http://codereview.chromium.org/258026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28141 0039d316-1c4b-4281-b951-d872f2087c98
* Add an optional WebFrame parameter to WebView::findFrameByName.darin@chromium.org2009-09-301-0/+10
| | | | | | | | | | | | | This parameter is used to support _self and other names that need to be evaluated relative to a subframe. R=jam BUG=23009 TEST=none Review URL: http://codereview.chromium.org/257005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27646 0039d316-1c4b-4281-b951-d872f2087c98
* Relanding keyboard code refactoring:jcampan@chromium.org2009-09-291-1/+2
| | | | | | | | | | | | | | | | http://codereview.chromium.org/248010/show Changing the KeyboardEvent to use a KeyboardCode instead of a w_char. Led to several places where I had to switch from VK_ to VKEY_. Also cleaned-up the table view OnKeyDown method. Since TableView is a NativeControl it can use the NativeControl::OnKeyDown directly. BUG=None TEST=Make sure short-cuts works as expected, especially in the omnibox. Review URL: http://codereview.chromium.org/251020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27444 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-14 13:15:19 +0000