Skip to content

Commit

Permalink
[wayland] removed pending initial bounds for popup.
Browse files Browse the repository at this point in the history
Removed |pending_initial_bounds_px_| which was no longer in use.

Bug: n/a
Change-Id: I098b1f8a4c18eff73f08697c1f06cc626a01cbf5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3227775
Commit-Queue: Alexander Dunaev <[email protected]>
Auto-Submit: Jun Liu <[email protected]>
Reviewed-by: Alexander Dunaev <[email protected]>
Cr-Commit-Position: refs/heads/main@{#932414}
  • Loading branch information
yjliu authored and Chromium LUCI CQ committed Oct 18, 2021
1 parent c674860 commit e46e227
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
6 changes: 1 addition & 5 deletions ui/ozone/platform/wayland/host/wayland_popup.cc
Original file line number Diff line number Diff line change
Expand Up @@ -37,11 +37,7 @@ WaylandPopup::~WaylandPopup() = default;
bool WaylandPopup::CreateShellPopup() {
DCHECK(parent_window() && !shell_popup_);

// Set pending initial bounds and notify the delegate.
if (!pending_initial_bounds_px_.IsEmpty()) {
SetBounds(pending_initial_bounds_px_);
pending_initial_bounds_px_ = gfx::Rect();
} else if (window_scale() != parent_window()->window_scale()) {
if (window_scale() != parent_window()->window_scale()) {
// If scale changed while this was hidden (when WaylandPopup hides, parent
// window's child is reset), update buffer scale accordingly.
UpdateWindowScale(true);
Expand Down
2 changes: 0 additions & 2 deletions ui/ozone/platform/wayland/host/wayland_popup.h
Original file line number Diff line number Diff line change
Expand Up @@ -59,8 +59,6 @@ class WaylandPopup : public WaylandWindow {

PlatformWindowShadowType shadow_type_ = PlatformWindowShadowType::kNone;

gfx::Rect pending_initial_bounds_px_;

// Helps to avoid reposition itself if HandlePopupConfigure was called, which
// resulted in calling SetBounds.
bool wayland_sets_bounds_ = false;
Expand Down

0 comments on commit e46e227

Please sign in to comment.