New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 763377 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Merge Transform related methods in cc::MathUtil with gfx::Transform

Project Member Reported by staraz@chromium.org, Sep 8 2017

Issue description

cc::MathUtil has methods like MapEnclosingClippedRect, MapClippedRect, ProjectEnclosingClippedRect and ProjectClippedRect. They take in a gfx::Transform and a gfx::Rect and returns the transformed Rect after the operation.

Conceptually, they perform similar operations to gfx::Transform::TransformRect. The cc::MathUtil methods exist because the Transform code doesn't do the right thing when there is a perspective projection that causes one of the transformed vertices to go to w < 0.

Merge the transform methods into Transform code to enable wider usage and reduce the complexity inside MathUtil.
 
Cc: sky@chromium.org pkasting@chromium.org
Status: Available (was: Untriaged)

Comment 3 by fsamuel@google.com, Oct 24 2017

Owner: samans@chromium.org
Status: Assigned (was: Available)
+samans@ for triage.
Owner: ----
Status: Available (was: Assigned)

Comment 5 by danakj@chromium.org, May 25 2018

Blocking: -722935
All of cc::MathUtil should move out of cc, cuz it is used more broadly. This isn't related to layering between viz and cc specifically though, unblocking that.

Comment 6 by staraz@chromium.org, May 28 2018

Cc: -staraz@chromium.org

Sign in to add a comment