125372 - forward arrow not active where it should be.

Status: RESOLVED FIXED

(Core :: DOM: Navigation, defect)

Description

[Miloslaw Smyk]

Mozilla 2002020611 (Linux/i686).

After clicking certain sequences on framed site, forward arrow of session
history misbehaves.

Steps to recreate problem:

1. Visit above URL.
2. Click java.lang in upper-left frame.
3. Click Math in lower-left frame.
4. Click a link to "E" in Field Summary table of main frame.
5. Move back in session history once (left arrow).
6. Observe that forward arrow (right) is not active.
7. Move back again, observe that forward arrow is now active and contains TWO items.

Step 6 is wrong, the arrow should be active.

Repeatibility: always.

Comment 1

[John Levon]

wfm linux checkout finish: Thu Feb 14 16:50:59 GMT 2002

Comment 2

[Miloslaw Smyk]

Most interesting. With 2002021606/linux-i686 the problem is still there. Are you
sure you followed the "steps to recreate" correctly? Some other people on
#mozilla also found this bug to happen to them. Can someone else verify?

Comment 3

[shrirang khanzode]

same problem seen on http://slip/shrir/License.pdf (sorry folks, this is an 
internal url). I see the same above mentioned problem, FWD button does not 
highlight the very first time I try this url. You can try accessing any pdf 
online directly and reproduce this issue.

Comment 4

[Miloslaw Smyk]

You mean, with PDF plugin active?

Comment 5

[Radha on family leave (not reading bugmail)]

Attachment: patch to docshell

Comment 6

[Adam Lock]

Comment on attachment 82107 [details] [diff] [review]
patch to docshell

Radha, can you submit the patch again? This one is a dud.

Comment 7

[Radha on family leave (not reading bugmail)]

Attachment: This is a better one.

Comment 8

[Adam Lock]

Comment on attachment 82126 [details] [diff] [review]
This is a better one.

r=adamlock

Comment 9

[Alec Flett]

Comment on attachment 82126 [details] [diff] [review]
This is a better one.

all these tweaks make me think that at some point we need to take a step back
and figure out why there are so many edge cases, and re-architect session
history.

in the mean time,
sr=alecf

Comment 10

[Radha on family leave (not reading bugmail)]

Fixed in the trunk. 

Comment 11

[Radha on family leave (not reading bugmail)]

ADT,  This fix has been in the trunk for a while. I would like it checked into
the branch as well. 

Comment 12

[Jaime Rodriguez, Jr.]

greggl/radha - How visible, or prevelent do we believe this will be out in the
wild? Do we really believe a lot of users will hit this case?

Comment 13

[Syd Logan]

adt1.0.1-

Comment 14

[Radha on family leave (not reading bugmail)]

Renominating for adt1.0.1. There is a fairly good chance of users hitting this
one, especially in sites using anchors extensively like this java documentation
site.   This is the third bug I have had  on this particular site.  The changes
are not massive either.  Please reconsider it for Mach V

Comment 15

[Miloslaw Smyk]

I support Radha on this one. The patch is simple and really useful. I've been
using it for a month with no problems at all.

Comment 16

[Jaime Rodriguez, Jr.]

this is still a minus. adt1.0.1- per adt