Record SkPicture with correct LCD text setting.

ui/compositor/layer.cc

Index: ui/compositor/layer.cc
diff --git a/ui/compositor/layer.cc b/ui/compositor/layer.cc
index 17ab8993305a0c0c5b0c49879f7185ba785ce29d..5bb87ea48c76ee671ae8347a31fba27e0f7816be 100644
--- a/ui/compositor/layer.cc
+++ b/ui/compositor/layer.cc
@@ -654,6 +654,7 @@ void Layer::RequestCopyOfOutput(scoped_ptr<cc::CopyOutputRequest> request) {
 
 void Layer::PaintContents(SkCanvas* sk_canvas,
                           const gfx::Rect& clip,
+                          bool can_paint_lcd_text,
                           gfx::RectF* opaque,
                           ContentLayerClient::GraphicsContextStatus gc_status) {
   TRACE_EVENT0("ui", "Layer::PaintContents");
@@ -663,6 +664,10 @@ void Layer::PaintContents(SkCanvas* sk_canvas,
     delegate_->OnPaintLayer(canvas.get());
 }
 
+bool Layer::PaintsLCDText() const {
+  return false;

[alokp, 2014/06/10 20:54:43]

ui::Layer::PaintsLCDText returns false. If it returns true and lcd text is not
allowed it is reasonable that the layer is invalidated.

[piman, 2014/06/10 23:48:31]

I'm pretty sure the UI paints LCD text today. Is it ok to return false here?

[alokp, 2014/06/10 23:59:41]

Irrespective of cc::Layer::can_paint_lcd_text? If it does take that into
account, we would need to plumb the new can_paint_lcd_text argument on
ui::Layer::PaintContents (see above).

[piman, 2014/06/11 00:02:27]

Yes. I'm pretty sure that when that was added to cc::Layer, it was not plumbed
into the UI. The UI may be making the (slightly incorrect) assumption that
things are opaque and will work.

[alokp, 2014/06/11 01:01:36]

Hmm. In that case I agree that it is better to keep
ContentLayerClient::DidChangeLayerCanUseLCDText. I will update the patch.

+}
+
 bool Layer::FillsBoundsCompletely() const { return fills_bounds_completely_; }
 
 bool Layer::PrepareTextureMailbox(