199337 - [minimo] make save as, nsHtmlContent serializer configurable

Status: RESOLVED FIXED

(Minimo Graveyard :: General, defect, P2)

Description

[Cathleen]

Comment 1

[Simon Fraser [no longer active]]

More info?

Comment 2

[Alec Flett]

sorry, so we want a switch to disable HTML content serialization for minimo,
since we don't do HTML composition... the footprint win is unknown right now,
but this bug will track the investigation.

the first place to start is in content/base/src/nsHTMLContentSerializer.cpp, and
see what else that requires.

Comment 3

[Simon Fraser [no longer active]]

Don't we use it for Copy and the implementations of various editor API calls?

Comment 4

[Kathleen :Brade]

I was just going to comment what smfr asked...
We need this to copy in the browser.

Comment 5

[Alec Flett]

excellent - more dependent code to remove :)

in any case, this is for minimo, i.e. a very small engine that might not even
need copy... as I've said in other bugs, the idea is not actually to remove it,
its to make it configurable so you can turn it off if your embedded application
doesn't need it.

Comment 6

[Doug Turner (:dougt)]

i will take a crack at this.

Comment 7

[Doug Turner (:dougt)]

Attachment: patch v.1

adds a MOZ_SERIALIZE default-on option to allow us to conditionally remove
serializers.  This option is only known to work along with
plaintext-editor-only.

Comment 8

[Doug Turner (:dougt)]

This patch will break copy/paste and I am not sure anyone wants that.  Maybe
there is a bit less we can cut while preserving common clipboard functions but I
don't think that will yield any footprint savings.

Comment 9

[Doug Turner (:dougt)]

moving minimo bugs to the new bugzilla product.

Comment 10

[André Pedralho]

It has been solved by 10LE Team!