Make retrieval of an already loaded target interruptible. There is no reason now to forbid it, since Skyframe lookups are interruptible.

--
MOS_MIGRATED_REVID=130429286
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java b/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java
index 8b5eb24..fed47ee 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java
@@ -56,7 +56,6 @@
 import com.google.devtools.build.lib.packages.Rule;
 import com.google.devtools.build.lib.packages.Target;
 import com.google.devtools.build.lib.packages.TargetUtils;
-import com.google.devtools.build.lib.pkgcache.LoadedPackageProvider;
 import com.google.devtools.build.lib.pkgcache.LoadingResult;
 import com.google.devtools.build.lib.rules.test.CoverageReportActionFactory;
 import com.google.devtools.build.lib.rules.test.CoverageReportActionFactory.CoverageReportActionsWrapper;
@@ -908,10 +907,10 @@
       }
 
       @Override
-      protected Target getTarget(Target from, Label label, NestedSetBuilder<Label> rootCauses) {
+      protected Target getTarget(Target from, Label label, NestedSetBuilder<Label> rootCauses)
+          throws InterruptedException {
         try {
-          return LoadedPackageProvider.getLoadedTarget(
-              skyframeExecutor.getPackageManager(), eventHandler, label);
+          return skyframeExecutor.getPackageManager().getTarget(eventHandler, label);
         } catch (NoSuchThingException e) {
           throw new IllegalStateException(e);
         }