19340 - Copy image location not null terminating string?

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[Joe Pranevich]

If you go to a page, http://50.lycos.com

Right click on the big "Lycos 50" image and select "Copy Image Location"


Select the URL bar at the top and type Ctrl-V to paste the URL of the image and
I get:

http://lygo.com/ly/c/td/50_type.gif¨?`1Qtext/plain


The first part is obviously correct. You can see however that it is not properly
terminated and other data can be seen.

Comment 1

[Joe Pranevich]

I can verify that this also happens for "Copy Link Location" menu items. This is
on today's (11/19/99) daily build.

Comment 2

[leger]

beppe, would this cut and paste action be yours?

Comment 3

[Hugh Kennedy]

This is also present in my Solaris 2.6 build of M11.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Looks like leger didn't hit "reassign".  Doing so, since that would allow
[e-mail] to read the comment addressed to her.

Comment 5

[rubydoo123]

yes it is ours, assigning to buster

Comment 6

[buster]

cc'ing some folks who own related code.  do any of you think this bug is yours?
I won't have time to investigate for a couple of days.

However, first things first:  I can't reproduce the problem because I get a
javascript error from nsContentMenu.js when I right-click on an image and select
"Copy Image Location"

JavaScript Error: TypeError: this.imageURL is not a function
URL: chrome://navigator/content/nsContextMenu.js
LineNo: 293

It looks like imageURL is used both as an attribute and a function.  I don't
know javascript very well, but this seems pretty odd.  Throwing over to bill to
fix this problem first.

Comment 7

[don]

Move to M15.

Comment 8

[mbp]

This is also present in M12-1 from Debian.

Comment 9

[don]

Move to M16 for now ...

Comment 10

[don]

Move to M21 target milestone.

Comment 11

[R.K.Aa.]

context menu over an image in mailnews is completely non-functional, apart from
"select all" which selects text, not image.

If i click the image, the "copy" item is later enabled in context menu.
If i then click "copy" - the alt-text is copied.

"View Image", "Save image as" and "Copy Image location" all do nothing, but
spawn this in console on linux 2000122306:

... entering nsContextMenu constructor in nsContextMenu.js
... leaving nsContextMenu constructor in nsContextMenu.js
JavaScript error:
 line 0: contextMenu has no properties

Comment 12

[R.K.Aa.]

regarding my comment above: that was fixed in bug 62964

Comment 13

[Bill Law]

Fixed a while back.

Comment 14

[sujay]

works now..the whole URL is pasted in the field...

verified in 1/24 build.