tree 6bc644cec18a5b3ba9dc4c04fa73e7652db37a3c
parent d9b4bc87aaf3e4a39e99ce9eaab702b53136d980
author Emilio Cobos Álvarez <emilio@crisal.io> 1619575113 +0000
committer moz-wptsync-bot <wptsync@mozilla.com> 1619690539 +0100

Make sure to consume the flusher when we hit the cascade data cache.

So that sheets are properly marked as committed. The issue is that we
have this "committed" optimization, to avoid doing work if somebody adds
and removes an stylesheet without us flushing in the meantime:

  https://searchfox.org/mozilla-central/rev/f018480dfed4fc583703a5770a6db9ab9dc0fb99/servo/components/style/stylesheet_set.rs#308-319

The "committed" bit is set when we consume the flusher (in each(..)).

However when we hit the cache, before this patch, we wouldn't consume
it, which means that we may fail to do some full rebuilds even though we
need them.

Fix it by making sure we call each() in that case.

We add a test (the test would show a red square before this patch, and a
lime square with the fix).

Differential Revision: https://phabricator.services.mozilla.com/D113206

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1707116
gecko-commit: 9f94f9b8e3cc41a80e60da89c7f83f40aeee9c58
gecko-reviewers: boris