162982 - Lacking default tooltip impl for embeddors

Status: RESOLVED WONTFIX

(Core Graveyard :: Embedding: APIs, defect)

Description

[Aaron Leventhal]

Embeddors have to impl nsITooltipListener, we don't provide a default impl.

I think this is unfriendly and creating extra work for most embeddors; we should
let them override the impl, but not force them to do something we should provide.

Comment 1

[Adam Lock]

The problem with providing a default implementation is that it is impossible to
do in a cross platform manner. It might be possible to work on a Win32 version.

Comment 2

[Aaron Leventhal]

What makes it impossible to do it XP?
Geecko renders everything else XP, why not tooltips?

Comment 3

[Adam Lock]

If the embedder wants an XP toolkit they should load some chrome into their
browser object with the appropriate tooltip js event handler. This can be done
right now.

The nsIToolTipListener is for the benefit of embedders who wish to implement a
tooltip behaviour in C++. Perhaps we could provide an XP version of this, but it
might be tricky considering the different paths that a tooltip would be invoked
from via chrome and via our listener class.

Comment 4

[Adam Lock]

Marking WONTFIX.

Embedders now have a sample tooltip implementation in mfcembed to reference. 

http://lxr.mozilla.org/seamonkey/source/embedding/tests/mfcembed/BrowserToolTip.h
http://lxr.mozilla.org/seamonkey/source/embedding/tests/mfcembed/BrowserToolTip.cpp

Comment 5

[Jan Kuemmel]

> If the embedder wants an XP toolkit they should load some chrome into their
> browser object with the appropriate tooltip js event handler. This can be done
> right now.

Can you give a hint how to load some chrome into a nsIWebBrowser, please?
Sure, I can call nsIWebNavigation.loadURI() with a chrome uri, but I also need to load a webpage.