The current implementation of setContentEditable does not take case insensitive into consideration. Layout test: set-value-caseinsensitive.html
Fix required for Qtwebkit 2.2
> set-value-caseinsensitive.html Where can one find this layout test?
document.getElementById("div1").contentEditable = "TRue"; 1. shouldBe('document.getElementById("div1").getAttribute("contentEditable")','"TRue"'); 2. shouldBe('document.getElementById("div1").getAttribute("contentEditable")','"true"'); I am not sure in the above tests, which one should be correct. I mean, should the attribute preserve the original string or should it convert to lower-case? Darin, can you help me? thanks!
The HTML5 specification says convert to lowercase. That’s what we should implement.
The case sensitive issue has been fixed with bug 52057. The remaining issue is identical to 52058. *** This bug has been marked as a duplicate of bug 52058 ***
Created attachment 80450 [details] patch for QtWebKit2.1.x only! This is a patch for QtWebKit2.1.x cherry-picking. DO NOT land in trunk.
Ademar, can you cherry-pick this patch to QtWebKit2.1.x? thanks!
(In reply to comment #7) > Ademar, can you cherry-pick this patch to QtWebKit2.1.x? thanks! I'm confused: 1. This bug is closed as a duplicate of bug 52058; 2. Bug 52058 is open and also depends on bug 52025. The right thing to do is to handle bug 52058 instead of this one, but it should be fixed on trunk before it's cherry-picked.
(In reply to comment #8) > (In reply to comment #7) > > Ademar, can you cherry-pick this patch to QtWebKit2.1.x? thanks! > > I'm confused: > > 1. This bug is closed as a duplicate of bug 52058; > 2. Bug 52058 is open and also depends on bug 52025. > > The right thing to do is to handle bug 52058 instead of this one, but it should be fixed on trunk before it's cherry-picked. I just re-use this bug for submitting this patch. The test cases in this patch are failing on trunk but pass on 2.1.x (I think trunk has a regression, but it will be fixed by 52058). Since 2.1.x is working fine, all this patch does is rebaseline the test results.
(In reply to comment #9) > (In reply to comment #8) > > (In reply to comment #7) > > > Ademar, can you cherry-pick this patch to QtWebKit2.1.x? thanks! > > > > I'm confused: > > > > 1. This bug is closed as a duplicate of bug 52058; > > 2. Bug 52058 is open and also depends on bug 52025. > > > > The right thing to do is to handle bug 52058 instead of this one, but it should be fixed on trunk before it's cherry-picked. > > I just re-use this bug for submitting this patch. The test cases in this patch are failing on trunk but pass on 2.1.x (I think trunk has a regression, but it will be fixed by 52058). Since 2.1.x is working fine, all this patch does is rebaseline the test results. OK, got it. Patch added to 2.1.x: 71743851334e7dabe9380d9ab5a4f597144f94c0 Thanks.