74912 - nsHTMLEditor::InsertCitedQuotation should use fast docFrag insertion when available

Status: RESOLVED INVALID

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

Description

[Joe Francis]

Once 74907 is fixed, make sure editor makes use of that work to increase mail 
reply performance.

Comment 1

[rubydoo123]

setting this to moz0.9.1, and setting depends on 74907

Comment 2

[selmer (gone)]

Joe, why is this moving out of our release?  jst has said he'll be doing his
part next week and we need the whole solution to see any benefit...

Comment 3

[Joe Francis]

Steve, I had 70+ 091 bugs which I mass moved to appease the bug gods.  The 
milestone is meaningless at this moment.  I'll pull bugs back as needed.  
Priority of this effort has not changed.  

If my ws work lands in time, johnny is ready in time, then there is no reason 
this won't happen for 091.

Comment 4

[selmer (gone)]

Updating dependency since 74907 was dup of 35294.

Comment 5

[Joe Francis]

I put patches to do exactly this in bug 39295.  They aren't ready to be checked 
in but they should be fine for allowing testing of layout performance 
improvements to docFrag insertion.

Comment 6

[rubydoo123]

moving to 1.0

Comment 7

[selmer (gone)]

Beth, are you really pushing this out of the release?  Joe's prior comments
indicated this would happen when ready, not that it wasn't going to happen at
all.  What can we do to help keep this in the release?

Comment 8

[Joe Francis]

awaiting review/approval

Comment 9

[Randell Jesup [:jesup] (needinfo me)]

What's up with this bug?  jfrancis: this is awaiting review/approval, but
there's no patch attached, and the bug you mention as having a patch in it
doesn't seem to have anything to do with this one (typo on bug number?)  Is this
still waiting on 35294?  (See comments there too.)

Comment 10

[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 11

[rubydoo123]

removing myself from the cc list

Comment 12

[basic]

we have bug 35295, bug 35294 and bug 39295
which one is this *really* waiting for? (My bet is on 35295 ;-)

Comment 13

[Joe Francis]

It was 35295, lLost in the mists of time.  I went through a big drill on this and 
in the end we didn't land it because of some impacts in clicking on blank lines 
of mailquotes.

However, it is possible that we could still make improvements here by using 
docfrags.  Instead of just hucking the quoted message in unmolested as a docfrag, 
we could try a hybrid approach: still do the newline->br conversion for plaintext 
quotes, but do it inside a docfrag rather than after the insertion into the 
document.  

How high a priority is this?  Do mail folks have new stats on mail compose 
startup performance that indicate this is high priority?

Comment 14

[Joe Francis]

The days of having a half dozen milestones out in front of us to divide bugs 
between seem to be gone, though I dont know why.  Lumping everything together as 
far out as I can.  I'll pull back things that I am working on as I go.

Comment 15

[Joe Francis]

[ushing these out as far as bugzilla will let me.  I'll pull them back as I work
on them.

Comment 16

[Sylvestre Ledru [:Sylvestre]]

Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information

Comment 17

[Makoto Kato [:m_kato]]

old bug, so if this bug is necessary, we should file a new bug since this is mail issue