Summary: | [css-grid] Percentage columns shouldn't include border and padding | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Manuel Rego Casasnovas <rego> | ||||||||||
Component: | Layout and Rendering | Assignee: | Manuel Rego Casasnovas <rego> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | achristensen, commit-queue, esprehn+autocc, glenn, jfernandez, kondapallykalyan, svillar | ||||||||||
Priority: | P2 | ||||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Bug Depends on: | 149002 | ||||||||||||
Bug Blocks: | 60731 | ||||||||||||
Attachments: |
|
Description
Manuel Rego Casasnovas
2015-09-08 14:18:41 PDT
Created attachment 260792 [details]
Patch
Created attachment 260793 [details]
Patch
Comment on attachment 260793 [details] Patch Rejecting attachment 260793 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 260793, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: -- saving rejects to file Source/WebCore/rendering/RenderGrid.cpp.rej patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/fast/css-grid-layout/grid-percent-track-margin-border-padding-expected.txt patching file LayoutTests/fast/css-grid-layout/grid-percent-track-margin-border-padding.html Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Sergio Villar Senin']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/154060 Created attachment 260853 [details]
Patch for landing
Comment on attachment 260853 [details] Patch for landing Clearing flags on attachment: 260853 Committed r189538: <http://trac.webkit.org/changeset/189538> All reviewed patches have been landed. Closing bug. Re-opened since this is blocked by bug 149002 Sorry. r189536 wouldn't rollout without rolling this out, too, because this touched RenderGrid.cpp after that patch. This might be good. Created attachment 260869 [details]
Rebased patch for landing
Comment on attachment 260869 [details] Rebased patch for landing Clearing flags on attachment: 260869 Committed r189550: <http://trac.webkit.org/changeset/189550> All reviewed patches have been landed. Closing bug. |