Don't set content type of not modified action result, as this confuse…

…s webkit browsers.
thatismatt · Jul 9, 2010 · 0ca1dfc · 0ca1dfc
1 parent 0979d48
commit 0ca1dfc
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/josi/actionresults.js b/lib/josi/actionresults.js
@@ -82,8 +82,9 @@ this.ContentResult = ContentResult = ActionResult.extend({
         .addListener('data', function(data){
           if (first) {
             first = false;
+            var contentType = utilities.mime.lookup(utilities.extension(filename));
             self.headers.push(['Transfer-Encoding', 'chunked']);
-            self.headers.push(['Content-Type', utilities.mime.lookup(utilities.extension(filename))]);
+            self.headers.push(['Content-Type', contentType]);
             res.writeHead(200, self.headers);
           }
           res.write(data, 'binary');
@@ -123,7 +124,7 @@ this.notFound = notFound = function(msg) {
 };
 
 this.notModified = notModified = function() {
-  return new ActionResult(null, [ ['Content-Type', 'text/html'] ], 304);
+  return new ActionResult(null, [], 304);
 };
 
 this.error = error = function(err, httpStatusCode) {