Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[css-layout-api] Add LayoutEdges. #13291

Merged
merged 1 commit into from Oct 15, 2018

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Oct 1, 2018

This allows web developers to access pre-computed sizes for the border,
scrollbar, and padding "edges".

This adds two types of tests:

  1. Uses the standard "green box" approach of testing the script values
    match what is expected.
  2. Positions four children at each corner (using edges) are positioned
    the same as using an abs-pos technique.

Bug: 726125
Change-Id: I3b53be9b44989d919cb657d4eabc6d3a1a79181b
Reviewed-on: https://chromium-review.googlesource.com/c/1252776
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Reviewed-by: Aleks Totic <atotic@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599680}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already reviewed downstream.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1252776 branch 2 times, most recently from 4eeb0f6 to b29df61 Compare October 12, 2018 20:25
@chromium-wpt-export-bot chromium-wpt-export-bot changed the title [css-layout-api] Add LayoutEdges & LayoutEdgeSizes. [css-layout-api] Add LayoutEdges. Oct 12, 2018
This allows web developers to access pre-computed sizes for the border,
scrollbar, and padding "edges".

This adds two types of tests:
1) Uses the standard "green box" approach of testing the script values
   match what is expected.
2) Positions four children at each corner (using edges) are positioned
   the same as using an abs-pos technique.

Bug: 726125
Change-Id: I3b53be9b44989d919cb657d4eabc6d3a1a79181b
Reviewed-on: https://chromium-review.googlesource.com/c/1252776
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Reviewed-by: Aleks Totic <atotic@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599680}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants