Page MenuHomePhabricator

Cache shouldDisplayMobileView for duration of request
Closed, ResolvedPublic

Description

I don't think the results of shouldDisplayMobileView can change during a request. It would be helpful if it were cached/memoized so other extensions could call it without performance concerns.

This basically just means that before returning, it stores the result in the instance (which is supposed to be used as a singleton), and uses it when called (if available).


Version: unspecified
Severity: normal

Details

Reference
bz43280