dontworryspain / ie7-js

Automatically exported from code.google.com/p/ie7-js
0 stars 0 forks source link

CONTENT regex is case insensitive #370

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. It is hard to reproduce since it is server specific (it occurred on 
cloudFlare servers)

What is the expected output? What do you see instead?
"content" words that the regular expression in IE9.js was searching for were in 
uppercase so the regular expression did not find anything - regex was expecting 
lowercase "content" (line 2140 in non minified IE9.js).

What version of the product are you using? On what operating system?
Was using IE9.js scripts from ie7-2.1(beta4).zip

Please provide any additional information below.
I have fixed this so that I converted regular expression to case insensitive on 
line 2221 in non minified IE9.js.
before: CONTENT: /content\s*:\s*([^;]*)(;|$)/,
patched: CONTENT: /content\s*:\s*([^;]*)(;|$)/i,

I'm not certain if this is the correct solution for the problem but it fixed 
the issue for me.

Original issue reported on code.google.com by Domen.Vr...@gmail.com on 14 Jun 2012 at 1:08

Attachments:

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
I preformed the tests with Internet explorer 9 in IE 7 compatibility mode

Original comment by Domen.Vr...@gmail.com on 14 Jun 2012 at 1:13