Mark layers that can use LCD text based on layer transform and opacity.

cc/tiled_layer.cc

 // Copyright 2011 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.
 
 #include "config.h"
 
 #include "cc/tiled_layer.h"
 
 #include "Region.h"
 #include "base/basictypes.h"
@@ skipping 258 matching lines @@
 
     return addedTile;
 }
 
 void TiledLayer::setNeedsDisplayRect(const gfx::RectF& dirtyRect)
 {
     invalidateContentRect(layerRectToContentRect(dirtyRect));
     ContentsScalingLayer::setNeedsDisplayRect(dirtyRect);
 }
 
-void TiledLayer::setUseLCDText(bool useLCDText)
+void TiledLayer::setCanUseLCDText(bool useLCDText)
 {
-    ContentsScalingLayer::setUseLCDText(useLCDText);
+    ContentsScalingLayer::setCanUseLCDText(useLCDText);
 
     LayerTilingData::BorderTexelOption borderTexelOption;
 #if OS(ANDROID)
+    DCHECK(!useLCDText);

[danakj, 2012/11/07 18:15:00]

This won't work well with making it a LayerTreeSetting.

Maybe DCHECK(layerTreeHost()->settings().canUseLCDText || !useLCDText) ?

[alokp, 2012/11/08 22:34:07]

I realized that if-else block is not needed anymore. For Android useLCDText is
always false, so it will always get border texels.

     // Always want border texels and GL_LINEAR due to pinch zoom.
     borderTexelOption = LayerTilingData::HasBorderTexels;
 #else
     borderTexelOption = useLCDText ? LayerTilingData::NoBorderTexels : LayerTilingData::HasBorderTexels;
 #endif
     setBorderTexelOption(borderTexelOption);
 }
 
 void TiledLayer::invalidateContentRect(const gfx::Rect& contentRect)
 {
@@ skipping 482 matching lines @@
     gfx::Rect prepaintRect = visibleContentRect();
     prepaintRect.Inset(-m_tiler->tileSize().width() * prepaintColumns,
                        -m_tiler->tileSize().height() * prepaintRows);
     gfx::Rect contentRect(gfx::Point(), contentBounds());
     prepaintRect.Intersect(contentRect);
 
     return prepaintRect;
 }
 
 }  // namespace cc