Offline page's FeaturePolicy [1] has a few fields that might have been useful in the past but are not anymore:
* is_supported_by_recent_tabs: not used in production code
* is_removed_on_cache_reset: finally reflects the exact same behavior as the namespace lifetime policy being TEMPORARY and should be consolidated with it.
These fields should be renamed (and potentially have comments reworded) to make their usage clearer:
* disabled_when_prefetch_disabled: this is easily confused with the Offline Prefetch feature when it fact it relates to the user setting for allowing web content to be prefetched for faster page loading.
* should_allow_download: this wording makes it sound like namespaces with this set to false couldn't be downloaded but in fact refers to allowing file downloads to proceed in case these pages are saved in the background.
[1] https://cs.chromium.org/chromium/src/components/offline_pages/core/offline_page_client_policy.h?q=%22struct+FeaturePolicy%22
Comment 1 by dim...@chromium.org
, Sep 24