Bug 188630

Summary: [LFC][BFC] Display::Box interface should reflect that padding is optional.
Product: WebKit Reporter: zalan <zalan>
Component: Layout and RenderingAssignee: zalan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, koivisto, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch koivisto: review+

Description zalan 2018-08-15 21:11:18 PDT 
ssia
Comment 1 zalan 2018-08-15 21:38:31 PDT 
Created attachment 347241 [details]
Patch
Comment 2 zalan 2018-08-16 07:40:39 PDT 
Committed r234925: <https://trac.webkit.org/changeset/234925>
Comment 3 Radar WebKit Bug Importer 2018-08-16 07:42:38 PDT 
<rdar://problem/43380732>
Comment 4 Simon Fraser (smfr) 2018-08-16 12:40:07 PDT 
Comment on attachment 347241 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=347241&action=review

> Source/WebCore/ChangeLog:8
> +        Certain type of boxes can't have paddings (see Layout::Box::isPaddingApplicable).

s/paddings/padding/

What's the utility of knowing that there is no padding vs. padding is zero? std::optional<> bloats class layout.
Comment 5 zalan 2018-08-16 12:44:29 PDT 
(In reply to Simon Fraser (smfr) from comment #4)
> Comment on attachment 347241 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=347241&action=review
> 
> > Source/WebCore/ChangeLog:8
> > +        Certain type of boxes can't have paddings (see Layout::Box::isPaddingApplicable).
> 
> s/paddings/padding/
> 
> What's the utility of knowing that there is no padding vs. padding is zero?
> std::optional<> bloats class layout.
From geometry point of view, there's no difference. This is about correctness. If we find std::optional excessive, it's fine to make it debug only and preserve the correctness part of it.