170439 - Document the string type for nsIWebNav->LoadURI()

Status: VERIFIED WONTFIX

(Core Graveyard :: Embedding: APIs, defect)

Description

[David Epstein]

Since we use NS_ConvertASCIItoUCS2() in LoadURI, should document the string uri
input as UCS2.

Comment 1

[Boris Zbarsky [:bzbarsky]]

nsIWebNav does not seem to have a LoadURI function on it.  Could you be more
specific about which function you think needs documentation?

This is a bug in the embedding test harness, right?  Not in nsIWebNavigation or
nsDocShell per se?

Comment 2

[David Epstein]

No I was referring to nsIWebNavigation->LoadURI(), not in any test driver. See
the above url for nsIWebNavigation.idl. The uri string input, first parameter,
needs to be documented. I suggested UCS2 since NS_ConvertASCIItoUCS2() is used
in embedding apps.

Comment 3

[Boris Zbarsky [:bzbarsky]]

It's a "wstring".  That's UCS2 (UTF-16, actually, but that's a superset of UCS2)
by definition.

Comment 4

[Darin Fisher]

mass reassign of various rpotts bugs to me

Comment 5

[Christian :Biesinger (don't email me, ping me on IRC)]

wstring is always UTF-16 in mozilla. this needs not be documented for each
interface separately.

Comment 6

[Darin Fisher]

v