1606277 - Feed from cache load optimization

Status: REOPENED

(Firefox :: New Tab Page, enhancement, P2)

Description

[Scott [:thecount] Downe]

Small cleanup that can happen, might also reduce load time for cache loads.

Right now if we load feed data from fresh or cache, we then filter blocked stories from those results.

If we instead, updated feed cache as stories got blocked, we wouldn't need to do it again from cached loads as it would already be up to date. We would only need to filter blocked for feeds if it was fresh.

This doesn't apply to spocs, that has a slightly more complex handling because it's got placements, frequency capping, and flight id migration, so the code paths are separate and different currently. See bug 1606276

Comment 1

[Scott [:thecount] Downe]

This is getting fixed in bug 1932196 as a side effect of a new storage mechanism.

Comment 2

[Scott [:thecount] Downe]

Reopening this.

I'm adding back filterRecommendations for cache that folks might have while they update.

Once ESR catches up to bug 1932196 we can remove filterRecommendations and close this.

Comment 3

[Scott [:thecount] Downe]

We can target this for 140 at the earliest, as that should give enough time to get bug 1932196 in ESR.