Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(92)

Issue 2315783002: Add ui::LayerObserver and use it to update Alt+Tab previews as needed. (Closed)

Created:
4 years, 3 months ago by Evan Stade
Modified:
4 years, 3 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2840
Project:
chromium
Visibility:
Public.

Description

Add ui::LayerObserver and use it to update Alt+Tab previews as needed. This fixes the Google Maps app part of the bug, and is necessary but not sufficient to fix the minimized window/swapped-out renderer part of the bug. BUG=636594 Review-Url: https://codereview.chromium.org/2254733003 Cr-Commit-Position: refs/heads/master@{#416071} (cherry picked from commit 696aa48c01ff79afbde347312f790fb54406d430)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+138 lines, -62 lines) Patch
M ash/common/wm/forwarding_layer_delegate.h View 2 chunks +20 lines, -10 lines 0 comments Download
M ash/common/wm/forwarding_layer_delegate.cc View 2 chunks +38 lines, -20 lines 0 comments Download
M ash/wm/drag_window_controller.cc View 1 chunk +4 lines, -4 lines 0 comments Download
M ash/wm/window_mirror_view.h View 1 chunk +2 lines, -1 line 0 comments Download
M ash/wm/window_mirror_view.cc View 1 chunk +4 lines, -5 lines 0 comments Download
M ui/compositor/BUILD.gn View 1 chunk +1 line, -0 lines 0 comments Download
M ui/compositor/layer.h View 4 chunks +7 lines, -0 lines 0 comments Download
M ui/compositor/layer.cc View 6 chunks +17 lines, -3 lines 0 comments Download
A ui/compositor/layer_observer.h View 1 chunk +26 lines, -0 lines 0 comments Download
M ui/wm/core/window_util.h View 1 chunk +4 lines, -4 lines 0 comments Download
M ui/wm/core/window_util.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M ui/wm/core/window_util_unittest.cc View 4 chunks +13 lines, -13 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Evan Stade
4 years, 3 months ago (2016-09-06 18:34:46 UTC) #1
Message was sent while issue was closed.
Committed patchset #1 (id:1) to pending queue manually as
ec6fc7355f1df38cfd7509a66b9fbede8fe7d657.

Powered by Google App Engine
This is Rietveld 408576698