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

Unified Diff: cc/layers/scrollbar_layer_impl.cc

Issue 18341009: Refactor cc scrollbar layers to separate solid-color vs desktop. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address tfarina@'s comments. Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/layers/scrollbar_layer_impl.cc
diff --git a/cc/layers/scrollbar_layer_impl.cc b/cc/layers/scrollbar_layer_impl.cc
index 4160bf0f5b4ff800e480e0312fbc3227174da93b..aafbc2cab995de1806e945b5b4566360b0e9f788 100644
--- a/cc/layers/scrollbar_layer_impl.cc
+++ b/cc/layers/scrollbar_layer_impl.cc
@@ -5,7 +5,6 @@
#include "cc/layers/scrollbar_layer_impl.h"
#include "cc/animation/scrollbar_animation_controller.h"
-#include "cc/layers/layer.h"
#include "cc/layers/quad_sink.h"
#include "cc/quads/solid_color_draw_quad.h"
#include "cc/quads/texture_draw_quad.h"
@@ -28,19 +27,13 @@ ScrollbarLayerImpl::ScrollbarLayerImpl(
LayerTreeImpl* tree_impl,
int id,
ScrollbarOrientation orientation)
- : LayerImpl(tree_impl, id),
+ : ScrollbarLayerImplBase(tree_impl, id, orientation),
track_resource_id_(0),
thumb_resource_id_(0),
- current_pos_(0.f),
- maximum_(0),
thumb_thickness_(0),
thumb_length_(0),
track_start_(0),
track_length_(0),
- orientation_(orientation),
- vertical_adjust_(0.f),
- visible_to_total_length_ratio_(1.f),
- scroll_layer_id_(Layer::INVALID_ID),
is_scrollable_area_active_(false),
is_scroll_view_scrollbar_(false),
enabled_(false),
@@ -49,10 +42,6 @@ ScrollbarLayerImpl::ScrollbarLayerImpl(
ScrollbarLayerImpl::~ScrollbarLayerImpl() {}
-ScrollbarLayerImpl* ScrollbarLayerImpl::ToScrollbarLayer() {
- return this;
-}
-
scoped_ptr<LayerImpl> ScrollbarLayerImpl::CreateLayerImpl(
LayerTreeImpl* tree_impl) {
return ScrollbarLayerImpl::Create(tree_impl,
@@ -61,7 +50,7 @@ scoped_ptr<LayerImpl> ScrollbarLayerImpl::CreateLayerImpl(
}
void ScrollbarLayerImpl::PushPropertiesTo(LayerImpl* layer) {
- LayerImpl::PushPropertiesTo(layer);
+ ScrollbarLayerImplBase::PushPropertiesTo(layer);
ScrollbarLayerImpl* scrollbar_layer = static_cast<ScrollbarLayerImpl*>(layer);
@@ -76,12 +65,19 @@ void ScrollbarLayerImpl::PushPropertiesTo(LayerImpl* layer) {
bool ScrollbarLayerImpl::WillDraw(DrawMode draw_mode,
ResourceProvider* resource_provider) {
- if (draw_mode == DRAW_MODE_RESOURCELESS_SOFTWARE &&
- !layer_tree_impl()->settings().solid_color_scrollbars)
+ if (draw_mode == DRAW_MODE_RESOURCELESS_SOFTWARE)
return false;
return LayerImpl::WillDraw(draw_mode, resource_provider);
}
+gfx::Rect ScrollbarLayerImpl::ComputeThumbQuadRect() const {
+ float track_length = track_length_;
+ if (orientation_ == VERTICAL)
+ track_length += vertical_adjust_;
+ return ComputeThumbQuadRectInternal(
+ thumb_length_, track_length_, track_start_, thumb_thickness_);
+}
+
void ScrollbarLayerImpl::AppendQuads(QuadSink* quad_sink,
AppendQuadsData* append_quads_data) {
bool premultipled_alpha = true;
@@ -97,16 +93,6 @@ void ScrollbarLayerImpl::AppendQuads(QuadSink* quad_sink,
gfx::Rect thumb_quad_rect = ComputeThumbQuadRect();
- if (layer_tree_impl()->settings().solid_color_scrollbars) {
- scoped_ptr<SolidColorDrawQuad> quad = SolidColorDrawQuad::Create();
- quad->SetNew(shared_quad_state,
- thumb_quad_rect,
- layer_tree_impl()->settings().solid_color_scrollbar_color,
- false);
- quad_sink->Append(quad.PassAs<DrawQuad>(), append_quads_data);
- return;
- }
-
if (thumb_resource_id_ && !thumb_quad_rect.IsEmpty()) {
gfx::Rect opaque_rect;
const float opacity[] = {1.0f, 1.0f, 1.0f, 1.0f};
@@ -147,117 +133,6 @@ void ScrollbarLayerImpl::AppendQuads(QuadSink* quad_sink,
}
}
-ScrollbarOrientation ScrollbarLayerImpl::Orientation() const {
- return orientation_;
-}
-
-float ScrollbarLayerImpl::CurrentPos() const {
- return current_pos_;
-}
-
-int ScrollbarLayerImpl::Maximum() const {
- return maximum_;
-}
-
-gfx::Rect ScrollbarLayerImpl::ScrollbarLayerRectToContentRect(
- gfx::RectF layer_rect) const {
- // Don't intersect with the bounds as in layerRectToContentRect() because
- // layer_rect here might be in coordinates of the containing layer.
- gfx::RectF content_rect = gfx::ScaleRect(layer_rect,
- contents_scale_x(),
- contents_scale_y());
- return gfx::ToEnclosingRect(content_rect);
-}
-
-gfx::Rect ScrollbarLayerImpl::ComputeThumbQuadRect() const {
- // Thumb extent is the length of the thumb in the scrolling direction, thumb
- // thickness is in the perpendicular direction. Here's an example of a
- // horizontal scrollbar - inputs are above the scrollbar, computed values
- // below:
- //
- // |<------------------- track_length_ ------------------->|
- //
- // |--| <-- start_offset
- //
- // +--+----------------------------+------------------+-------+--+
- // |<|| |##################| ||>|
- // +--+----------------------------+------------------+-------+--+
- //
- // |<- thumb_length ->|
- //
- // |<------- thumb_offset -------->|
- //
- // For painted, scrollbars, the length is fixed. For solid color scrollbars we
- // have to compute it. The ratio of the thumb's length to the track's length
- // is the same as that of the visible viewport to the total viewport, unless
- // that would make the thumb's length less than its thickness.
- //
- // vertical_adjust_ is used when the layer geometry from the main thread is
- // not in sync with what the user sees. For instance on Android scrolling the
- // top bar controls out of view reveals more of the page content. We want the
- // root layer scrollbars to reflect what the user sees even if we haven't
- // received new layer geometry from the main thread. If the user has scrolled
- // down by 50px and the initial viewport size was 950px the geometry would
- // look something like this:
- //
- // vertical_adjust_ = 50, scroll position 0, visible ratios 99%
- // Layer geometry: Desired thumb positions:
- // +--------------------+-+ +----------------------+ <-- 0px
- // | |v| | #|
- // | |e| | #|
- // | |r| | #|
- // | |t| | #|
- // | |i| | #|
- // | |c| | #|
- // | |a| | #|
- // | |l| | #|
- // | | | | #|
- // | |l| | #|
- // | |a| | #|
- // | |y| | #|
- // | |e| | #|
- // | |r| | #|
- // +--------------------+-+ | #|
- // | horizontal layer | | | #|
- // +--------------------+-+ | #| <-- 950px
- // | | | #|
- // | | |##################### |
- // +----------------------+ +----------------------+ <-- 1000px
- //
- // The layer geometry is set up for a 950px tall viewport, but the user can
- // actually see down to 1000px. Thus we have to move the quad for the
- // horizontal scrollbar down by the vertical_adjust_ factor and lay the
- // vertical thumb out on a track lengthed by the vertical_adjust_ factor. This
- // means the quads may extend outside the layer's bounds.
-
- int thumb_length = thumb_length_;
- float track_length = track_length_;
- if (orientation_ == VERTICAL)
- track_length += vertical_adjust_;
-
- if (layer_tree_impl()->settings().solid_color_scrollbars) {
- thumb_length = std::max(
- static_cast<int>(visible_to_total_length_ratio_ * track_length),
- thumb_thickness_);
- }
-
- // With the length known, we can compute the thumb's position.
- float ratio = current_pos_ / maximum_;
- float max_offset = track_length - thumb_length;
- int thumb_offset = static_cast<int>(ratio * max_offset) + track_start_;
-
- gfx::RectF thumb_rect;
- if (orientation_ == HORIZONTAL) {
- thumb_rect = gfx::RectF(thumb_offset, vertical_adjust_,
- thumb_length, thumb_thickness_);
- } else {
- thumb_rect = gfx::RectF(0.f, thumb_offset,
- thumb_thickness_, thumb_length);
- }
-
- return ScrollbarLayerRectToContentRect(thumb_rect);
-}
-
void ScrollbarLayerImpl::DidLoseOutputSurface() {
track_resource_id_ = 0;
thumb_resource_id_ = 0;

Powered by Google App Engine
This is Rietveld 408576698