GitLab

Instead of calling XWindow::SetVisualId from DWTHX11:SetVisualId,
the latter was moved to DWTHLinux and now stores the pending
visual id value as base::Optional<int> and passes it to the
PlatformWindowInitProperties on DWTHLinux::AddAdditionalInitProperties
call.

Having SetVisualId calling XWindow is dangerous and leads to segfaults
and XWindow, which is X11Window, which is PlatformWindow, might not
exist by that time.

Bug: 1003347, 990756
Change-Id: Ic62a2974078f27f7f813e44919b51b6539783f2d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803274Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Commit-Queue: Maksim Sisov <msisov@igalia.com>
Cr-Commit-Position: refs/heads/master@{#698735}