Closed Bug 851485 Opened 11 years ago Closed 11 years ago

After reload anchor link not respected

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Version:
22 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla22
Tracking Flags:
Tracking Status
firefox21 --- unaffected
firefox22 + fixed

People

(Reporter: rodrigozeh, Assigned: MatsPalmgren_bugz)

References

Details

(Keywords: regression)

Attachments

(2 files)

fix
3.12 KB, patch
bzbarsky
: review+
Details | Diff | Splinter Review
mochitest
2.89 KB, patch
Details | Diff | Splinter Review 
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.
Component: Untriaged → Layout
Product: Firefox → Core
Is this a regression from bug 842853?
Sure looks like it to me, since it regressed in the right timeframe.

We should presumably only ignore one GoToAnchor call?
Blocks: 842853
tracking-firefox22: --- → ?
Keywords: regression
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.
Assignee: nobody → matspal
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
OS: Windows 7 → All
Hardware: x86_64 → All
Attached patch fix — — Splinter Review 
I guess we should try to fix this ASAP.  I'll write a test later.
https://tbpl.mozilla.org/?tree=Try&rev=de89a7f1bcfc
Attachment #725400 - Flags: review?(roc)
Attachment #725400 - Flags: review?(bzbarsky)
Comment on attachment 725400 [details] [diff] [review]
fix

r=me
Attachment #725400 - Flags: review?(bzbarsky) → review+
Comment on attachment 725400 [details] [diff] [review]
fix

Thanks, I'll land it as soon as inbound opens and Try is green...
Attachment #725400 - Flags: review?(roc)
Attached patch mochitest — — Splinter Review 

    
  
Flags: in-testsuite+

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/6ce79dc8036b
https://hg.mozilla.org/mozilla-central/rev/501804a40144
Status: ASSIGNED → RESOLVED
Closed: 11 years ago

          status-firefox22:
          affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla22

    
  
No longer blocks: 857336
Depends on: 857336

    
  
No longer depends on: 857336

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: