New issue
Advanced search Search tips

Issue 875382 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 9
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome
Pri: 2
Type: Bug



Sign in to add a comment

[Widget] Excess NOTREACHED logs when window is being dragged.

Project Member Reported by apaci...@chromium.org, Aug 17

Issue description

There is currently a NOTREACHED() when getting the HitTest while the
window is resizing. This adds a lot of noise in the logs when the
window is being dragged.

This only happens when aspect ratio is set for a window.

This behavior is expected to be unhandled here, hence the current default handling.
 
Status: Available (was: Assigned)
Owner: ----
Owner: mlamouri@chromium.org
Status: Started (was: Available)
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 9

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2a2d916562540ef4c0ca3ee33b2ffecf70e1e29b

commit 2a2d916562540ef4c0ca3ee33b2ffecf70e1e29b
Author: Mounir Lamouri <mlamouri@chromium.org>
Date: Tue Oct 09 20:35:43 2018

Widget: only enforce aspect ratio when resizing.

THis is fixing a NOTREACHED() error when running
GetWindowResizeHitTest() as it expects the window to be resized, not
dragged.

Bug:  875382 
Change-Id: I0cc3a8323427818a885d10c70852b0682fc5f865
Reviewed-on: https://chromium-review.googlesource.com/c/1262408
Reviewed-by: Robert Flack <flackr@chromium.org>
Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#598067}
[modify] https://crrev.com/2a2d916562540ef4c0ca3ee33b2ffecf70e1e29b/ash/wm/default_window_resizer_unittest.cc
[modify] https://crrev.com/2a2d916562540ef4c0ca3ee33b2ffecf70e1e29b/ash/wm/window_resizer.cc

Status: Fixed (was: Started)

Sign in to add a comment