Not a huge concern, because it's only an issue while manually changing your preff from an on state to another on state.
If we go from discovery stream off to discovery stream on via pref change, things work as expected. When it is enabled, we clear cache and fetch new data. This works fine because it was previously off and thus the store would by default b empty.
If we go from discovery stream on to discovery stream off, we clear cache and clear the store. Also works as expected.
If we're changing the pref from an on state to another on state (example, turn spocs on from off) We clear cache and fetch new data. The issue here is if Discovery Stream was already on, we already have something in state which also loads.
- Turn on discovery stream via setting
- Use a slow layout, dev-test-feeds probably works best.
- Open a new tab.
Expected: You should see nothing but search while it loads.
Actual: You see top sites while it loads. If the layout was a bit different, things might jump around the screen. This is because when it tries to render, it checks to see if we're able to load data, and even thought the feeds are still loading, the jsx sees old stuff in the cache thus tries to display an older layout.