Remove Content-Type workaround for cached resources

Remove Content-Type workaround for cached resources

BUG=27364
TEST=DevToolsSanityTest.TestCachedResourceMimeType

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=33206

[apavlov, 2009-11-26 12:41:51]

Please have a look.

[yurys, 2009-11-26 12:56:21]

LGTM

http://codereview.chromium.org/437089/diff/3/6
File webkit/glue/devtools/js/tests.js (right):

http://codereview.chromium.org/437089/diff/3/6#newcode374
webkit/glue/devtools/js/tests.js:374: this.addSniffer(WebInspector,
'updateResource',
are you sure WebInspector.updateResponse cannot have been called before this
line?

http://codereview.chromium.org/437089/diff/3/6#newcode384
webkit/glue/devtools/js/tests.js:384: test.assertEquals('text/html',
payload.mimeType);
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]

http://codereview.chromium.org/437089/diff/3/6
File webkit/glue/devtools/js/tests.js (right):

http://codereview.chromium.org/437089/diff/3/6#newcode374
webkit/glue/devtools/js/tests.js:374: this.addSniffer(WebInspector,
'updateResource',
On 2009/11/26 12:56:22, Yury Semikhatsky wrote:
> are you sure WebInspector.updateResponse cannot have been called before this
> line?

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.

http://codereview.chromium.org/437089/diff/3/6#newcode384
webkit/glue/devtools/js/tests.js:384: test.assertEquals('text/html',
payload.mimeType);
On 2009/11/26 12:56:22, Yury Semikhatsky wrote:
> is it possible to fix mime type for .js files? if not please add comment why
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).

> mime type is text/html
The comment is the previous line.