92188 - [pp]Default plug dialog displays when viewing flash animation

Status: VERIFIED FIXED

(Core Graveyard :: Plug-ins, defect)

Description

[Chris Petersen]

Build: 2001072403
Platform: Mac OS 9.1
Expected Results: No plug-in dialog should appear
What I get: Flash animation is rendered but default plug-in dialog appears.

Steps to reproduce:

1) Go to http://www.hallmark.com

2) Click on Free e-cards icon 

3) Click on any of the links that open a flash-created card.

4) Notice embedded flash object is render by plug-in .

Comment 1

[Chris Petersen]

With Mark Attinasi's help, looks like they are creating a embed tag with no
source which is causing the default plug-in to activate.

Comment 2

[Chris Petersen]

Attachment: simpler test case from Hallmark's page

Comment 3

[Chris Petersen]

The HTML page is created the same way when viewed on both windows and mac
builds. However, the windows build doesn't have a problem with the embed no
source tag. No default plug-in is displayed on windows.

Comment 4

[Chris Petersen]

Embed tag reference from the HTML sample


<!-- Embed tag with no source -->
<embed SRC="" hidden=true autostart="TRUE" loop="3" WIDTH="0" HEIGHT="0">

Comment 5

[av (gone)]

This is likely to be fixed with my today's check in. Please try.

Comment 6

[av (gone)]

Or apply http://bugzilla.mozilla.org/showattachment.cgi?attach_id=43437

Comment 7

[Chris Petersen]

Using the 2001-07-24-18 branch build, the problem is still occuring on Mac.

Comment 8

[Peter Lubczynski]

Mac only? ---> peterl

Comment 9

[Peter Lubczynski]

This is FIXED on the trunk 2001072508 on Mac.

This bug does not have a PDT+ so it will not be fixed on the branch.

Comment 10

[Tracy Walker [:tracy]]

verified fixed on linux commercial trunk build 2001-07-25-08-trunk

Comment 11

[av (gone)]

Peter, did you fix it?

Comment 12

[Peter Lubczynski]

I did not fix this, it was fixed with bug 91140.

Comment 13

[av (gone)]

That's what I thought, but 91140 _is_ PDT+, so I would expect this one to work 
on both branch and the trunk.

Comment 14

[Peter Lubczynski]

My mistake, it was bug 90152 that fixed this because of it's mime-type checks.