80116 - Need "content analyzing" APIs

Status: RESOLVED WONTFIX

(Core :: DOM: Core & HTML, defect)

Description

[Dan Rosen]

Several places in the code need to figure out difficult things about "what sort
of content I'm looking at" -- for example, to figure out whether a content node
is within a link, you need to determine whether you or any of your ancestors are
an HTML <a>, an imagemap <area> with |href| attributes, an XLink, etc.

Code which needs this includes UI code for context menus (eg., you only want to
show the "copy link location" context menu item, and enable that command, if you
clicked on a link), as well as drag and drop (different content behaves
differently in DnD).

We need a single "content analyzing" API to determine useful information about
content nodes.

For reference, see code currently in nsContextMenu.js, nsPresShell.cpp, etc.

Comment 1

[Dan Rosen]

<spam>

Netscape 6.0 went out with Copy Link Location working with imagemaps. The next
Netscape RTM should not go out with it broken. Bug 80116 and bug 78021 should be
fixed for 0.9.2.

This should be a simple task of filling in a new XPCOM API with Bill Law's
content-sniffing code in nsContextMenu.js. (More content analyzing APIs should
be filled in post-RTM as needed).

</spam>

Comment 2

[Dan Rosen]

Marked nsBranch on behalf of blocked bug 78021. This should be minimal, though,
in the spirit of low-risk and high-impact. For the branch, this will be one or
two methods on a new interface which will support only that functionality needed
to detect imagemaps.

Comment 3

[Dan Rosen]

[spam] -> 0.9.5

Comment 4

[Dan Rosen]

[spam] [e-mail]'s bugs subject to redistribution by chofmann. R!

Comment 5

[Dan Rosen]

oops, this is still mine.

Comment 6

[Boris Zbarsky [:bzbarsky]]

jst, peterv, sicking, what do you think?

Comment 7

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

I'm not entierly sure what all is requested here. Just a way to check if a node
is a link and what url it has?

Comment 8

[Johnny Stenback (:jst)]

I think I don't care. Reopen if you disagree and want to take this somewhere...