[webkit-reviews] review granted: [Bug 198889] [LFC][Floats] Add bottom value to FloatingState::Constraints : [Attachment 372198] Patch

[bugzilla-daemon at webkit.org]

Antti Koivisto <koivisto at iki.fi> has granted zalan <zalan at apple.com>'s request
for review:
Bug 198889: [LFC][Floats] Add bottom value to FloatingState::Constraints
https://bugs.webkit.org/show_bug.cgi?id=198889

Attachment 372198: Patch

https://bugs.webkit.org/attachment.cgi?id=372198&action=review




--- Comment #5 from Antti Koivisto <koivisto at iki.fi> ---
Comment on attachment 372198
  --> https://bugs.webkit.org/attachment.cgi?id=372198
Patch

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

> Source/WebCore/layout/LayoutUnits.h:63
> +    void moveBy(LayoutSize);

IntPoint calls this move. Generally it would be good to match existing
interfaces unless there is a good reason not to.