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

Issue 556043002: athena: Change the scrolling behaviour of the windows in overview mode. (Closed)

Created:
6 years, 3 months ago by sadrul
Modified:
6 years, 3 months ago
Reviewers:
oshima
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

athena: Change the scrolling behaviour of the windows in overview mode. Change the transforms of the windows in overview mode to better match the mocks. Notable change in the code is that instead of computing the terminal transforms at the start, and then computing the intermediate transforms from those, we compute less amount of information at start, and compute the transform at each state from those. This makes it easier to perform non-linear transforms during the scrolls. BUG=403849 R=oshima@chromium.org Committed: https://crrev.com/4bf71ed7540d1ead878e1891d6e819789863bf60 Cr-Commit-Position: refs/heads/master@{#294108}

Patch Set 1 #

Total comments: 8

Patch Set 2 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+66 lines, -46 lines) Patch
M athena/wm/window_overview_mode.cc View 1 11 chunks +66 lines, -46 lines 0 comments Download

Messages

Total messages: 6 (1 generated)
sadrul
6 years, 3 months ago (2014-09-09 22:03:14 UTC) #2
oshima
lgtm with nits https://codereview.chromium.org/556043002/diff/1/athena/wm/window_overview_mode.cc File athena/wm/window_overview_mode.cc (right): https://codereview.chromium.org/556043002/diff/1/athena/wm/window_overview_mode.cc#newcode43 athena/wm/window_overview_mode.cc:43: float min_y; please document them. https://codereview.chromium.org/556043002/diff/1/athena/wm/window_overview_mode.cc#newcode61 ...
6 years, 3 months ago (2014-09-10 00:19:42 UTC) #3
sadrul
https://codereview.chromium.org/556043002/diff/1/athena/wm/window_overview_mode.cc File athena/wm/window_overview_mode.cc (right): https://codereview.chromium.org/556043002/diff/1/athena/wm/window_overview_mode.cc#newcode43 athena/wm/window_overview_mode.cc:43: float min_y; On 2014/09/10 00:19:42, oshima wrote: > please ...
6 years, 3 months ago (2014-09-10 04:38:40 UTC) #4
sadrul
Committed patchset #2 (id:20001) to pending queue manually as f85c940 (presubmit successful).
6 years, 3 months ago (2014-09-10 05:18:07 UTC) #5
commit-bot: I haz the power
6 years, 3 months ago (2014-09-10 05:24:03 UTC) #6
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/4bf71ed7540d1ead878e1891d6e819789863bf60
Cr-Commit-Position: refs/heads/master@{#294108}

Powered by Google App Engine
This is Rietveld 408576698