81210 - Image insert not working

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect)

Description

[Tracy Walker [:tracy]]

seen on commercial builds:

windows 2001-05-16-06-trunk
linux 2001-05-16-05-trunk
mac 2001-05-16-04-trunk

-open composer
-click on Image icon (or Insert | Image)
-from the the dialogue, "Choose file", select an image to insert (I tried jpg 
and gif images)
-type in some alternate text, then clcik okay.

the image is not inserted into the composer window.

Comment 1

[Chris McAfee]

why is this a blocker?  Blocker means people can't get work done; ok,
don't insert images.  I think this should get downgraded.

Comment 2

[sujay]

I see the problem..

HTML is inserted in the compose window.

but when you save the fil and hit Browse...you can see the image in the
browse window.

Comment 3

[J.J. Enser]

Is this really a blocker and can it be reduced to 'critical' ?
Tree opening todays depends on this.

Comment 4

[Chris McAfee]

Agree with jj, bumping down to critical.
Talk to me or jj, or comment here to change this.

Comment 5

[kinmoz]

The editor is actually inserting the image into the document, but whatever loads 
the image on the layout side is failing. Here is the warning I'm seeing in the 
console:

The link to file:///Y:/mozilla/dist/WIN32_D.OBJ/bin/res/samples/raptor.jpg was b
locked by the security manager.
Remote content may not link to local content.
/content/EdImageProps.xul

Remote images (http, etc) load fine in composer. This bug may be related to the 
XPCDOM landing or security changes?

Comment 6

[Kevin McCluskey (gone)]

I have a 5/15/2001 9:15pm build on WINNT and it works correctly. Regression must
have happened after that.

Comment 7

[Kevin McCluskey (gone)]

In my 5/15/2001 9:15 build I can add images referenced through a http: URL, but
not file URL's.

Inserting this image works:
http://www.netscape.com/ex/shak/images/home2001a/hp_icon_netscapelogo.gif

Inserting this image fails:
file:///S:/mozilla/dist/WIN32_D.OBJ/bin/res/samples/raptor.jpg

Comment 8

[Mitchell Stoltz (not reading bugmail)]

My fault. I turned on a security check for image loading yesterday. I'll take a
look and back it out if necessary.

Comment 9

[Charles Manske]

So can this be fixed soon please? It is blocking my work.

Comment 10

[Mitchell Stoltz (not reading bugmail)]

Fixed (backed out image loading security check).

Comment 11

[sujay]

verified in 5/22 build.