Summary: | Move Length-type CSS properties from DeprecatedStyleBuilder to the new Style Builder | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||||
Component: | CSS | Assignee: | Chris Dumez <cdumez> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | barraclough, benjamin, commit-queue, darin, kling, koivisto | ||||||||
Priority: | P2 | Keywords: | BlinkMergeCandidate | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
URL: | https://src.chromium.org/viewvc/blink?view=rev&revision=150500 | ||||||||||
Bug Depends on: | 137910 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Chris Dumez
2014-10-22 21:21:40 PDT
Created attachment 240328 [details]
WIP Patch
Created attachment 240329 [details]
Patch
Comment on attachment 240329 [details]
Patch
r=me, but I think we should keep the converter functions in the header file so they can be inlined as-needed in the generated code.
Created attachment 240346 [details]
Patch
(In reply to comment #3) > Comment on attachment 240329 [details] > Patch > > r=me, but I think we should keep the converter functions in the header file > so they can be inlined as-needed in the generated code. Ok, I moved them out-of-class in the header and marked them as inline. Comment on attachment 240346 [details] Patch Clearing flags on attachment: 240346 Committed r175123: <http://trac.webkit.org/changeset/175123> All reviewed patches have been landed. Closing bug. |