resourceloader: Don't cache CSSMin::remap() calls
authorTimo Tijhof <krinklemail@gmail.com>
Wed, 2 Mar 2016 23:55:28 +0000 (23:55 +0000)
committerTimo Tijhof <krinklemail@gmail.com>
Thu, 3 Mar 2016 00:10:14 +0000 (00:10 +0000)
commit570cff11684fdd7a6002cba653d7cfeba9db9571
treec4e5553f67a67d2920f9294af644f34f6224de94
parentc99e3d537ede1628f1bc2d833e5285a2dd083399
resourceloader: Don't cache CSSMin::remap() calls

The output of this function is not deterministic. This function
is what tranforms urls to include content hashes and also embeds
content as data URI. The calls to this function are expected to be
up to date and matching the version hash as constructed by the
current web request.

Previously, the content hash as constructed based on the collective
file hash of all references images was accurate but the generated
css associated with it could be stale.

Unless we abuse MemoizedCallable to take a separate cache buster
(with $localFileRefs?), it's best to remove this as it isn't
a deterministic function and shouldn't be memoized.

Bug: T128668
Change-Id: Icb87ddc585d7320ac48619446bb8b9cbe5f4780e
includes/resourceloader/ResourceLoaderFileModule.php