Bug 32333 – Layout test for video element on HTTP

• Status: RESOLVED
• Resolution: FIXED
• Component: Media

[Hin-Chung Lam]

Add a new layout test that can test the user agent's ability to handle caching of video resources.

To start with a simple case, this test will do the following:

1. Load the resource, until we receive canplaythrough event
2. Remove the element
3. Load the resource the second time, until we receive canplaythrough event

If the user agent performs caching of video resource, the first load will have the resource goes into cache and the second load will work fine too.

If the user agent doesn't do any caching, both load will always work fine.

[Hin-Chung Lam]

Created attachment 44564 [details]
new test + expectation

[Eric Carlson]

Comment on attachment 44564 [details]
new test + expectation

As we discussed in irc, the patch looks good but it would be helpful to have more information in the ChangeLog about why it is necessary to copy the test movies instead of using the originals like the other http media tests do.

r=me with some extra comments

[Hin-Chung Lam]

Thanks for the review, will fix the change log before I commit.

[Eric Seidel (no email)]

Attachment 44564 [details] was posted by a committer and has review+, assigning to Hin-Chung Lam for commit.

[Eric Seidel (no email)]

Looks like this has been r+'d for a little over two weeks.  I assume things are just a bit slow due to the holiday season.  Or maybe this was already landed?

[Hin-Chung Lam]

(In reply to comment #5)
> Looks like this has been r+'d for a little over two weeks.  I assume things are
> just a bit slow due to the holiday season.  Or maybe this was already landed?

Sorry for late response. This has been submitted as http://trac.webkit.org/changeset/51966.

I'm closing it, thanks for reminding me.