Summary: | Add parsing support for object-position | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> | ||||||||||
Component: | CSS | Assignee: | Simon Fraser (smfr) <simon.fraser> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | dino, sam, simon.fraser, zalan | ||||||||||
Priority: | P2 | ||||||||||||
Version: | Safari 9 | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 122811 | ||||||||||||
Attachments: |
|
Description
Simon Fraser (smfr)
2016-03-05 12:04:09 PST
Created attachment 273090 [details]
Patch
Created attachment 273092 [details]
Patch
Comment on attachment 273092 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=273092&action=review > Source/WebCore/css/CSSParser.cpp:2891 > + RefPtr<CSSPrimitiveValue> val1; > + RefPtr<CSSPrimitiveValue> val2; Can we use the whole word here? value1 and value2? > Source/WebCore/css/StyleBuilderConverter.h:319 > + Length length; Please move this to below where it is initialize. Created attachment 273093 [details]
Patch
Created attachment 273094 [details]
Patch
|