Merge chromium and blink geometry classes |
|||||
Issue descriptionWe need to merge the geometry classes (ui/gfx/geometry/) with the same in base/. Related animation-specific work: http://crbug.com/577016 (the part for gfx::CubicBezier unification) This is a tracking bug.
,
Aug 9 2016
Probably related: Previously, the idea was to add WebFoo to gfx::Foo conversion operators for the case like this: https://cs.chromium.org/chromium/src/third_party/WebKit/Source/platform/animation/CompositorScrollOffsetAnimationCurve.cpp?type=cs&q=CompositorScrollOffsetAnimationCurve.cpp&sq=package:chromium&l=76 Anyway, we need to unify platform/geometry/FloatPoint.h and WebPoint and gfx::Point/gfx::ScrollOffset.
,
Aug 10 2016
Please make sure the same things are inline, extra copies aren't introduced, we keep the inline assembly versions, etc. :)
,
Aug 10 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Aug 10 2017
,
Nov 3 2017
I am interested to work on this. Can someone guide me? Thanks
,
Nov 3 2017
Have you read the blink-dev thread on using gfx types in blink? I think that laid out thoughts on how to move forward.
,
Nov 5
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Nov 5
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by danakj@chromium.org
, Aug 9 2016