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

Unified Diff: cc/layers/layer_impl.h

Issue 102733006: Use bit fields inside cc to reduce memory usage. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Modified after the review comments! Created 7 years 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
« no previous file with comments | « cc/layers/layer.cc ('k') | cc/layers/layer_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layers/layer_impl.h
diff --git a/cc/layers/layer_impl.h b/cc/layers/layer_impl.h
index 13e2876880f134cb1241d599430585b93b9c0dee..35147913e2acc95baf79bd5b8cd6901a588c3817 100644
--- a/cc/layers/layer_impl.h
+++ b/cc/layers/layer_impl.h
@@ -52,7 +52,6 @@ class QuadSink;
class Renderer;
class ScrollbarAnimationController;
class ScrollbarLayerImplBase;
-class Layer;
struct AppendQuadsData;
@@ -576,40 +575,40 @@ class CC_EXPORT LayerImpl : public LayerAnimationValueObserver,
gfx::Size bounds_;
gfx::Vector2d scroll_offset_;
LayerScrollOffsetDelegate* scroll_offset_delegate_;
- bool scrollable_;
- bool should_scroll_on_main_thread_;
- bool have_wheel_event_handlers_;
- bool user_scrollable_horizontal_;
- bool user_scrollable_vertical_;
- Region non_fast_scrollable_region_;
- Region touch_event_handler_region_;
- SkColor background_color_;
- bool stacking_order_changed_;
-
+ bool scrollable_ : 1;
+ bool should_scroll_on_main_thread_ : 1;
+ bool have_wheel_event_handlers_ : 1;
+ bool user_scrollable_horizontal_ : 1;
+ bool user_scrollable_vertical_ : 1;
+ bool stacking_order_changed_ : 1;
// Whether the "back" of this layer should draw.
- bool double_sided_;
+ bool double_sided_ : 1;
// Tracks if drawing-related properties have changed since last redraw.
- bool layer_property_changed_;
+ bool layer_property_changed_ : 1;
+
+ bool masks_to_bounds_ : 1;
+ bool contents_opaque_ : 1;
+ bool is_root_for_isolated_group_ : 1;
+ bool preserves_3d_ : 1;
+ bool use_parent_backface_visibility_ : 1;
+ bool draw_checkerboard_for_missing_tiles_ : 1;
+ bool draws_content_ : 1;
+ bool hide_layer_and_subtree_ : 1;
+ bool force_render_surface_ : 1;
+
+ // Set for the layer that other layers are fixed to.
+ bool is_container_for_fixed_position_layers_ : 1;
+ Region non_fast_scrollable_region_;
+ Region touch_event_handler_region_;
+ SkColor background_color_;
- bool masks_to_bounds_;
- bool contents_opaque_;
float opacity_;
SkXfermode::Mode blend_mode_;
- bool is_root_for_isolated_group_;
gfx::PointF position_;
- bool preserves_3d_;
- bool use_parent_backface_visibility_;
- bool draw_checkerboard_for_missing_tiles_;
gfx::Transform sublayer_transform_;
gfx::Transform transform_;
- bool draws_content_;
- bool hide_layer_and_subtree_;
- bool force_render_surface_;
-
- // Set for the layer that other layers are fixed to.
- bool is_container_for_fixed_position_layers_;
// This property is effective when
// is_container_for_fixed_position_layers_ == true,
gfx::Vector2dF fixed_container_size_delta_;
« no previous file with comments | « cc/layers/layer.cc ('k') | cc/layers/layer_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698