Closed Bug 244150 Opened 21 years ago Closed 19 years ago

Multiple GET requests are made when "@import" rule is empty

Categories

(SeaMonkey :: General, enhancement)

Product:
SeaMonkey
Component:
General
Platform:
x86
Linux
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED EXPIRED

People

(Reporter: harry, Unassigned)

References

Details

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20031107 Debian/1.5-3
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20031107 Debian/1.5-3

The following piece of code demonstrates the problem/feature

      1 #!/usr/bin/perl
      2 use warnings;
      3 use strict;
      4 print "Content-type: text/html\n\n";
      5 print STDERR "$$\n";
      6 print <<EOF;
      7 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"    
      8     "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
      9 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en">
     10 <head>
     11     <meta http-equiv="Content-Type" content="text/html;
charset=ISO-8859-1"/>
     12     <title>Harry Jacksons Home Page</title>
     13     <style type="text/css"> \@import "";</style>
     14     <style type="text/css"> \@import "";</style>
     15     <style type="text/css"> \@import "";</style>
     16     <style type="text/css"> \@import "";</style>
     17 </head>
     18 <body>
     19     <a href="http://www.hjackson.org">Hello Harry Jackson</a>
     20 </body>
     21 </html>
     22 EOF


The offending lines are the 

     13     <style type="text/css"> \@import "";</style>
     14     <style type="text/css"> \@import "";</style>
     15     <style type="text/css"> \@import "";</style>
     16     <style type="text/css"> \@import "";</style>

these are empty import statements and for some reason Mozilla asks for the page
that they where found in. In the case above 4 requests will be handed off to Apache.

Reproducible: Always
Steps to Reproduce:
1. Create a Perl CGI with the folloing code

#!/usr/bin/perl
use warnings;
use strict;
print "Content-type: text/html\n\n";
print STDERR "$$\n"; 
print <<EOF;
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"    
    "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en">
<head>
    <meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1"/>
    <title>Harry Jacksons Home Page</title>
    <style type="text/css"> \@import "";</style>
    <style type="text/css"> \@import "";</style>
    <style type="text/css"> \@import "";</style>
    <style type="text/css"> \@import "";</style>
</head>
<body>
    <a href="http://www.hjackson.org">Hello Harry Jackson</a>
</body>
</html>
EOF


2. Monitor your apache error log
3. Use Mozilla to request the page 

Actual Results:  
You will see 4 processes spanwed by Apache to handle 4 seperate requests.

Expected Results:  
To be honest I am not sure. It could be said that empty @import statements
exhibit undefined behaviout since I was unable to find anything defining the
behaviour at w3c.

I also checked this with IE and Lynx to see if either exhibited this behaviour
and both ignored the empty @import or at least appeared to.

This is the reason I have put it in as an enhancement because I was unable to
find anything defining empty @import rule behaviour.
The relative URI "" is resolved relative to the base URI (the page) and the
resulting URI is the URI of the page.

So you are in fact importing the page itself, and we should be loading and
parsing it as a stylesheet.

The fact that we're making 4 GET requests (instead of 1) probably has to do with
bug 84582 and the fact that the cache doesn't support reading from a cache entry
that's open for writing...
Depends on: 84582
Product: Browser → Seamonkey
This is an automated message, with ID "auto-resolve01".

This bug has had no comments for a long time. Statistically, we have found that
bug reports that have not been confirmed by a second user after three months are
highly unlikely to be the source of a fix to the code.

While your input is very important to us, our resources are limited and so we
are asking for your help in focussing our efforts. If you can still reproduce
this problem in the latest version of the product (see below for how to obtain a
copy) or, for feature requests, if it's not present in the latest version and
you still believe we should implement it, please visit the URL of this bug
(given at the top of this mail) and add a comment to that effect, giving more
reproduction information if you have it.

If it is not a problem any longer, you need take no action. If this bug is not
changed in any way in the next two weeks, it will be automatically resolved.
Thank you for your help in this matter.

The latest beta releases can be obtained from:
Firefox:     http://www.mozilla.org/projects/firefox/
Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html
Seamonkey:   http://www.mozilla.org/projects/seamonkey/

This bug has been automatically resolved after a period of inactivity (see above
comment). If anyone thinks this is incorrect, they should feel free to reopen it.
Status: UNCONFIRMED → RESOLVED
Closed: 19 years ago
Resolution: --- → EXPIRED
You need to log in before you can comment on or make changes to this bug.