Title: [124052] trunk/Source/WebCore
Revision
124052
Author
e...@google.com
Date
2012-07-30 11:00:29 -0700 (Mon, 30 Jul 2012)

Log Message

[chromium] Turn back on ScrollbarLayerChromium for Windows
https://bugs.webkit.org/show_bug.cgi?id=92409

Reviewed by James Robinson.

This reverts r122791. As GDI issues were addressed in Chromium via
http://src.chromium.org/viewvc/chrome?view=rev&revision=148215,
scrollbar parts can now be composited on the thread.

* page/scrolling/chromium/ScrollingCoordinatorChromium.cpp:
(WebCore::createScrollbarLayer):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (124051 => 124052)


--- trunk/Source/WebCore/ChangeLog	2012-07-30 17:56:38 UTC (rev 124051)
+++ trunk/Source/WebCore/ChangeLog	2012-07-30 18:00:29 UTC (rev 124052)
@@ -1,3 +1,17 @@
+2012-07-30  Adrienne Walker  <e...@google.com>
+
+        [chromium] Turn back on ScrollbarLayerChromium for Windows
+        https://bugs.webkit.org/show_bug.cgi?id=92409
+
+        Reviewed by James Robinson.
+
+        This reverts r122791. As GDI issues were addressed in Chromium via
+        http://src.chromium.org/viewvc/chrome?view=rev&revision=148215,
+        scrollbar parts can now be composited on the thread.
+
+        * page/scrolling/chromium/ScrollingCoordinatorChromium.cpp:
+        (WebCore::createScrollbarLayer):
+
 2012-07-30  Sheriff Bot  <webkit.review....@gmail.com>
 
         Unreviewed, rolling out r124025.

Modified: trunk/Source/WebCore/page/scrolling/chromium/ScrollingCoordinatorChromium.cpp (124051 => 124052)


--- trunk/Source/WebCore/page/scrolling/chromium/ScrollingCoordinatorChromium.cpp	2012-07-30 17:56:38 UTC (rev 124051)
+++ trunk/Source/WebCore/page/scrolling/chromium/ScrollingCoordinatorChromium.cpp	2012-07-30 18:00:29 UTC (rev 124052)
@@ -126,11 +126,8 @@
         scrollbarGraphicsLayer->setContentsOpaque(isOpaqueRootScrollbar);
 
     // FIXME: Mac scrollbar themes are not thread-safe to paint.
-    // FIXME: Win scrollbars on XP Classic themes do not paint valid alpha
-    // values due to GDI. This needs to be fixed in theme code before it
-    // can be turned on here.
     bool platformSupported = true;
-#if OS(DARWIN) || OS(WINDOWS)
+#if OS(DARWIN)
     platformSupported = false;
 #endif
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to