582310 - Create a local suite of 8 unique web pages

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[u279076]

This is a tracking bug for creating a suite of 8 unique local web pages for use in various Mozmill tests.  The candidate pages are listed below.  The local copy should be stripped down of any JS and CSS.  It should only have text content and render an image common to all pages (ie. the header image)

Remote URLs:
http://www.mozilla.org
http://www.mozilla.org/about/mission.html
http://www.mozilla.org/community/
http://www.mozilla.org/contribute/
http://www.mozilla.org/projects/
http://www.mozilla.org/about/governance.html
http://www.mozilla.org/about/organizations.html
http://www.mozilla.org/grants/

Please place the files here...

litmus-data:
firefox/layout/mozilla.html
firefox/layout/mozilla_mission.html
firefox/layout/mozilla_community.html
firefox/layout/mozilla_contribute.html
firefox/layout/mozilla_projects.html
firefox/layout/mozilla_governance.html
firefox/layout/mozilla_organizations.html
firefox/layout/mozilla_grants.html

mozmill-data:
firefox/test-files/layout/mozilla.html
firefox/test-files/layout/mozilla_mission.html
firefox/test-files/layout/mozilla_community.html
firefox/test-files/layout/mozilla_contribute.html
firefox/test-files/layout/mozilla_projects.html
firefox/test-files/layout/mozilla_governance.html
firefox/test-files/layout/mozilla_organizations.html
firefox/test-files/layout/mozilla_grants.html

Comment 1

[u279076]

Attachment: Proof of Concept: www.mozilla.org

Here is a proof of concept for the local copy of www.mozilla.org.  It consists of the following files:

mozilla.html -> 4 kb
favicon.ico -> 4 kb
bg_body_1.jpg -> 12 kb

We can reduce the physical dimensions of the image.  I can probably get this down to 3 kb.

Overall, Henrik, can you provide me any feedback on this proof of concept?  I'd like to get your feedback before proceeding with the other test pages.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

As long as we have a couple of elements in that page we can access via id, link, name, or else it looks good. Shouldn't a PNG be smaller here for the b/w dino head?

Comment 3

[u279076]

(In reply to comment #2)
> As long as we have a couple of elements in that page we can access via id,
> link, name, or else it looks good. Shouldn't a PNG be smaller here for the b/w
> dino head?

The PNG is 5x bigger (61 kb vs 12 kb)

Comment 4

[u279076]

Attachment: Patch v1 (default)

This patch creates a minimized clone of the 8 Mozilla.org pages mentioned in the original comment.

Aaron, please check this for any errors then flag Henrik for additional review, thanks.

Comment 5

[Aaron Train [:aaronmt]]

Comment on attachment 461345 [details] [diff] [review]
Patch v1 (default)

Read through all pages. Checked all links, matching opening and close blocks and general formatting. Couple of small nits:

mozilla_contribute - You have a indented paragraph tag that is a couple tabs in

mozilla_grants - Perhaps we may want id's on the list items to match the consistency of the other pages

mozilla_mission - To keep consistency lets use 'organizations' or 'org' for all files with the similar id 

mozilla.html - To keep consistency lets use 'organizations' or 'org' for all files with similar id

Comment 6

[Aaron Train [:aaronmt]]

(In reply to comment #5) 
> mozilla_grants - Perhaps we may want id's on the list items to match the
> consistency of the other pages
> 

mozilla_governance, sorry

Comment 7

[u279076]

Attachment: Patch v2 (litmus-data)

Comment 8

[u279076]

Comment on attachment 462494 [details] [diff] [review]
Patch v2 (litmus-data)

Henrik, can you give this a quick once-over before I commit it?

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Comment on attachment 462494 [details] [diff] [review]
Patch v2 (litmus-data)

diff --git a/firefox/layout/.DS_Store b/firefox/layout/.DS_Store
new file mode 100644

Please remove that file from the patch. Don't we list it in .hgignore? Normally it shouldn't get added.

>+    <a href="mozilla.html">
>+        <img id="mozilla_logo" name="mozilla_logo" src="logo.jpg" />
>+    </a>

Would be nice if you could give this or any other a node a name too. 

>+    <link rel="shortcut icon" type="image/ico" href="favicon.ico" />

We should check-in different favicons for those pages so we can differentiate between those. Please also put additional files under test-files/images.

Otherwise it looks good.

Comment 10

[u279076]

Attachment: Patch v3 (litmus-data)

Comment 11

[u279076]

Landed on litmus-data:
http://hg.mozilla.org/qa/litmus-data/rev/d8de398be9fa

Comment 12

[u279076]

Attachment: Patch v2 (default)

Comment 13

[Aaron Train [:aaronmt]]

Comment on attachment 462607 [details] [diff] [review]
Patch v2 (default)

(In reply to comment #12)
> Created attachment 462607 [details] [diff] [review]
> Patch v2 (default)
>+            <ul id="areas_of_interest">
>+                <li id="browser_choice">
>+                    <h5>Web Browser Choice</h5> 
>+					<p>

r+, but please adjust that indented paragraph

Comment 14

[u279076]

Attachment: Patch v3 (default)

I'm not sure why the <p> is showing up indented so much in the bugzilla diff.  It is not indented this much in the raw html file.  In fact it is inline with the <h5>.

Lets see if this version of the patch fixes it.

Comment 15

[u279076]

(In reply to comment #14)
> Created attachment 462672 [details] [diff] [review]
> Patch v3 (default)
>
> Lets see if this version of the patch fixes it.

That seems to have got it.  Not sure what happened there.  Anyway, I'll check-in this patch in the morning.

Comment 16

[u279076]

Landed:
http://hg.mozilla.org/qa/mozmill-tests/rev/64c3bcad8869 [default]
http://hg.mozilla.org/qa/mozmill-tests/rev/21282e76b983 [mozilla1.9.2]
http://hg.mozilla.org/qa/mozmill-tests/rev/450cdf8ce866 [mozilla1.9.1]

Comment 17

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Move of Mozmill Test related project bugs to newly created components. You can
filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.