Add more checks for subresource redirect

Adds checks for whether navigation is committed and only for http/https
navigations.

Bug: 1055573
Change-Id: Ib65801efbe1cd66362c885e5b2f0442de65aa918
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2078784Reviewed-by: Michael Crouse <mcrouse@chromium.org>
Commit-Queue: rajendrant <rajendrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#745280}

chrome/browser/subresource_redirect/subresource_redirect_observer.cc

@@ -118,7 +118,9 @@ void SubresourceRedirectObserver::DidFinishNavigation(
     content::NavigationHandle* navigation_handle) {
   DCHECK(navigation_handle);
   if (!navigation_handle->IsInMainFrame() ||
-      navigation_handle->IsSameDocument()) {
+      !navigation_handle->HasCommitted() ||
+      navigation_handle->IsSameDocument() ||
+      !navigation_handle->GetURL().SchemeIsHTTPOrHTTPS()) {
     return;
   }
   auto* optimization_guide_decider = GetOptimizationGuideDeciderFromWebContents(