851485 - After reload anchor link not respected

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[rdz_bug]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:22.0) Gecko/20130314 Firefox/22.0
Build ID: 20130314030914

Steps to reproduce:

1. Open http://www.neogaf.com/forum/showthread.php?p=50224464#post50224464
2. Scroll to any other position (except top, see Bug 849219) 
3. Reload the page
4. Go to be bottom and click on "RETURN TO TOP"


Actual results:

Nothing.


Expected results:

Firefox should have scrolled the page to the top.

Comment 1

[Mats Palmgren (inactive)]

Is this a regression from bug 842853?

Comment 2

[Boris Zbarsky [:bzbarsky]]

Sure looks like it to me, since it regressed in the right timeframe.

We should presumably only ignore one GoToAnchor call?

Comment 3

[Mats Palmgren (inactive)]

Yeah, I actually did that in the patch I submitted for review, but I couldn't
really explain why it was needed so we decided to take it out in the end.

Comment 4

[Mats Palmgren (inactive)]

Attachment: fix

I guess we should try to fix this ASAP.  I'll write a test later.
https://tbpl.mozilla.org/?tree=Try&rev=de89a7f1bcfc

Comment 5

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 725400 [details] [diff] [review]
fix

r=me

Comment 6

[Mats Palmgren (inactive)]

Comment on attachment 725400 [details] [diff] [review]
fix

Thanks, I'll land it as soon as inbound opens and Try is green...

Comment 7

[Mats Palmgren (inactive)]

Attachment: mochitest

Comment 8

[Mats Palmgren (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6ce79dc8036b
https://hg.mozilla.org/integration/mozilla-inbound/rev/501804a40144

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/6ce79dc8036b
https://hg.mozilla.org/mozilla-central/rev/501804a40144