34477 - Editor embedding tracking bug

Status: VERIFIED DUPLICATE of bug 157128

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

Description

[rusty.lynch]

Not too long ago, the editor was not required for simple web browsing. Recently
a dependancy has been added for the editor in order to type into form input
fields.  For running embedded mozilla apps on small devices this is a real
limitation since the editor library, libender.so, is very large.

If an embeded app doesn't want to use the editor like composer or as a mail/news
client, then the app shouldn't have to carry around the extra library luggage.

Has anyone else thought about this?

Comment 1

[rubydoo123]

We do need to provide a skeleton of the editor library for samller devices, 
providing mozilla with a subset of the code. Eliminating HTML specific code, 
possibly undo/redo, etc. Assigning to Kin to spearhead the investigation and cc 
remainder of the team.

Comment 2

[Kathleen :Brade]

Can someone clarify this point for me:

I assume we'll still need some "HTML" editing stuff so that we can show IME input 
highlighting or spellcheck highlighting or ?  True?

Comment 3

[rusty.lynch]

I can speak from my needs as an embedded mozilla developer on a resource limited
device.  I am not using any of the IME functionality in the editor library.  So,
for me all I need is the ability to enter/delete text in text area's and input 
fields.

I posted a message on the embedding and small-devices news groups to try to get 
input from other developers with simular constraints.

Comment 4

[Simon Fraser [no longer active]]

Currently, text widgets are implemented using composer on top of an HTML 
subdocument, and typing in such a widget will involve much of the code in the 
editor DLL. It would be non-trivial to split up the editor library so that a 
smaller version of the library could be used for widget-only apps.

Comment 5

[kinmoz]

Accepting bug.

Comment 6

[rubydoo123]

assigning this bug to sfraser as the embedding tracking bug, cc kin as well

Comment 7

[rubydoo123]

updated summary to reflect the new standing of this bug

Comment 8

[rubydoo123]

added embed keyword

Comment 9

[Simon Fraser [no longer active]]

Mo zill a

Comment 10

[Simon Fraser [no longer active]]

Adding dependency on the font list bug 62035.

Comment 11

[rubydoo123]

reassigning to me for tracking purpose, will be adding dependency on task bugs

Comment 12

[Simon Fraser [no longer active]]

Add skin switching bug 43350.

Comment 13

[rubydoo123]

moving to mozilla1.0

Comment 14

[rubydoo123]

removing 26288 from the tracking

Comment 15

[Kathleen :Brade]

-->chofmann

Comment 16

[chris hofmann]

add valeski

Comment 17

[saari (gone)]

giving this to mjudge as a starting point for tracking the editor embedding work

Comment 18

[Asa Dotzler [:asa]]

Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1 
(you can query for this string to delete spam or retrieve the list of bugs I've 
moved)

Comment 19

[Asa Dotzler [:asa]]

don't move bugs that are in the 1.0 dependency tree. sorry.

Comment 20

[rubydoo123]

removing myself from the cc list

Comment 21

[Dave Barrowman]

No need to mark tracking bugs with 'topembed'.

Comment 22

[Charles Manske]

Ummm. What is this bug for? It seems the real embedding tracking bug is
bug 157128?

Comment 23

[Kathleen :Brade]

This is a duplicate of bug 157128 (or vice versa) but the following are not on
bug 157128:

66343: not sure if this is necessary; what is the state of text services?
66344: I would argue this is not a dependency at all as long as it is possible
for someone to be able to write an extension.

41301:  iframe is editable instead of page
70229:  embedding api todo list (is this now obsolete?)
103889: layout group tracking bug (is that bug obsolete or being actively used?)

This bug is missing these bugs (possibly more?):
79895:  editortypes
121648: editorshell removal
133598: <editor> tag in xul (cleanup)
137173: embedding fileset is incomplete
157097: command nodes hold state
157098: command nodes need to trigger update
157100: sample Macintosh application
157101: sample Linux application
157104: use of nsIEditingSession in Composer/Mail/etc
157105: "make editable" api needs to be recursive
157106: remove docshell dependency on editor
157109: need api to iterate over all editors in a session
157111: make JS commands accessible to embedders
157114: remove code that points to xul command nodes
157116: doCommand should pass nsIEditor not nsIEditorShell as param
157117: sandboxing of editor

Comment 24

[Charles Manske]

Saari, mjudge: Seems like we should dup this to bug 157128 and add bug 41301 and 
bug 70229 (and maybe bug 103889?) as blockers to that bug?

Comment 25

[Marek Z. Jeziorek]

batch: adding topembed per Gecko2 document
http://rocknroll.mcom.com/users/marek/publish/Gecko/Gecko2Tasks.html

Comment 26

[saari (gone)]

*** This bug has been marked as a duplicate of 157128 ***

Comment 27

[sujay]

verified.

Comment 28

[Michael Buckland]

Bulk adding topembed keyword.  Gecko/embedding needed.