Remove file error checking in downloadmanager tests that check for deletion

Just going to keep retrying until the path has been deleted. Probably
better than the test crashing after the first time?

Bug: 1156112
Change-Id: Ia5e7e97a09bcfe347b63b156dde558fd59009f68
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2629673Reviewed-by: Michael Crouse <mcrouse@chromium.org>
Commit-Queue: Sophie Chang <sophiechang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#843731}

chrome/browser/optimization_guide/prediction/prediction_model_download_manager_unittest.cc

@@ -166,15 +166,6 @@ class PredictionModelDownloadManagerTest : public testing::Test {
       if (!path_exists)
         return true;
 
-      base::File::Error file_error = base::File::GetLastFileError();
-      // In the event this does not fix the flake, log the error so we know what
-      // it is.
-      // TODO(crbug/1156112): Remove this log once the flake has been resolved.
-      DLOG(ERROR) << "Path Exists Error: " << file_error;
-
-      if (file_error != base::File::FILE_ERROR_ACCESS_DENIED)
-        return !path_exists;
-
       // Retry if the last file error is access denied since it's likely that
       // the file is in the process of being deleted.
     }