jalmenarez / minify

Automatically exported from code.google.com/p/minify
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

CSS minify should correctly handle other protocols than http(s) #18

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Simpify a css file with for instance url (chrome://myproject/skin/image.gif)

What is the expected output? What do you see instead?
The chrome protocol (and protocols other than http(s)) are handled as
relative paths and the output is something like:
/myproject/skin/chrome://myproject/skin/image.gif
instead of unchanged (since it is an absolute path):
chrome://myproject/skin/image.gif

What version of the product are you using? On what operating system?
minify 1.0.1 on Fedora.

Please provide any additional information below.
The resolveFilePath method should not only check for http and https schemes
but should check there is a protocol scheme at the beginning: ^.*://

Original issue reported on code.google.com by jerome.c...@gmail.com on 20 Sep 2007 at 3:09

GoogleCodeExporter commented 9 years ago
The upcoming alpha release will only alter URIs that are relative 
(images/my.gif) and
protocol-relative (//images/my.gif).

Original comment by mrclay....@gmail.com on 28 Feb 2008 at 12:15

GoogleCodeExporter commented 9 years ago
SVN does not have this issue

Original comment by mrclay....@gmail.com on 14 May 2008 at 4:26