Title: [91812] trunk/Source/WebCore
Revision
91812
Author
[email protected]
Date
2011-07-26 19:28:07 -0700 (Tue, 26 Jul 2011)

Log Message

[chromium] Avoid clearing the framebuffer when compositing in release builds
https://bugs.webkit.org/show_bug.cgi?id=65210

Reviewed by Kenneth Russell.

We clear the default framebuffer to blue when compositing so it's
easier to spot regions we miss. This is useful for debugging, but eats
bandwidth unnecessarily in release builds.

* platform/graphics/chromium/LayerRendererChromium.cpp:
(WebCore::LayerRendererChromium::drawLayers):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (91811 => 91812)


--- trunk/Source/WebCore/ChangeLog	2011-07-27 02:25:44 UTC (rev 91811)
+++ trunk/Source/WebCore/ChangeLog	2011-07-27 02:28:07 UTC (rev 91812)
@@ -1,3 +1,17 @@
+2011-07-26  James Robinson  <[email protected]>
+
+        [chromium] Avoid clearing the framebuffer when compositing in release builds
+        https://bugs.webkit.org/show_bug.cgi?id=65210
+
+        Reviewed by Kenneth Russell.
+
+        We clear the default framebuffer to blue when compositing so it's
+        easier to spot regions we miss. This is useful for debugging, but eats
+        bandwidth unnecessarily in release builds.
+
+        * platform/graphics/chromium/LayerRendererChromium.cpp:
+        (WebCore::LayerRendererChromium::drawLayers):
+
 2011-07-26  Jeffrey Pfau  <[email protected]>
 
         New XML tokenizer

Modified: trunk/Source/WebCore/platform/graphics/chromium/LayerRendererChromium.cpp (91811 => 91812)


--- trunk/Source/WebCore/platform/graphics/chromium/LayerRendererChromium.cpp	2011-07-27 02:25:44 UTC (rev 91811)
+++ trunk/Source/WebCore/platform/graphics/chromium/LayerRendererChromium.cpp	2011-07-27 02:28:07 UTC (rev 91812)
@@ -495,10 +495,12 @@
 
     useRenderSurface(m_defaultRenderSurface);
 
+#ifndef NDEBUG
     // Clear to blue to make it easier to spot unrendered regions.
     m_context->clearColor(0, 0, 1, 1);
     m_context->colorMask(true, true, true, true);
     m_context->clear(GraphicsContext3D::COLOR_BUFFER_BIT);
+#endif
     // Mask out writes to alpha channel: subpixel antialiasing via Skia results in invalid
     // zero alpha values on text glyphs. The root layer is always opaque.
     m_context->colorMask(true, true, true, false);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to