[Webkit-unassigned] [Bug 18610] text-indent: negative not supported

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=18610


Vineet Chaudhary (vineetc) <rgf748 at motorola.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ap at webkit.org,
                   |                            |rgf748 at motorola.com




--- Comment #4 from Vineet Chaudhary (vineetc) <rgf748 at motorola.com>  2011-10-21 06:33:57 PST ---
(In reply to comment #2)

> "If a negative length value cannot be supported, it should be converted to the nearest value that can be supported." 
> 
> So big negative numbers should not be ignored, it should be "decreased" to  supported value. Am I right, Ian?

After debugging this issue below are my observations.
1) Whenever it sets text-indent value it get its length calling primitiveValue->computeLength<Lenght>(....);

2) As Length.h no longer expects 28 bit integers so max text-indent value in "em" could be +/-"8388607em" and in "px" +/-"134217727px", for greater values it resets to "0"
ref. http://trac.webkit.org/browser/trunk/Source/WebCore/css/CSSPrimitiveValue.cpp?rev=91969#L280

I tested this test on latest firefox(7.0.1), opera and IE which passes this test.

Is this is expected behavior as per our Length.h implementation? Or we should change this?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.