cc: Toggle LCD text at raster time instead of record time.

cc/layers/content_layer.h

 // Copyright 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CC_LAYERS_CONTENT_LAYER_H_
 #define CC_LAYERS_CONTENT_LAYER_H_
 
 #include "base/basictypes.h"
 #include "cc/base/cc_export.h"
 #include "cc/layers/tiled_layer.h"
@@ skipping 51 matching lines @@
   LayerUpdater* Updater() const override;
 
  private:
   // TiledLayer implementation.
   void CreateUpdaterIfNeeded() override;
 
   void UpdateCanUseLCDText();
 
   ContentLayerClient* client_;
   scoped_refptr<ContentLayerUpdater> updater_;
-  bool can_use_lcd_text_last_frame_;
+  bool can_use_lcd_text_for_update_;
+  bool ignore_lcd_text_change_;

[reveman, 2014/11/11 21:16:49]

Suggestion that you can ignore if you like as this is not something new to this
patch:

We could use an enum here instead of two bools. ie. "enum {
CAN_USE_LCD_TEXT_UNKNOWN, CAN_USE_LCD_TEXT_TRUE, CAN_USE_LCD_TEXT_FALSE }". I
think that will make it easier to see what state transitions are allowed.

[danakj, 2014/11/12 17:22:10]

Done. Though poking around here I realized content layer doesn't actually do
anything with this LCD flag anymore (ui::Layer doesn't take any flag for it). So
I removed it from here, but I did the enum thing in PictureLayer.

 
   DISALLOW_COPY_AND_ASSIGN(ContentLayer);
 };
 
 }  // namespace cc
 #endif  // CC_LAYERS_CONTENT_LAYER_H_