Do not revalidate when ResourceLoaderOptions.synchronousPolicy is different

third_party/WebKit/Source/core/fetch/ResourceLoaderOptions.h

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 86 matching lines @@
   // Answers the question "can a separate request with these different options
   // be re-used" (e.g. preload request) The safe (but possibly slow) answer is
   // always false.
   bool canReuseRequest(const ResourceLoaderOptions& other) const {
     // dataBufferingPolicy differences are believed to be safe for re-use.
     // FIXME: check allowCredentials.
     // FIXME: check credentialsRequested.
     // FIXME: check contentSecurityPolicyOption.
     // initiatorInfo is purely informational and should be benign for re-use.
     // requestInitiatorContext is benign (indicates document vs. worker)
-    // synchronousPolicy (safe to re-use an async XHR response for sync, etc.)
+    if (synchronousPolicy != other.synchronousPolicy)
+      return false;
     return corsEnabled == other.corsEnabled;
     // securityOrigin has more complicated checks which callers are responsible
     // for.
   }
 
   // When adding members, CrossThreadResourceLoaderOptionsData should be
   // updated.
   DataBufferingPolicy dataBufferingPolicy;
 
   // Whether HTTP credentials and cookies are sent with the request.
@@ skipping 67 matching lines @@
 struct CrossThreadCopier<ResourceLoaderOptions> {
   using Type = CrossThreadResourceLoaderOptionsData;
   static Type copy(const ResourceLoaderOptions& options) {
     return CrossThreadResourceLoaderOptionsData(options);
   }
 };
 
 }  // namespace blink
 
 #endif  // ResourceLoaderOptions_h