45445 - Assertion in GetPointFromOffset when doing delete-word past a line break

Status: VERIFIED WORKSFORME

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

Description

[Akkana Peck]

Paste some lines into the editor.  Set the caret a few words into one of the
lines (not the first one) and repeatedly do delete-word-backward.  (On Unix,
this is wired to control-W.  Not sure if Windows has an equivalent; you may have
to add an entry to editorBindings.xul or platformEditorBindings.xul.)

When it gets to the beginning of the line and has to wrap back to the previous
line, there's an assert:

###!!! ASSERTION: invalid offset passed to GetPointFromOffset: '0', file
nsTextFrame.cpp, line 3052
###!!! Break: at file nsTextFrame.cpp, line 3052

though the delete seems to work fine, so maybe the assertion isn't needed.

Comment 1

[Akkana Peck]

Will look at it in the M18 timeframe, and at least evaluate whether the assert
is important.

Comment 2

[Akkana Peck]

This magically got fixed.

Comment 3

[Blake Ross]

*SPAM*: Changing the QA contact of all open/resolved Selection bugs from 
[e-mail] to [e-mail].  After the many great years of service 
Eli has given to Mozilla, it's time for him to move on; he has accepted a 
position at Eazel.  We'll be sad to see him go, and I'll do my best to fill his 
spot...

Comment 4

[Akkana Peck]

It's baaack.

Comment 5

[kinmoz]

Moving to M19 as per [e-mail].

Comment 6

[rubydoo123]

setting to future

Comment 7

[Akkana Peck]

Anthony is taking over selection bugs.

Comment 8

[anthonyd]

moving to 0.9 to at least investigate this in case its important.

anthonyd

Comment 9

[rubydoo123]

asserts need to be cleared up, leaving in moz0.9 and leaving at p3

Comment 10

[anthonyd]

setting to mozilla 0.9.1

Comment 11

[Kathleen :Brade]

Akkana--is this assert still present?

moving to mozilla0.9.2

Comment 12

[Akkana Peck]

It's gone in today's build.

Comment 13

[rubydoo123]

marking verified per Akkana's comment