Fixing crash in PictureLayerImpl::MarkVisibleResourcesAsRequired when low-res tiles are disabled

cc/trees/layer_tree_settings.h

Index: cc/trees/layer_tree_settings.h
diff --git a/cc/trees/layer_tree_settings.h b/cc/trees/layer_tree_settings.h
index e85ef7d4db27a62cbba2be0ac6b7cca9773856a4..3b529f8c94761b1371e57adb2789e3843eafdd16 100644
--- a/cc/trees/layer_tree_settings.h
+++ b/cc/trees/layer_tree_settings.h
@@ -15,7 +15,10 @@ namespace cc {
 
 class CC_EXPORT LayerTreeSettings {
  public:
-  LayerTreeSettings();
+  LayerTreeSettings(
+      bool impl_side_painting_ = false,
+      bool create_low_res_tiling_ = true
+  );

[reveman, 2014/05/13 15:38:05]

Not sure we should do this. Why should these settings be singled out and
adjustable using the ctor? If we add everything, then that would result in
cryptic calls to this ctor. I think it's better to keep the default ctor not
take any arguments and instead set them for each test case. Possible create some
derived class such as ImplSidePaintingSettings where convenient but keep those
in an anonymous namespace when possible.

[Sergey, 2014/05/14 05:53:35]

Done.

   ~LayerTreeSettings();
 
   bool impl_side_painting;