Closed Bug 1546590 Opened 5 years ago Closed 5 years ago

Hardcoded layout results in only one new tab page rendered with discovery stream content

Categories

(Firefox :: New Tab Page, defect, P1)

defect

Tracking

()

VERIFIED FIXED
Firefox 68
Iteration:
68.3 - Apr 15 - 28
Tracking Status
firefox68 --- verified

People

(Reporter: Mardak, Assigned: thecount)

References

(Regression)

Details

(Keywords: github-merged, regression)

Attachments

(3 files)

https://github.com/mozilla/activity-stream/pull/4904/files#r277979670

The expiration logic doesn't handle hardcoded layout correctly resulting in layout being refreshed and subsequently spocs endpoint which makes it look like spocs are not loaded preventing the rendering of content.

The code in bug 1543800 introduced the regression while bug 1545122 flipped the pref to turn it on.

Regressed by: 1545122
Keywords: regression

Leaving open to temporarily disable hardcoded layout

Keywords: leave-open
Pushed by elee@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7be98a7e5fc4
Disable hardcoded layout to avoid expiration issues r=andreio

Maybe I was too proactive in reverting the pref. This problem seems to only happen for new profiles where the cache has no existing layout entry. So existing people on nightly have a cached layout and will see the hardcoded layout somewhat correctly.. maybe (?)

Blocks: 1546808
Keywords: leave-open
Pushed by elee@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d4f22be5eb41
Reenable hardcoded layout with expiration fixed r=andreio
Status: NEW → RESOLVED
Closed: 5 years ago
Keywords: github-merged
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68

I have verified that this issue is no longer reproducible with the latest Firefox Nightly (68.0a1 Build ID - 20190429215338) installed, on Windows 10 x64, Arch Linux and Mac 10.13.3. Now, the articles are correctly loaded on each "New Tab" page.

Status: RESOLVED → VERIFIED
Component: Activity Streams: Newtab → New Tab Page
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: