Create flash content test pages for endurance tests

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: vladmaniac, Assigned: vladmaniac)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Assignee)

Description

7 years ago
We need to have some test web pages with flash content, specifically: 

*embedded
*flash content via <object> tab to work with firefox browser
*iframes
(Assignee)

Updated

7 years ago
Assignee: nobody → vlad.maniac
Status: NEW → ASSIGNED

Comment 1

7 years ago
A Pivotal Tracker story has been created for this Bug: https://www.pivotaltracker.com/story/show/16302789
(Assignee)

Comment 2

7 years ago
Created attachment 548763 [details] [diff] [review]
test pages with flash content v1.0

Added first set of test pages for endurance tests. 
Those assume that we will put the pages in the litmus data repo under 
/litmus-data/firefox/plugins/flash folder. will correct if otherwise. 

Thanks!
Attachment #548763 - Flags: review?(anthony.s.hughes)
(Assignee)

Updated

6 years ago
Assignee: vlad.mozbugs → nobody
Component: Mozmill Tests → Infrastructure
QA Contact: mozmill-tests → infrastructure
(Assignee)

Comment 3

6 years ago
We cannot handle the plugin itself for flash so we need to do this using javascript. Therefore we need to modify the testpages to have multiple custom controls for videos in JS. This way we can access them with Mozmill. 

Cancelling r? request for this patch due to this decision.
(Assignee)

Updated

6 years ago
Attachment #548763 - Flags: review?(anthony.s.hughes)
For the initial tests, we want to simply load the flash content and gather metrics based on loading content. If your original patch satisfies this role please re-mark as r?
(Assignee)

Updated

6 years ago
Attachment #548763 - Flags: review?(anthony.s.hughes)
(Assignee)

Comment 5

6 years ago
Yes, there are simple pages with flash content as required.
Moving this bug out of Infrastructure into Mozmill Tests. Test pages still belong to tests. Infrastructure is more about the backend needed for test execution.
Component: Infrastructure → Mozmill Tests
QA Contact: infrastructure → mozmill-tests
Comment on attachment 548763 [details] [diff] [review]
test pages with flash content v1.0

>Bug 674513 - Create flash content test pages for endurance tests. r=ashughes, r=hskupin

Henrik won't be reviewing so you can remove him from your message.

>+++ b/firefox/plugins/flash/test_embedded.html

Please include the type being embedded to the filename.

>+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
>+"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
>+<html>
>+<head>
>+<title>Embedded Flash video test</title>
>+</head>

Make sure child nodes are indented.
Title should be "Embedded Flash video (FLV) test page"

>+<h1>Embedded flash video: .flv file </h1>

Use the flash video file name instead of ".flv file"

>+ <EMBED SRC="sample-flv-video.flv"
>+        HEIGHT=480 WIDTH=640>

WIDTH on its own line

Can you make similar changes to the other files? I'll re-review that patch.
Attachment #548763 - Flags: review?(anthony.s.hughes) → review-
Comment on attachment 548763 [details] [diff] [review]
test pages with flash content v1.0

>+<h1> Embedded flash video: .flv file </h1>
>+ <EMBED SRC="sample-flv-video.flv"
>+        HEIGHT=480 WIDTH=640>

Please always use lowercase tags and attributes.
Additionally, if ever in doubt (since we don't really have an HTML style guide), refer to the other test HTML files we have in litmus-data.
See also the specs for XHTML/HTML if something is unclear. I should have noted that: http://www.w3.org/TR/2002/REC-xhtml1-20020801/
(Assignee)

Updated

6 years ago
Assignee: nobody → vlad.mozbugs
(Assignee)

Comment 11

6 years ago
Created attachment 553427 [details] [diff] [review]
test pages with flash content v 1.1

Nits fixed - hope the new indentation looks decent
Attachment #548763 - Attachment is obsolete: true
Attachment #553427 - Flags: review?(anthony.s.hughes)
Comment on attachment 553427 [details] [diff] [review]
test pages with flash content v 1.1

>+++ b/firefox/plugins/flash/test_embedded_flv_video.html

Let's call this test_flv_embed.html (use a similar nomenclature for all test pages)

>+  <title>Embedded Flash video (FLV) test page</title>

Let's call this "FLV video via &lt;embed&gt;" (use a similar nomenclature for all test pages)

>+  <h1>Embedded flash video: sample-flv-video.flv file</h1>

Let's call this "Sample FLV Video:" (use a similar nomenclature for all test pages)

With those changes, I can land this patch.
Attachment #553427 - Flags: review?(anthony.s.hughes) → review-
(Assignee)

Comment 13

6 years ago
Created attachment 553713 [details] [diff] [review]
test pages with flash content v 1.2

Fixed
Attachment #553427 - Attachment is obsolete: true
Attachment #553713 - Flags: review?(anthony.s.hughes)
Comment on attachment 553713 [details] [diff] [review]
test pages with flash content v 1.2

>+  <h1>Sample FLV Video:</h1>
>+  <embed src="sample-flv-video.flv"
>+         height=480 
>+         width=640>

This is just a nit but I see no closing tag for <embed>. If no closing tag is necessary, please use <embed />.

Since you have to update the patch anyway, please switch around width and height so it matches what you have done in the other test pages.

Thanks
Attachment #553713 - Flags: review?(anthony.s.hughes) → review-
(Assignee)

Comment 15

6 years ago
Created attachment 557475 [details] [diff] [review]
Patch v1.3 [checked-in]

Fixed nits
Attachment #553713 - Attachment is obsolete: true
Attachment #557475 - Flags: review?(anthony.s.hughes)
Comment on attachment 557475 [details] [diff] [review]
Patch v1.3 [checked-in]

Looks good. Is this patch for mozmill-tests or litmus-data?
Attachment #557475 - Flags: review?(anthony.s.hughes) → review+
It's for litmus-data. We don't have a firefox sub folder in our test repository.
Comment on attachment 557475 [details] [diff] [review]
Patch v1.3 [checked-in]

Landed:
http://hg.mozilla.org/qa/litmus-data/rev/7e5084fd3762 (default)
Attachment #557475 - Attachment description: test pages with flash content v1.3 → Patch v1.3 [checked-in]
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.