Summary: | REGRESSION(r189567): Elements with aspect ratios are not handled correctly inside flexbox | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dave Hyatt <hyatt> | ||||
Component: | Layout and Rendering | Assignee: | Dave Hyatt <hyatt> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, esprehn+autocc, glenn, jonlee, kondapallykalyan, simon.fraser, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=146020 | ||||||
Attachments: |
|
Description
Dave Hyatt
2016-05-24 14:05:10 PDT
Created attachment 279702 [details]
Patch
Attachment 279702 [details] did not pass style-queue:
ERROR: Source/WebCore/rendering/RenderFlexibleBox.cpp:878: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4]
ERROR: Source/WebCore/rendering/RenderFlexibleBox.cpp:879: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4]
ERROR: Source/WebCore/rendering/RenderFlexibleBox.cpp:885: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4]
ERROR: Source/WebCore/rendering/RenderFlexibleBox.cpp:886: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4]
ERROR: Source/WebCore/rendering/RenderFlexibleBox.cpp:887: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4]
ERROR: Source/WebCore/rendering/RenderFlexibleBox.cpp:888: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4]
ERROR: Source/WebCore/rendering/RenderFlexibleBox.cpp:889: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4]
ERROR: Source/WebCore/rendering/RenderFlexibleBox.cpp:892: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4]
ERROR: Source/WebCore/rendering/RenderFlexibleBox.cpp:948: An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement. [readability/control_flow] [4]
Total errors found: 9 in 7 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 279702 [details] Patch Clearing flags on attachment: 279702 Committed r201516: <http://trac.webkit.org/changeset/201516> All reviewed patches have been landed. Closing bug. |