166465 - Document the type of string inputs to Editor Embedding APIs

Status: RESOLVED WONTFIX

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

Description

[David Epstein]

public APIs are nsIEditingSession, nsICommandManager, and nsICommandParams. In
these idls, there are several methods with string parameter inputs. Need
comments to explain what type of string formats they are (i.e. ASCII, UCS2,
CString, ...).

All these have "string" inputs:

nsICommandManager: addCommandObserver(), removeCommandObserver(),
isCommandSupported(), isCommandEnabled(), getCommandState(), doCommand(). 

nsICommandParams:  getValueType(), get/setBooleanValue(), get/setLongValue(),
get/setDoubleValue(), get/setStringValue(), get/setCStringValue(),
get/setISupportsValue(), removeValue()

Comment 1

[Kathleen :Brade]

Do we want to change the API to take nsA(C)String?
We also need to document for each command whether the params are cstrings or
wstrings etc.

Comment 2

[Michael Dunn]

What is the next step for this bug?

Comment 3

[mjudge]

i think this is a living bug since we need to continuously fix the strings we
support. the existing document is not on mozilla yet.  I will see about posting
this soonest.

Comment 4

[chris hofmann]

unsetting target milestone

Comment 5

[David Epstein]

this bug needs to be reassigned

Comment 6

[Kathleen :Brade]

-->editor

Comment 7

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, >=3 years untouched DOM/Storage bug's priority.

If you have reason to believe this is wrong, please write a comment and ni :jstutte.

Comment 8

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

I think that we have no plan to support embedding now.