GitLab

This is an attempt to fix or at least mitigate crash https://crbug.com/1021490.

A base::WeakPtr is used in to bind XWindow::DelayedResize task to
delayed_resize_task_, which is a CancelableCallback, which is not needed and
potentially make it harder to understand the aforementioned crash. So use
base::Unretained() instead of WeakPtr in that case.

Additionally, dcheck Close() is called before XWindow's dtor.

Bug: 1021490
Change-Id: Ibf8b2f185dc020258905f6bf5e26305c51e7a215
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1917079
Commit-Queue: Nick Yamane <nickdiego@igalia.com>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#746917}