diff options
author | mkwst@chromium.org <mkwst@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-24 12:10:25 +0000 |
---|---|---|
committer | mkwst@chromium.org <mkwst@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-24 12:10:25 +0000 |
commit | 7b14f39769532717317b8a48d0fc03bc66a6cc0b (patch) | |
tree | 7ad4bd344d4ce65d36a6945d36e37adf9dcaac07 /cc/test/fake_painted_scrollbar_layer.h | |
parent | 10587c5fbaba1b747884792a8e6e6403f73d7fe1 (diff) | |
download | chromium_src-7b14f39769532717317b8a48d0fc03bc66a6cc0b.zip chromium_src-7b14f39769532717317b8a48d0fc03bc66a6cc0b.tar.gz chromium_src-7b14f39769532717317b8a48d0fc03bc66a6cc0b.tar.bz2 |
Revert of Pinch/Zoom Infrastructure & Plumbing CL (https://codereview.chromium.org/23983047/)
Reason for revert:
This patch broke several blink-side compositing/rtl tests[1][2]. It's
unclear whether these tests are valuable, given some comments
in TestExpectations, but since I know nothing about compositing
I'll leave the decision about what to do with these tests up to
someone who does.
[1]: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20Blink&tests=compositing/rtl/rtl-iframe-absolute-overflow-scrolled.html,compositing/rtl/rtl-iframe-absolute-overflow.html,compositing/rtl/rtl-iframe-fixed-overflow-scrolled.html,compositing/rtl/rtl-iframe-fixed-overflow.html
[2]: http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.7/builds/22131
Original issue's description:
> Pinch/Zoom Infrastructure & Plumbing CL
>
> This CL supplies the necessary changes to CC to support the
> inner/outer viewport model for pinch-zoom and fixed-position
> elements. The specification for these changes is contained in
> the document "Layer-based Solution for Pinch Zoom / Fixed
> Position".
>
> It incorporates a change to how scrollbar parameters are
> computed (removes the notion of max_scroll_offset as a
> quantity set be the embedder, and instead inferred from the
> relative sizes of a clip layer w.r.t. the scroll layer).
>
> Scrollbars are generalized so that a layer may have more than
> two scrollbars, and the parameters of the scrollbar are set
> w.r.t. the sizes and positions of a clip and a scroll layer.
> Further, changes to the scrip/scroll layer automatically
> notify any attached scrollbars.
>
> The CL also removes existing references to root_scroll_layer
> from LTH, LTI and LTHI and replaces them with either
> Inner/OuterViewportScrollLayer (dual-layer operation only
> exists at present if the --enable-pinch-virtual-viewport flag
> is specified, otherwise behavior is unchanged).
>
> Logic is added to (i) combine scroll offsets for the two
> viewports before passing it to the embedder, and (ii)
> splitting any offsets received from the embedder
> appropriately between the two viewports.
>
> This CL relies on https://codereview.chromium.org/138453004/
> for changes in Blink to support it.
>
> BUG=148816
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=246812
TBR=enne@chromium.org,aelias@chromium.org,joi@chromium.org,sky@chromium.org,piman@chromium.org,tony@chromium.org,wjmaclean@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=148816
Review URL: https://codereview.chromium.org/146713002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@246842 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/test/fake_painted_scrollbar_layer.h')
-rw-r--r-- | cc/test/fake_painted_scrollbar_layer.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/cc/test/fake_painted_scrollbar_layer.h b/cc/test/fake_painted_scrollbar_layer.h index 736a450..9b1b13d 100644 --- a/cc/test/fake_painted_scrollbar_layer.h +++ b/cc/test/fake_painted_scrollbar_layer.h @@ -16,7 +16,7 @@ namespace cc { class FakePaintedScrollbarLayer : public PaintedScrollbarLayer { public: static scoped_refptr<FakePaintedScrollbarLayer> - Create(bool paint_during_update, bool has_thumb, Layer* scrolling_layer); + Create(bool paint_during_update, bool has_thumb, int scrolling_layer_id); int update_count() const { return update_count_; } void reset_update_count() { update_count_ = 0; } @@ -44,7 +44,7 @@ class FakePaintedScrollbarLayer : public PaintedScrollbarLayer { private: FakePaintedScrollbarLayer(FakeScrollbar* fake_scrollbar, - Layer* scrolling_layer); + int scrolling_layer_id); virtual ~FakePaintedScrollbarLayer(); int update_count_; |