40262 - Percent height on EMBED gets turned into pixel size if missing quote

Status: VERIFIED DUPLICATE of bug 47755

(Tech Evangelism Graveyard :: English US, defect, P3)

Description

[Walter Ebeling]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; WinNT4.0; en-US; m16) Gecko/20000522
BuildID:    2000052208

The Flash animations on the right side of the homepage are way too
small.Everything else works correctly. Check out Netscape for a correct display.

Reproducible: Always
Steps to Reproduce:
1.Load www.signal-iduna.de
2.
3.

Expected Results:  The size of the animation boxes should be approximately 4 to
5 times wider and higher. The page seems to be optimized for 1024x768
resolution, if you reduce the window size to 640x480, the middle frame will have
scrollbars and the correct width of the animated boxes will become visible.

Comment 1

[WD]

It appears that the Flash object is ignoring the height=100% and width=100%
From the source:
<embed src="rechts/blanko.swf" quality=high width="100%" height="100% type= ....

Notice: There is no closing '"' for the width=100% specification. I'm not sure
if this is causing the problem, or if it's an issue with the Flash.
In Netscape or Internet Explorer the Flash animation takes up the whole screen.

Comment 2

[karnaze (gone)]

Reassigning to Andrei.

Comment 3

[Chris Petersen]

Yes, This problem is related to the missing end quote for height. If you download 
the sample url and fix the html file, the flash animation is played at correct 
size in NS6.

Comment 4

[av (gone)]

Should we fix it then ignoring absence of the quotation mark?

Comment 5

[Peter Lubczynski]

The testcase for this bug is gone. Also, isn't this a parser bug?

Comment 6

[karnaze (gone)]

Reassigning to peterl and moving to m0.9.1

Comment 7

[karnaze (gone)]

Reassigning to peterl.

Comment 8

[Peter Lubczynski]

Petersen/Shrir, 

Can you try this again. Adding qawanted

Comment 9

[shrirang khanzode]

i don't see the page at all. Something in german is printed out (file not found 
or something..)

Comment 10

[Peter Lubczynski]

If you have time, can you whip up a testcase so this can be closed out, moved, 
or fixed. Thanks!

Comment 11

[karnaze (gone)]

Moving to m0.9

Comment 12

[Peter Lubczynski]

Attachment: simple testcase that fails

Comment 13

[Peter Lubczynski]

Attachment: simple testcase that works

Comment 14

[Peter Lubczynski]

I just checked in viewer and it looks like content thinks that the percentage 
height should be a pixel height instead. Over to parser.....


old Summary: Flash Animation in frames are too small

Comment 15

[harishd]

Peter, parser just collects attributes. It does not process them. However, the
root cause of the problem seems to be the missing quote. I don't think we should
even fix it or atleast this is not serious enough to get in for m0.9. Inclined
to mark WONTFIX. 

May be we can evangelise. For now futuring the bug.

Comment 16

[Peter Lubczynski]

Can the parser just catch a %<space> in an attribute and make it equal to a 
%"<space>   dunno? What's the spec say?

Comment 17

[Andreas M. "Clarence" Schneider]

This is a dupe of bug 47755 and there is no need to evangelize (URL gone).
47755 is WONTFIX, but we have an assigned bug about the same issue: bug 44485.

Comment 18

[Andreas M. "Clarence" Schneider]

*** This bug has been marked as a duplicate of 47755 ***

Comment 19

[Bob Clary [:bc] (inactive)]

All Evangelism Bugs are now in the Product Tech Evangelism. See bug 86997 for
details.

Comment 20

[Bob Clary [:bc] (inactive)]

this isn't really a dupe but the site is gone.