Bug 30503

Summary:

[Qt] Plugins: Create manual tests for testing plugins

Product:

WebKit

Reporter:

Girish Ramakrishnan <girish>

Component:

Plug-ins

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, kenneth

Priority:

Keywords:

Version:

528+ (Nightly build)

Hardware:

OS:

All

Bug Depends on:

Bug Blocks:

30662

Attachments:

Description	Flags
manual tests (1)	none
manual tests (2)	none

Description Girish Ramakrishnan 2009-10-19 00:49:03 PDT

Here's a list of stuff that I have tested when implementing windowless plugins. So, I would like to add manual html test file which just loads up a flash plugin and mentions the below bullet points (with more explanation):

1. Test all the cases in windowed and windowless mode. Maybe I will create two flash instances, one in windowed and other in windowless.
2. Test resize of plugin
3. Test movement of plugin
4. Test with scrolling
5. Test with different paint systems
6. Test with the WebKit css animation stuff
7. If using Qt, test with graphicsview and qwebview
8. Test printing
9. Test with zoom
10. Test taking a screenshot (in QtLauncher)

Anything else?

I have marked it as [Qt] but this might be interesting for other ports. So, I will add all the above in a single HTML as WebCore/manual-tests/plugins-netscape.html. Thoughts?

Comment 1 Kenneth Rohde Christiansen 2009-10-19 05:08:43 PDT

Maybe do a plugin subdirectory ? Maybe even

plugins/windowed
plugins/windowless

?

Comment 2 Girish Ramakrishnan 2009-10-19 05:12:03 PDT

11. Test focus
12. Full screen mode and back (is this even supported?)
13. Full page mode (supported?)

Comment 3 Girish Ramakrishnan 2009-10-20 05:34:11 PDT

Created attachment 41501 [details]
manual tests (1)

Files will be created under WebCore/manual-tests/plugins

Comment 4 Girish Ramakrishnan 2009-10-20 07:32:22 PDT

Created attachment 41506 [details]
manual tests (2)

Oops, forgot the ChangeLog entry.

Comment 5 Kenneth Rohde Christiansen 2009-10-20 11:29:44 PDT

Looks good, though maybe it is not good depending on external resources as src="http://www.youtube.com/v/loXfcsXRB-w&#038;hl=en&#038;fs=1"

Comment 6 Girish Ramakrishnan 2009-10-20 18:30:14 PDT

(In reply to comment #5)
> Looks good, though maybe it is not good depending on external resources as
> src="http://www.youtube.com/v/loXfcsXRB-w&#038;hl=en&#038;fs=1"

I copied that from existing WebCore/manual-tests/plugins-test.html.

Comment 7 Girish Ramakrishnan 2009-10-20 18:31:59 PDT

(In reply to comment #6)
> (In reply to comment #5)
> > Looks good, though maybe it is not good depending on external resources as
> > src="http://www.youtube.com/v/loXfcsXRB-w&#038;hl=en&#038;fs=1"
> 
> I copied that from existing WebCore/manual-tests/plugins-test.html.

Scratch that. I meant existing WebCore/manual-tests/qt/plugin-sibling-frame-include.html

Comment 8 Holger Freyther 2009-10-21 05:57:24 PDT

Comment on attachment 41506 [details]
manual tests (2)

It would be cool if we could move to a flash site that is guranteed to stay. :)

Comment 9 WebKit Commit Bot 2009-10-21 06:08:18 PDT

Comment on attachment 41506 [details]
manual tests (2)

Clearing flags on attachment: 41506

Committed r49902: <http://trac.webkit.org/changeset/49902>

Comment 10 WebKit Commit Bot 2009-10-21 06:08:22 PDT

All reviewed patches have been landed.  Closing bug.