Close all SkyFunction states if the evaluation is interrupted.

Not doing this caused test_interrupted_children_waited to fail with --experimental_worker_for_repo_fetching=platform because SkyFunction states were not closed when the SkyFunction was interrupted in flight, which caused processes spawned by repository fetching not to be interrupted in turn.

Work towards #10515 .

RELNOTES: None.
PiperOrigin-RevId: 554477869
Change-Id: Ie6d2289b5bfdc7f4a73fc9dd29a40a5fa79b6269

src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java

@@ -3643,9 +3643,7 @@ public void onEvaluationStarted(UnnecessaryTemporaryStateDropper dropper) {
 
         @Override
         public void onEvaluationFinished() {
-          // TODO(wyv): Consider calling dropper.drop() here. That would effectively cause us to
-          //   clear the SkyKeyComputeState cache at the end of an evaluation, potentially releasing
-          //   any hogged memory (for example, in case of an interrupted evaluation).
+          SkyframeExecutor.this.dropper.drop();
           SkyframeExecutor.this.dropper = NULL_UNNECESSARY_TEMPORARY_STATE_DROPPER;
         }
       };