Remove Content-Type workaround for cached resources

webkit/glue/devtools/js/tests.js

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 
 /**
  * @fileoverview This file contains small testing framework along with the
  * test suite for the frontend. These tests are a part of the continues build
  * and are executed by the devtools_sanity_unittest.cc as a part of the
  * Interactive UI Test suite.
@@ skipping 345 matching lines @@
           test.releaseControl();
         }
       }, true);
 
   WebInspector.panels.resources._enableResourceTracking();
   this.takeControl();
 };
 
 
 /**
+ * Tests the mime type of a cached (HTTP 304) resource.
+ */
+TestSuite.prototype.testCachedResourceMimeType = function() {
+  this.showPanel('resources');
+
+  var test = this;
+  var hasReloaded = false;
+
+  this.addSniffer(WebInspector, 'updateResource',

[yurys, 2009/11/26 12:56:22]

are you sure WebInspector.updateResponse cannot have been called before this
line?

[apavlov, 2009/11/26 13:01:45]

The test logic is a replica of the TestSuite.prototype.testResourceHeaders which
seems to work fine. I believe enabling the resource tracking at the end of the
method induces resource reloading.

+      function(identifier, payload) {
+        var resource = this.resources[identifier];
+        if (resource.mainResource) {
+          // We are only interested in secondary resources in this test.
+          return;
+        }
+
+        if (payload.didResponseChange) {
+          // Test server uses a default mime type for JavaScript files.
+          test.assertEquals('text/html', payload.mimeType);

[yurys, 2009/11/26 12:56:22]

is it possible to fix mime type for .js files? if not please add comment why
mime type is text/html

[apavlov, 2009/11/26 13:01:45]

I'm somewhat reluctant to fix the server in the same CL (even though it's a good
idea to add the most used mime types).
The comment is the previous line.

+          if (!hasReloaded) {
+            hasReloaded = true;
+            // Reload inspected page to update all resources.
+            test.evaluateInConsole_(
+                'window.location.reload(true);',
+                 function() {});
+          } else {
+            test.releaseControl();
+          }
+        }
+
+      }, true);
+
+  WebInspector.panels.resources._enableResourceTracking();
+  this.takeControl();
+};
+
+
+/**
  * Tests that profiler works.
  */
 TestSuite.prototype.testProfilerTab = function() {
   this.showPanel('profiles');
 
   var test = this;
   this.addSniffer(WebInspector.panels.profiles, 'addProfileHeader',
       function(typeOrProfile, profile) {
         if (!profile) {
           profile = typeOrProfile;
@@ skipping 1492 matching lines @@
 /**
  * Run specified test on a fresh instance of the test suite.
  * @param {string} name Name of a test method from TestSuite class.
  */
 uiTests.runTest = function(name) {
   new TestSuite().runTest(name);
 };
 
 
 }