Don’t warn about missing MIME for 204 responses.

It’s expected for 204 No Content responses to lack MIME type, since it’s not applicable.
micmro · Feb 11, 2017 · 0a5eb99 · 0a5eb99
1 parent 606eb0e
commit 0a5eb99
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/ts/waterfall/row/svg-indicators.ts b/src/ts/waterfall/row/svg-indicators.ts
@@ -66,7 +66,9 @@ export function getMimeTypeIcon(entry: WaterfallEntry): Icon {
     output.push(makeIcon("noGzip", "no gzip"));
   }
 
-  if (!harEntry.response.content.mimeType && heuristics.isInStatusCodeRange(harEntry, 200, 299)) {
+  if (!harEntry.response.content.mimeType &&
+    heuristics.isInStatusCodeRange(harEntry, 200, 299) &&
+    harEntry.response.status !== 204) {
     output.push(makeIcon("warning", "No MIME Type defined"));
   }