New issue
Advanced search Search tips

Issue 847551 link

Starred by 1 user
Status: Available
Owner: ----
Cc:
dnicoara@chromium.org
dcasta...@chromium.org
afakhry@chromium.org
dsinclair@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Unify the NativeDisplayDelegate API to use the same type of display ID

Project Member Reported by afakhry@chromium.org, May 29 2018 
Either use the |int64_t| display ID, or the |DisplaySnapshot|, not both.

This comment came along during this review: https://chromium-review.googlesource.com/c/chromium/src/+/1070317/11/ui/display/types/native_display_delegate.h#85

Comment 1 by dnicoara@chromium.org, May 29 2018 

Just to add more detail: It was previously necessary to use DisplaySnapshot as an argument. However as the display management code was cleaned up I believe it should be sufficient to pass the display ID as the identifier.

Comment 2 by khushals...@chromium.org, Jun 1 2018

Status: Available (was: Untriaged)
►

Sign in to add a comment