text-indent: negative not supported

[jasneet]

I Steps:
Go to 
http://www.hixie.ch/tests/adhoc/css/text/indent/001.html

II Issue:
word Fail repetitiously seen under : You should not see any text below.

III Conclusion:
issue with text-indent: negative

IV Other browsers:
IE7: ok
FF3: not ok
Opera9.24: ok

V Nightly tested: 32005

[Anatoli Papirovski]

This is failing because of the astronamically big number to which the text-indent is set, it also fails in Firefox 3.

Per CSS spec:
"The value of 'text-indent' may be negative, but there may be implementation-specific limits."

Personally, I don't think this would be worth fixing... but then again, I can't say I have any authority on this matter. Heh.

[Robert Blaut]

(In reply to comment #1)
> This is failing because of the astronamically big number to which the
> text-indent is set, it also fails in Firefox 3.
> 
> Per CSS spec:
> "The value of 'text-indent' may be negative, but there may be
> implementation-specific limits."
> 
> Personally, I don't think this would be worth fixing... but then again, I can't
> say I have any authority on this matter. Heh.
> 

Anatoli, notice other quote from CSS 2.1 specification:

"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?

[Ian 'Hixie' Hickson]

If it's what the spec says...

[Vineet Chaudhary (vineetc)]

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

[Brent Fulgham]

This test case appears to work properly.

[Radar WebKit Bug Importer]

<rdar://problem/96847948>