Bug 52059 – setContentEditable should be case insensitive

• Component: HTML Editing
• Status: RESOLVED DUPLICATE

[Chang Shu]

The current implementation of setContentEditable does not take case insensitive into consideration.
Layout test:
set-value-caseinsensitive.html

[Suresh Voruganti]

Fix required for Qtwebkit 2.2

[Alexey Proskuryakov]

> set-value-caseinsensitive.html

Where can one find this layout test?

[Chang Shu]

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!

[Darin Adler]

The HTML5 specification says convert to lowercase. That’s what we should implement.

[Chang Shu]

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 ***

[Chang Shu]

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.

[Chang Shu]

Ademar, can you cherry-pick this patch to QtWebKit2.1.x? thanks!

[Ademar Reis]

(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.

[Chang Shu]

(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.

[Ademar Reis]

(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.