• Alexander Dunaev's avatar
    [ozone/wayland] Fixed TODO in WaylandWindow::SetBounds. · 118dd603
    Alexander Dunaev authored
    Implementations of PlatformWindow::SetBounds() in WaylandWindow and X11Window
    are different: the latter calls delegate's OnBoundsChanged always, while the
    former returns early if the new bounds were the same as the existing ones.
    There is a comment in the X11 implementation that says that we need to call
    delegate always because the call might be a result of the device scale factor
    being changed.  We had a TODO in the Wayland implementation saying that we
    need to investigate this and maybe call the delegate always as well.
    
    The simple experiment has shown that when the device scale factor is changed,
    the window gets new bounds that are not the same as the existing ones, so
    the delegate is called anyway.
    
    As a conclusion, no change is needed, and the TODO can be removed.
    
    R=msisov@igalia.com
    
    Bug: 958314
    Change-Id: I95fcffb43d09702d3dbcc6c2083d2087e7c166f1
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807823Reviewed-by: default avatarMaksim Sisov <msisov@igalia.com>
    Commit-Queue: Alexander Dunaev <adunaev@igalia.com>
    Cr-Commit-Position: refs/heads/master@{#697154}
    118dd603
wayland_window.cc 36.5 KB