DevTools: Ignore mimetype mismatch for Image, Script, StyleSheet

Source/devtools/front_end/sdk/NetworkManager.js

 /*
  * 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 44 matching lines @@
     RequestUpdateDropped: "RequestUpdateDropped"
 }
 
 WebInspector.NetworkManager._MIMETypes = {
     "text/html":                   {"document": true},
     "text/xml":                    {"document": true},
     "text/plain":                  {"document": true},
     "application/xhtml+xml":       {"document": true},
     "text/css":                    {"stylesheet": true},
     "text/xsl":                    {"stylesheet": true},
     "image/jpg":                   {"image": true},

[pfeldman, 2015/02/24 20:07:18]

While you are here, you might want to delete these then.

[paulirish, 2015/02/24 20:11:33]

Done.

     "image/jpeg":                  {"image": true},
     "image/pjpeg":                 {"image": true},
     "image/png":                   {"image": true},
     "image/gif":                   {"image": true},
     "image/bmp":                   {"image": true},
     "image/svg+xml":               {"image": true, "font": true, "document": true},
     "image/vnd.microsoft.icon":    {"image": true},
     "image/webp":                  {"image": true},
     "image/x-icon":                {"image": true},
     "image/x-xbitmap":             {"image": true},
@@ skipping 182 matching lines @@
         // If status is an error, content is likely to be of an inconsistent type,
         // as it's going to be an error message. We do not want to emit a warning
         // for this, though, as this will already be reported as resource loading failure.
         // Also, if a URL like http://localhost/wiki/load.php?debug=true&lang=en produces text/css and gets reloaded,
         // it is 304 Not Modified and its guessed mime-type is text/php, which is wrong.
         // Don't check for mime-types in 304-resources.
         if (networkRequest.hasErrorStatusCode() || networkRequest.statusCode === 304 || networkRequest.statusCode === 204)
             return true;
 
         var resourceType = networkRequest.resourceType();
-        if (resourceType === undefined
-            || resourceType === WebInspector.resourceTypes.Other
-            || resourceType === WebInspector.resourceTypes.Media
-            || resourceType === WebInspector.resourceTypes.XHR
-            || resourceType === WebInspector.resourceTypes.WebSocket)
+
+        if (resourceType !== WebInspector.resourceTypes.Stylesheet
+            || resourceType !== WebInspector.resourceTypes.Document

[pfeldman, 2015/02/24 20:02:31]

I wonder if you want to replace || with &&.

[paulirish, 2015/02/24 20:03:59]

Not a bad idea. :)  Thx.

[paulirish, 2015/02/24 20:11:33]

Done.

+            || resourceType !== WebInspector.resourceTypes.TextTrack)
             return true;
 
         if (!networkRequest.mimeType)
             return true; // Might be not known for cached resources with null responses.
 
         if (networkRequest.mimeType in WebInspector.NetworkManager._MIMETypes)
             return resourceType.name() in WebInspector.NetworkManager._MIMETypes[networkRequest.mimeType];
 
         return false;
     },
@@ skipping 417 matching lines @@
             target.networkAgent().emulateNetworkConditions(this._networkConditions.offline, this._networkConditions.latency,
                 this._networkConditions.throughput, this._networkConditions.throughput);
         }
     }
 }
 
 /**
  * @type {!WebInspector.MultitargetNetworkManager}
  */
 WebInspector.multitargetNetworkManager;