gpu shader cache should not depend on //net |
|||
Issue descriptionThe gpu shader cache code [1] currently has a dependency on //net. This is because the shader cache uses disk_cache, which lives in net. It'd be nice to not have this dependency. For example, the shader cache could use some interface for storing/retrieving the cache. The implementation that sits behind the interface could directly access the filesystem, or store the cache in an indexeddb, or talk to a filesystem service etc. [1] https://cs.chromium.org/chromium/src/gpu/ipc/host/shader_disk_cache.h?l=18
,
Dec 19 2016
That is a correct explanation of what I tried to describe, yes. One of the motivations for avoiding the direct dependence/use of disk_cache is that using a mojom service like indexeddb/filesystem would possibly allow the gpu-host to be sandboxed more than it currently is. The //net dependency currently is in //gpu/ipc/host, which is a standalone target (in the sense that no other code in gpu depends on this code). So it could have lived elsewhere too (e.g. in //components/gpu_host). But I couldn't come up with a better location.
,
Feb 16 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Feb 16 2018
|
|||
►
Sign in to add a comment |
|||
Comment 1 by rsleevi@chromium.org
, Dec 19 2016