Bug 256040

I tried to find for cases in 'CSSPropertyCaptionSide' but couldn't find except this:

https://searchfox.org/wubkat/source/Source/WebCore/css/parser/CSSPropertyParser.cpp#1139

and then this:

https://searchfox.org/wubkat/source/Source/WebCore/css/CSSProperties.json#3071

______

Let me try to remove 'left' and 'right' from .json file from above and see if it works.

Although, if we need to remove elsewhere as well, appreciate if someone else can guide. Thanks!

(In reply to Ahmad Saleem from comment #1)
> I tried to find for cases in 'CSSPropertyCaptionSide' but couldn't find
> except this:
> 
> https://searchfox.org/wubkat/source/Source/WebCore/css/parser/
> CSSPropertyParser.cpp#1139
> 
> and then this:
> 
> https://searchfox.org/wubkat/source/Source/WebCore/css/CSSProperties.
> json#3071
> 
> ______
> 
> Let me try to remove 'left' and 'right' from .json file from above and see
> if it works.
> 
> Although, if we need to remove elsewhere as well, appreciate if someone else
> can guide. Thanks!

Removing just from CSSProperties.json makes us pass this bit.

I see mentions of left/right associated to captionside in 

https://searchfox.org/wubkat/rev/36b39fa88451d43c2afb6996534a13381260008e/Source/WebCore/rendering/style/RenderStyleConstants.h#751-756

https://searchfox.org/wubkat/rev/36b39fa88451d43c2afb6996534a13381260008e/Source/WebCore/css/CSSPrimitiveValueMappings.h#561-565

https://searchfox.org/wubkat/rev/36b39fa88451d43c2afb6996534a13381260008e/Source/WebCore/rendering/style/RenderStyleConstants.cpp#222-231


but captionside was added by Sam Weinig very early in 2008.

That said, I wonder about left and right because of the spec:

> These two values are added only for implementations that support left and right values for caption-side. The left and right values themselves are defined to be line-relative.
— https://w3c.github.io/csswg-drafts/css-logical/#caption-side


The two values being `inline-start` and `inline-end`.


> line-relative
> Interpreted relative to the orientation of the line box. The line-relative directions are line-left, line-right, line-over, and line-under.
— https://w3c.github.io/csswg-drafts/css-writing-modes-4/#line-relative

<rdar://problem/108892083>

(In reply to Ahmad Saleem from comment #2)
> (In reply to Ahmad Saleem from comment #1)
> > I tried to find for cases in 'CSSPropertyCaptionSide' but couldn't find
> > except this:
> > 
> > https://searchfox.org/wubkat/source/Source/WebCore/css/parser/
> > CSSPropertyParser.cpp#1139
> > 
> > and then this:
> > 
> > https://searchfox.org/wubkat/source/Source/WebCore/css/CSSProperties.
> > json#3071
> > 
> > ______
> > 
> > Let me try to remove 'left' and 'right' from .json file from above and see
> > if it works.
> > 
> > Although, if we need to remove elsewhere as well, appreciate if someone else
> > can guide. Thanks!
> 
> Removing just from CSSProperties.json makes us pass this bit.

Plus I run following:

Tools/Scripts/run-webkit-tests fast/table imported/w3c/web-platform-tests/css/ imported/w3c/web-platform-tests/html/semantics


to see, if we fail any 'caption' tests except just two progressions. I think we don't have any regressions or failure but also to highlight, we haven't imported 'css-table' from the WPT yet (which is bug 252516). So we don't know full breakage as well but just wanted to give what I have tried so far on this bug.

Let's hope that it will be useful to solve this issue. :) 
https://github.com/WebKit/WebKit/pull/13920

Ahmad, 
/css/css-tables are imported.
Bug bug 252516 is closed.

(In reply to Karl Dubost from comment #8)
> Ahmad, 
> /css/css-tables are imported.
> Bug bug 252516 is closed.

Thanks @Karl. Appreciate it.

PR - https://github.com/WebKit/WebKit/pull/14069

Committed 264262@main (402ed699af6c): <https://commits.webkit.org/264262@main>

Reviewed commits have been landed. Closing PR #14069 and removing active labels.