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

Unified Diff: cc/content_layer.cc

Issue 11360093: Mark layers that can use LCD text based on layer transform and opacity. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixed build Created 8 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/content_layer.h ('k') | cc/content_layer_unittest.cc » ('j') | cc/layer.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/content_layer.cc
diff --git a/cc/content_layer.cc b/cc/content_layer.cc
index b34ce107e0ced197af3d767f4dc3b8c67f74016f..fcf714bcb8bd15ac935a82b9b409b2f319baa442 100644
--- a/cc/content_layer.cc
+++ b/cc/content_layer.cc
@@ -4,6 +4,7 @@
#include "cc/content_layer.h"
+#include "base/debug/trace_event.h"
#include "base/metrics/histogram.h"
#include "base/time.h"
#include "cc/bitmap_content_layer_updater.h"
@@ -14,6 +15,9 @@
namespace cc {
+// Maximum number of times LCD text setting can be changed.
+static const int kLCDTextMaxChangeCount = 1;
+
ContentLayerPainter::ContentLayerPainter(ContentLayerClient* client)
: m_client(client)
{
@@ -42,6 +46,7 @@ scoped_refptr<ContentLayer> ContentLayer::create(ContentLayerClient* client)
ContentLayer::ContentLayer(ContentLayerClient* client)
: TiledLayer()
, m_client(client)
+ , m_lcdTextChangeCount(0)
{
}
@@ -103,4 +108,28 @@ void ContentLayer::setContentsOpaque(bool opaque)
m_updater->setOpaque(opaque);
}
+bool ContentLayer::canUseLCDTextWillChange()
+{
+ // Always allow disabling LCD text.
+ if (canUseLCDText())
+ return true;
+
+ return m_lcdTextChangeCount < kLCDTextMaxChangeCount;
+}
+
+void ContentLayer::canUseLCDTextDidChange()
+{
+ TiledLayer::canUseLCDTextDidChange();
+
+ if (m_lcdTextChangeCount > 0) {
+ // Do not record the first time LCD text is enabled because
+ // it does not really cause any invalidation.
+ TRACE_EVENT_INSTANT0("cc", "ContentLayer::canUseLCDTextDidChange");
+ }
+ ++m_lcdTextChangeCount;
+
+ // Need to repaint the layer with different text AA setting.
+ setNeedsDisplay();
+}
+
} // namespace cc
« no previous file with comments | « cc/content_layer.h ('k') | cc/content_layer_unittest.cc » ('j') | cc/layer.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698