diff options
author | bokan <bokan@chromium.org> | 2015-03-25 17:10:37 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-03-26 00:11:24 +0000 |
commit | aa27483190bdfbe34189e2d39c355dfda131fa8a (patch) | |
tree | 4ae09adfde05f24e8dabd397daae9f99d14a1a2f /cc/cc.gyp | |
parent | 170b95cf9565b12eb42d419400740b81ce6d7721 (diff) | |
download | chromium_src-aa27483190bdfbe34189e2d39c355dfda131fa8a.zip chromium_src-aa27483190bdfbe34189e2d39c355dfda131fa8a.tar.gz chromium_src-aa27483190bdfbe34189e2d39c355dfda131fa8a.tar.bz2 |
Move viewport scrolling logic into separate class
The viewport is made up of two scrolling layers now, the inner
and outer viewport scroll layers. These layers do not follow the
usual scroll bubbling logic since they're supposed to appear as
one viewport to the user (i.e. we should rail). Forcing it to
be scrolled via LayerTreeHostImpl::ScrollBy caused that method to
become increasingly complex.
This patch creates a Viewport class that allows LTHI::ScrollBy to
call Viewport::ScrollBy and scroll the viewport as a unit. I've
tried to simplify the logic in both these methods as much as I
could without changing behavior.
BUG=443724
Review URL: https://codereview.chromium.org/986443003
Cr-Commit-Position: refs/heads/master@{#322268}
Diffstat (limited to 'cc/cc.gyp')
-rw-r--r-- | cc/cc.gyp | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -248,6 +248,8 @@ 'layers/video_layer.h', 'layers/video_layer_impl.cc', 'layers/video_layer_impl.h', + 'layers/viewport.cc', + 'layers/viewport.h', 'output/begin_frame_args.cc', 'output/begin_frame_args.h', 'output/bsp_tree.cc', |