session restore doesn't restore windows correctly with multi displays |
|||
Issue descriptionRepro step: 1) Create and plac two browser windows, one in internal display, another in external display. 2) Log out 3) Log in Expected: two windows should be restored in original position in each displays Actual: two windows are restored in one display. warx@ can you check if 67 has this issue first?
,
Apr 25 2018
Re #0 and #1, this bug also happens on m67: 67.0.3396.16, so it is not a regression from issue 834775 .
,
Apr 25 2018
OK, so something different, oh joy :) +msw@ just in case this is mustash related.
,
Apr 25 2018
Yeah, this only seems to repro if #mus is enabled in about:flags. Evan, could this be related to Mus' general window positioning? Otherwise, maybe the external display is being destroyed before closing the window or not initialized before we recreate the window, so it's placed on the internal display as a fallback. Either way, we're putting the current window service incarnation on hold for now. This shouldn't reach any release channels (it's only on for local dev builds). We can reduce priority here for now, and circle back later.
,
Apr 25 2018
Is mus enabled on tot + dev build?
,
Apr 25 2018
Yes
,
Apr 25 2018
,
May 14 2018
,
May 14 2018
msw@ who is working on new window service as a library?
,
May 14 2018
sky@ is leading the effort with support from jamescook@, estade@, stevenjb@, and myself.
,
Sep 5
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/215736ef840b1d6fb28e57a10fcd38ebc2a617ec commit 215736ef840b1d6fb28e57a10fcd38ebc2a617ec Author: Evan Stade <estade@chromium.org> Date: Tue Sep 04 21:40:12 2018 Mash: place windows on correct displays during session restore. Bug: 836449 Change-Id: Ic69bffc92dc7a033ae970e81399f16dd0cd64260 Reviewed-on: https://chromium-review.googlesource.com/1205010 Reviewed-by: James Cook <jamescook@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#588645} [modify] https://crrev.com/215736ef840b1d6fb28e57a10fcd38ebc2a617ec/ash/wm/top_level_window_factory.cc |
|||
►
Sign in to add a comment |
|||
Comment 1 by steve...@chromium.org
, Apr 24 2018