Chromium Code Reviews Chromium Code Reviews
Issue
2829723004:
Revert of Do not hide shadow underlay for max/fullscreen state even if the client disables the shad… (Closed)
DescriptionRevert of Do not hide shadow underlay for max/fullscreen state even if the client disables the shadow. (patchset #3 id:40001 of https://codereview.chromium.org/2820493004/ )
Reason for revert:
This change broke basic WM interaction of ARC and caused browser crash. https://crbug.com/713560
Original issue's description:
> Do not hide shadow underlay for max/fullscreen state even if the client disables the shadow.
>
> During the state transition, ARC preserves windows being deleted,
> however the main window is deleted (thus, window frame is empty) which disables the shadow.
>
> BUG=711514
> TEST=covered by unit tests.
> manual. Instal & start "Clash Royal", then F4 to toggle fullscreen.
>
> Review-Url: https://codereview.chromium.org/2820493004
> Cr-Commit-Position: refs/heads/master@{#464819}
> Committed: https://chromium.googlesource.com/chromium/src/+/ae72f92d7caba6d32ba2a11f32943f29af15c3d8
TBR=reveman@chromium.org,oshima@chromium.org
BUG=713560
Review-Url: https://codereview.chromium.org/2827233002
Cr-Commit-Position: refs/heads/master@{#466063}
(cherry picked from commit b650cb773adbf72f3f731e0bc56e22d55c521391)
Review-Url: https://codereview.chromium.org/2829723004 .
Cr-Commit-Position: refs/branch-heads/3071@{#90}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}
Committed: https://chromium.googlesource.com/chromium/src/+/29b3bfe7c7371600d717d66502307b1e676d7c2c
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
||||||||||||||||||||||||||||
