Closed
Bug 1403414
Opened 7 years ago
Closed 7 years ago
History entry is named "New Tab" when navigating directly to a video from about:home and New Tab
Categories
(Firefox :: New Tab Page, defect)
Tracking
()
VERIFIED
FIXED
Firefox 58
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox55 | --- | unaffected |
firefox56 | --- | unaffected |
firefox57 | - | wontfix |
firefox58 | - | verified |
firefox59 | --- | verified |
People
(Reporter: alice0775, Assigned: Mardak)
References
Details
(Keywords: regression)
[Tracking Requested - why for this release]: Regression history system
Reproducible: always
Steps to Reproduce:
1. Open https://goo.gl/K9rrxQ
Actual Results:
History entry is named "New Tab" instead of tab title "toystory.mp4"
Expected Results:
History entry should be named "toystory.mp4"
Comment hidden (obsolete) |
Reporter | ||
Comment 2•7 years ago
|
||
Steps to Reproduce:
1. Create New profile and start Nightly with the profile and exit browser and restart
2. Open https://goo.gl/K9rrxQ
Reporter | ||
Comment 3•7 years ago
|
||
[Tracking Requested - why for this release]:Regression history system
tracking-firefox57:
--- → ?
Reporter | ||
Updated•7 years ago
|
Version: 58 Branch → 57 Branch
Reporter | ||
Comment 4•7 years ago
|
||
Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=01b9bea7d2ba241fba343aedf6decdcf2fac851f&tochange=3ce5a9419d5b7a976d442901c04995c22e014bf1
Regressed by: 3ce5a9419d5b Ursula Sarracini — Bug 1396274 - Disable Activity Stream about:home in browser_aboutHome.js r=Mardak
@usarracini
Your patch seems to cause the problem, Can you look at this?
Blocks: 1396274
Flags: needinfo?(usarracini)
Assignee | ||
Comment 5•7 years ago
|
||
What history entry are you looking at? I visited the provided url and long clicking the back/forward button from that page or from this bugzilla page shows "toystory.mp4" ?
Reporter | ||
Comment 6•7 years ago
|
||
(In reply to Ed Lee :Mardak from comment #5)
"History" in Menubar, History sidebar, History of Library Window and History widget as well as session history of back/forward button.
Reporter | ||
Comment 7•7 years ago
|
||
You should follow the str.
Probably, Open from the URL from New Tab page is broken.
Component: Places → Activity Streams: Newtab
Product: Toolkit → Firefox
Assignee | ||
Comment 8•7 years ago
|
||
I did follow the STR. It had one step. I opened the link.
Reporter | ||
Comment 9•7 years ago
|
||
(In reply to Ed Lee :Mardak from comment #8)
> I did follow the STR. It had one step. I opened the link.
???
STR:
1. Start Nightly with new profile and exit browser and then restart browser again with the profile
2. Close Default browser dialog if any
3. Open https://goo.gl/K9rrxQ, you should type the url in address bar and hit [Enter]
4. Observe "History" in Menubar, History sidebar, History of Library Window and History widget as well as session history of back/forward button.
Reporter | ||
Comment 10•7 years ago
|
||
Alternate str:
1. Clear Recent History(Ctrl+Shift+Del) every thing
2. Open New Tab (Ctrl+T)
3. Open https://goo.gl/K9rrxQ, you should type the url in address bar and hit [Enter]
4. Observe "History" in Menubar, History sidebar, History of Library Window and History widget as well as session history of back/forward button.
Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=0be298129be1221d42b269a61ab3911349738e70&tochange=07a7f3038e14c1594c80431318d3cdd6c8d928a6
Regressed by: 07a7f3038e14 Ed Lee — Bug 1381569 - Turn on Activity Stream by default for Nightly. r=mconley
Setting browser.newtabpage.activity-stream.enabled = false fixes the issue in both case(Comment 9 and Comment 10)
Blocks: 1381569
Comment 11•7 years ago
|
||
Bizarre...
Keeping browser.newtabpage.activity-stream.enabled = true but turning browser.newtabpage.activity-stream.aboutHome.enabled = false fixes the issue, which means it's not activity stream on newtab which is causing the problem, but loading it as about:home that's causing the issue.
When opening the link in a about:newtab, I can't reproduce this issue (even with activity stream enabled), it's only reproducible if we open the link in a new window (which would be about:home). Possibly something going on with the way we switch content processes for sites that are "files" rather than websites.
I can look into what's going on here.
Flags: needinfo?(usarracini)
This seems like a bad regression, tracked for 57.
Assignee | ||
Comment 13•7 years ago
|
||
[Tracking Requested - why for this release]: Not sure if tracking-firefox57 was set in comment 12 assuming that all history pages only showed "New Tab" as the title.
I can't reproduce this issue for any normal html web pages including pages that have no title. I can't reproduce this with images or pdfs.
Activity Stream does do a `document.title = "New Tab"` and removing that does make this bug go away but then the history entry shows up with the wrong title for about:home.
This might just get fixed anyway when Activity Stream prerenders with strings, so it can just do `<title>New Tab</title>`, but until then, the title is only set after it gets its strings.
Component: Activity Streams: Newtab → Bookmarks & History
Summary: History entry is always named "New Tab" instead of tab title → History entry is named "New Tab" when navigating directly to a video from about:home
Reporter | ||
Comment 14•7 years ago
|
||
I can reproduce the problem on 58.0a1 and 57.0b3 when open the URL from New Tab as well as Home tab. See STR comment #10.
Summary: History entry is named "New Tab" when navigating directly to a video from about:home → History entry is named "New Tab" when navigating directly to a video from about:home and New Tab
Updated•7 years ago
|
Updated•7 years ago
|
Component: Bookmarks & History → Activity Streams: Newtab
Assignee | ||
Comment 15•7 years ago
|
||
mak, we'll fix this on the activity stream side in https://github.com/mozilla/activity-stream/issues/3790 but it's not really clear how this is happening in the first place. How would setting the title dynamically on one page cause it to be set on a new page?
Assignee: nobody → edilee
Comment 16•7 years ago
|
||
I don't know, history takes the title from the docshell, see the UpdateGlobalHistoryTitle calls there. Those calls take the current uri as input, I can only imagine for some reason the title is set on the wrong docshell, or the media docshell is particular. If you can reproduce, you could try debugging from there.
Assignee | ||
Comment 17•7 years ago
|
||
Fixed by https://github.com/mozilla/activity-stream/pull/3755 to be uplifted via bug 1413550.
Blocks: 1413550
Assignee | ||
Comment 18•7 years ago
|
||
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Updated•7 years ago
|
Flags: qe-verify+
QA Contact: iulia.cristescu
Comment 19•7 years ago
|
||
I managed to reproduce the bug using an older version of Nightly (2017-09-26) on Windows 10 x64. I tested this bug using the steps from comment 2.
I retested everything using latest Nightly 59 and beta 58.0b3 on Windows 10 x64, Ubuntu 16.04 x64 and macOS 10.12, but the bug is not reproducing anymore.
Updated•5 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•