1328584 - History Item navigates to incorrect URL

Status: RESOLVED FIXED

(Firefox for iOS :: General, defect, P1)

Description

[Maurya Talisetti]

I visited http://m.huffpost.com/US sometime last month. And so, it is correctly listed in my History Panel (under "Last month"). Now, when I tap it, it takes me to, what looks like, the URL referenced by another history item.

In the screenshot I've attached of last month's history panel, you'll see that it lists "huffpost.com/US, huffpost.in, google, tim cook (google result)". When I tapped "huffpost.com/US", it took me to URLs referenced by these other history items. See the history panel screenshot of "today". It's not just google search URLs that it takes me to. Above was just an example. It took me to other sites from my history like msn.com, goodreads.com, etc.

Comment 1

[Maurya Talisetti]

Attachment: Last Month Section.PNG

Comment 2

[Maurya Talisetti]

Attachment: Today Section.PNG

Comment 3

[Maurya Talisetti]

This actually happens on "Firefox Beta 6 - Build 8". I don't seem to have the privileges to edit the "version" section.

Comment 4

[Farhan Patel [:farhan]]

This is pretty major. Here is how you can replicate it. 

History is split into 4 sections Today/Yesterday/Last Week/ Last month. 

1) Have a history in all 4 sections. 
2) Delete history in the Today section.
3) Click on an item in the last section. 

It should go to the correct Item. Instead it chooses the wrong item. 

The reason why this doesnt happen that often is once you click on an item in a single section you'll get new content in the Today section. And the issue will stop happening.

Comment 5

[Farhan Patel [:farhan]]

Attachment: Pull Request

Comment 6

[Farhan Patel [:farhan]]

master https://github.com/mozilla-mobile/firefox-ios/commit/205a3264794045c3ea7eea9f55f819a3d60bfcd7

Comment 7

[Farhan Patel [:farhan]]

v6.x d9020a33d4b43e9f209b256b2e3bc2a37096837a

Comment 8

[Farhan Patel [:farhan]]

whoops. Too many tabs. Wrong commit. 

v6.x 87ad08bf44e0f68d39e49ecdb5d3d82100386362