59577 - Hang during keyboard navigation

Status: VERIFIED FIXED

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

Description

[dprice (gone)]

Browser hangs while trying to arrow down into a <TABLE> that's in a <SPAN>

Comment 1

[dprice (gone)]

Attachment: Attaching test case

Comment 2

[dprice (gone)]

Attachment: attached a fix, r=mjudge

Comment 3

[dprice (gone)]

now all I need to do is tell you how to reproduce it ;)

Simply click to place the cursor in the top left corner, and hold down the down
arrow.  You'll hang when the it tries to move into the table.

Comment 4

[anthonyd]

setting milestone

Comment 5

[Blake Ross]

simon, can you sr?

Comment 6

[anthonyd]

*** Bug 61111 has been marked as a duplicate of this bug. ***

Comment 7

[anthonyd]

change is in my tree, looks good.  r=mjudge, kin
need a= from buster.

can you check it out for me buster and give me your a=?

thanks,
anthonyd

Comment 8

[anthonyd]

waiting on approval to check in.

anthonyd

Comment 9

[buster]

I'm on vacation, but fortunately checking my mail :)
waterson, can you sr for me please?

Comment 10

[Chris Waterson]

I'd prefer if you tested that ``blockFrame != aPos->mResultFrame'' in the
enclosing if statement (where you test aPos->mResultFrame). There doesn't appear
to be any need to execute the ``else'' clause where you force result to be NS_OK...

Comment 11

[anthonyd]

Attachment: new patch with watersons recommended change, please re-review

Comment 12

[anthonyd]

setting to p1.
waiting on re-review from waterson.

anthonyd.

Comment 13

[Keyser Sose]

*** Bug 61230 has been marked as a duplicate of this bug. ***

Comment 14

[anthonyd]

waterson, buster,

can i get a kwik review with watersons change?

anthonyd

Comment 15

[buster]

sr=buster

Comment 16

[Blake Ross]

So can we get this in?

Comment 17

[anthonyd]

fix checked in.

anthonyd

Comment 18

[rubydoo123]

catching up on verifications