Bug 1822308 - Re-tighten GPU sandbox's filesystem access

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

security/sandbox/win/src/sandboxbroker/sandboxBroker.cpp

@@ -1211,14 +1211,14 @@ void SandboxBroker::SetSecurityLevelForGPUProcess(int32_t aSandboxLevel) {
       sandbox::SBOX_ALL_OK == result,
       "With these static arguments AddRule should never fail, what happened?");
 
-  // TEMPORARY WORKAROUND - Blocking access to the filesystem breaks Raptor
+  // The GPU process needs to write to a shader cache for performance reasons
-  // tests and any other piece of software that uses screen recording, which
+  if (sProfileDir) {
-  // may hinder developers that are developing/testing on Firefox
+    AddCachedDirRule(mPolicy, sandbox::TargetPolicy::FILES_ALLOW_DIR_ANY,
-  result = mPolicy->AddRule(sandbox::TargetPolicy::SUBSYS_FILES,
+                     sProfileDir, u"\\shader-cache"_ns);
-                            sandbox::TargetPolicy::FILES_ALLOW_ANY, L"*");
+
-  MOZ_RELEASE_ASSERT(
+    AddCachedDirRule(mPolicy, sandbox::TargetPolicy::FILES_ALLOW_ANY,
-      sandbox::SBOX_ALL_OK == result,
+                     sProfileDir, u"\\shader-cache\\*"_ns);
-      "With these static arguments AddRule should never fail, what happened?");
+  }
 
   // The process needs to be able to duplicate shared memory handles,
   // which are Section handles, to the broker process and other child processes.