New issue
Advanced search Search tips

Issue 597090 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Move display layout parenting logic to C++

Project Member Reported by steve...@chromium.org, Mar 22 2016

Issue description

Currently the display layout re-parenting logic for 3+ displays resides in JS and is used by the UI when the layout tree structure changes.

We should move this logic into C++ so that it can be shared with the model when a new display is added or the size/resolution changes.

Since we only apply this logic when a drag event completes, any minor lag from the JS/C++ round trip should not be apparent to the user. (The UI will receive a complete update regardless and will recenter and resize the new layout).

 
Labels: Pri-3
Cc: afakhry@chromium.org
I added some logic to reparent displays while applying the layout and de-intersecting displays here: https://cs.chromium.org/chromium/src/ui/display/display_layout.cc?type=cs&q=MaybeReparentTargetDisplay&l=136. Maybe we can extend and reuse that.
Cc: steve...@chromium.org
Owner: malaykeshav@chromium.org
This is not somehting I'm going to have to work on any time soon. -> malaykeshav@ to consider doing or to archive.

Sign in to add a comment