Gzip test stalls in a number of browsers

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Run all Network tests or solely the Gzip test with one of the browsers
listed further down.
2. Notice how the test never outputs the result.

What is the expected output? What do you see instead?
The expected output is either "Your browser does support compression." or
"Your browser doesn't support compression.".

Some browsers fail to output a result when the response of the XHR request
for "/static/base.css" is not gzipped and the response headers contain no
"Content-Encoding" header. The javascript variable contentEncoding becomes
null as a result of the call to getResponseHeader('Content-Encoding') and
thus the call to contentEncoding.toLowerCase() results in a TypeError
("Cannot convert undefined or null to Object"). A trivial patch to fix this
is attached.

In the versions of Safari tested the response is gzipped and the response
header "Content-Encoding: gzip" is present (checked with Wireshark) but the
getResponseHeader('Content-Encoding') call still returns null and thus the
TypeError soon follows.

As a side note: App Engine doesn't seem to output gzipped responses to
Opera, versions 8.53, 9.20, 10.01 and 10.50, even though they send an
"Accept-Encoding" header containing "gzip". Thus, the result of the gzip
test for these versions of Opera is wrong (at least since the gzip test
change in r561:
http://code.google.com/p/browserscope/source/diff?path=/trunk/categories/network
/templates/tests/gzip.html&format=side&r=561).

What version of the product are you using? On what operating system?
The following browsers are the ones I've found to have this issue:
* Opera 10.50 on Windows XP
* Safari 3.1 on Windows XP
* Safari 4.0.5 on Windows XP
* Epiphany 2.28.0 on Ubuntu Linux 9.10
* Konqueror 4.3.2 on Ubuntu Linux 9.10
* Opera 10.01 on Ubuntu Linux 9.10

Original issue reported on code.google.com by robin.e....@gmail.com on 23 Apr 2010 at 1:49

Attachments:

• gzip-test-typeerror-fix.patch

[GoogleCodeExporter]

Thanks for the patch. I will take care of this shortly.

Original comment by steve.lamm on 23 Apr 2010 at 4:05

[GoogleCodeExporter]

Committed and pushed.

Original comment by steve.lamm on 23 Apr 2010 at 6:00

• Changed state: Fixed