Closed Bug 655273 Opened 12 years ago Closed 12 years ago

Back button right-click menu doesn't have page title for history entries created by pushState


(Firefox :: Menus, defect)

4.0 Branch
Not set





(Reporter: justin.lebar+bug, Assigned: justin.lebar+bug)



(Keywords: regression)


(2 files, 1 obsolete file)


 * Click to view first page of forward.
 * Navigate forward to the second page of the forward.
 * Right-click on back button.

Expected results:

 * Resulting menu has two entries titled "20 Things I Learned About Browsers and the Web".

Actual results:

 * Resulting menu has one entry titled "20 Things..." and another entry with a URL.

A favicon is also missing, but I've filed bug 655270 on that.
Can you please tell us which version of Firefox you are using? Also is that a regression from bug 500328?
(In reply to comment #1)
> Can you please tell us which version of Firefox you are using?

I'm on Mac 10.6 trunk.

> Also is that a regression from bug 500328?

I doubt it ever worked, but I haven't checked.
Just tested and before the fix on bug 500328 has been landed the title was visible in the back button. Means it is a regression from Firefox 3.6.
Keywords: regression
OS: Mac OS X → All
Hardware: x86 → All
Version: unspecified → 4.0 Branch
The page I linked doesn't use pushState if pushState isn't there; I think that's why you're seeing that it "works" in FF 3.6.
Attached file Testcase
This simple testcase illustrates the problem.

Notice that it wouldn't have done anything before we had pushstate, pre 4.0.
Assignee: nobody → justin.lebar+bug
If I have not misread STR in comment #0,
UA spoofing is helps.And title and favicon appears in Back/Forward popup menu.
"Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b5pre) Gecko/20100827 Minefield/4.0b5pre"

Regression window(cached m-c hourly):
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b5pre) Gecko/20100827 Minefield/4.0b5pre ID:20100827075702
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b5pre) Gecko/20100827 Firefox/4.0b5pre ID:20100827101721
Triggered by:
2914a4cdd7e2	Dan Witte — Bug 588874 - Replace Minefield with Firefox in UA string. r=khuey, sr=jst, a=beta5+
So I think bug 655273 and bug 655270 are TE.

However, Another Bug I found as follows.
Only entry page is registered in the popup, No entry is registered for each page.
Should I filed a bug?

Regression window:
Works, Entry of each page are registered in the back/Forward popup
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.2a1pre) Gecko/20110329 Firefox/4.2a1pre ID:20110330085433
Fails, Only front page is registered in the back/Forward popup
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.2a1pre) Gecko/20110331 Firefox/4.2a1pre ID:20110331030432
> So I think bug 655273 and bug 655270 are TE.

What does "TE" mean?

Changeset 2914a4cdd7e2 is, as you noticed, a UA string change.  So before that change, I bet the website is doing something different -- it's probably not using history.pushState at all.

Were you spoofing the UA when you bisected the second time?

I think it's much more sensible to bisect on the testcase I attached rather than on the website in comment 0, since that site is clearly doing complicated things.  But note that when we initially landed bug 500328, we used the title parameter in pushState (now it's completely ignored), so you'd *still* get different behavior.

I think at this point whether or not this is a regression is academic.  I think I can fix it, no problem, so I don't need any more bisections right now.

TE means Tech Evangelism.

Anyway, I filed a Bug 655328 for the another bug of comment#6.
Attached patch Patch v1 (obsolete) — Splinter Review
Attachment #531095 - Flags: review?(jonas)
Closed: 12 years ago
Resolution: --- → FIXED
And backed out, because the test added here was failing:

shistory is null at chrome://mochitests/content/chrome/docshell/test/chrome/test_bug655273.html:31

It works for me, but I only ran the test with TEST_PATH set.  Perhaps if I run the test as part of the larger suite, it'll fail.
Resolution: FIXED → ---
Attached patch Patch v2Splinter Review
Changed the test to a browser-chrome test.  Seems to work better this way...
Attachment #532623 - Flags: review?(jonas)
Attachment #531095 - Attachment is obsolete: true
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Mozilla/5.0 (X11; Linux i686; rv:7.0a1) Gecko/20110527 Firefox/7.0a1

Verified issue using the STR from Comment 0 and the testcase on: Win XP, Win 7, Mac OS X, Ubuntu.
You need to log in before you can comment on or make changes to this bug.