Title: [152083] trunk
Revision
152083
Author
commit-qu...@webkit.org
Date
2013-06-27 00:49:32 -0700 (Thu, 27 Jun 2013)

Log Message

Source/WebCore: Remove ENABLE_CSS_COMPOSITING guards around -webkit-background-blend mode related code. The same flag guards -webkit-blend-mode code, but those guards are not touched by this change.

https://bugs.webkit.org/show_bug.cgi?id=117619

Patch by Horia Iosif Olaru <ol...@adobe.com> on 2013-06-27
Reviewed by Dean Jackson.

Test: css3/compositing/effect-background-blend-mode-tiled.html

* WebCore.exp.in:
* bindings/generic/RuntimeEnabledFeatures.h:
* css/CSSComputedStyleDeclaration.cpp:
(WebCore::ComputedStyleExtractor::propertyValue):
* css/CSSParser.cpp:
(WebCore::CSSParser::parseValue):
(WebCore::CSSParser::parseFillProperty):
* css/CSSProperty.cpp:
(WebCore::CSSProperty::isInheritedProperty):
* css/CSSPropertyNames.in:
* css/DeprecatedStyleBuilder.cpp:
(WebCore::DeprecatedStyleBuilder::DeprecatedStyleBuilder):

LayoutTests: Add an extra test to check if tiled background images get blended as expected. This test passed before this change. The tests for this change are the existing background-blend-mode tests.

https://bugs.webkit.org/show_bug.cgi?id=117619

Patch by Horia Iosif Olaru <ol...@adobe.com> on 2013-06-27
Reviewed by Dean Jackson.

* css3/compositing/effect-background-blend-mode-tiled-expected.txt: Added.
* css3/compositing/effect-background-blend-mode-tiled.html: Added.
* platform/mac/css3/compositing/effect-background-blend-mode-tiled-expected.png: Added.

Modified Paths

Added Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (152082 => 152083)


--- trunk/LayoutTests/ChangeLog	2013-06-27 07:28:05 UTC (rev 152082)
+++ trunk/LayoutTests/ChangeLog	2013-06-27 07:49:32 UTC (rev 152083)
@@ -1,3 +1,15 @@
+2013-06-27  Horia Iosif Olaru  <ol...@adobe.com>
+
+        Add an extra test to check if tiled background images get blended as expected. This test passed before this change. The tests for this change are the existing background-blend-mode tests.
+        
+        https://bugs.webkit.org/show_bug.cgi?id=117619
+
+        Reviewed by Dean Jackson.
+
+        * css3/compositing/effect-background-blend-mode-tiled-expected.txt: Added.
+        * css3/compositing/effect-background-blend-mode-tiled.html: Added.
+        * platform/mac/css3/compositing/effect-background-blend-mode-tiled-expected.png: Added.
+
 2013-06-26  Kihong Kwon  <kihong.k...@samsung.com>
 
         Vibration cannot be canceled during pattern vibration.

Added: trunk/LayoutTests/css3/compositing/effect-background-blend-mode-tiled-expected.txt (0 => 152083)


--- trunk/LayoutTests/css3/compositing/effect-background-blend-mode-tiled-expected.txt	                        (rev 0)
+++ trunk/LayoutTests/css3/compositing/effect-background-blend-mode-tiled-expected.txt	2013-06-27 07:49:32 UTC (rev 152083)
@@ -0,0 +1 @@
+

Added: trunk/LayoutTests/css3/compositing/effect-background-blend-mode-tiled.html (0 => 152083)


--- trunk/LayoutTests/css3/compositing/effect-background-blend-mode-tiled.html	                        (rev 0)
+++ trunk/LayoutTests/css3/compositing/effect-background-blend-mode-tiled.html	2013-06-27 07:49:32 UTC (rev 152083)
@@ -0,0 +1,39 @@
+<!DOCTYPE HTML>
+<script>
+  if (window.testRunner)
+    window.testRunner.dumpAsText(true);
+</script>
+<style>
+
+li
+{
+	margin: 5px;
+	width: 130px;
+	height: 130px;
+	background:  url("resources/ducky.png") 0 0 /50% 50%, linear-gradient(to right, #00ffff 0%, rgba(0,0,255,0) 24%, #ff0000 50%, #ffff00 75%, #00ff00 100%) 0 0 /100% 100%;
+	display: block;
+	float: left;
+}
+</style>
+<!-- this file should contain a duck on top of a gradient with every type of blending -->
+<body style="background-color: green">
+<ul>
+<li style="-webkit-background-blend-mode: normal, normal"></li>
+<li style="-webkit-background-blend-mode: multiply, normal"></li>
+<li style="-webkit-background-blend-mode: screen, normal"></li>
+<li style="-webkit-background-blend-mode: overlay, normal; opacity:.9"></li>
+<li style="-webkit-background-blend-mode: darken, normal"></li>
+<li style="-webkit-background-blend-mode: lighten, normal"></li>
+<li style="-webkit-background-blend-mode: color-dodge, normal"></li>
+<li style="-webkit-background-blend-mode: color-burn, normal"></li>
+<li style="-webkit-background-blend-mode: hard-light, normal"></li>
+<li style="-webkit-background-blend-mode: soft-light, normal"></li>
+<li style="-webkit-background-blend-mode: difference, normal"></li>
+<li style="-webkit-background-blend-mode: exclusion, normal"></li>
+<li style="-webkit-background-blend-mode: hue, normal;"></li>
+<li style="-webkit-background-blend-mode: saturation, normal"></li>
+<li style="-webkit-background-blend-mode: color, normal"></li>
+<li style="-webkit-background-blend-mode: luminosity, normal"></li>
+</ul>
+</body>
+

Added: trunk/LayoutTests/platform/mac/css3/compositing/effect-background-blend-mode-tiled-expected.png


(Binary files differ)
Property changes on: trunk/LayoutTests/platform/mac/css3/compositing/effect-background-blend-mode-tiled-expected.png ___________________________________________________________________

Added: svn:mime-type

Modified: trunk/Source/WebCore/ChangeLog (152082 => 152083)


--- trunk/Source/WebCore/ChangeLog	2013-06-27 07:28:05 UTC (rev 152082)
+++ trunk/Source/WebCore/ChangeLog	2013-06-27 07:49:32 UTC (rev 152083)
@@ -1,3 +1,26 @@
+2013-06-27  Horia Iosif Olaru  <ol...@adobe.com>
+
+        Remove ENABLE_CSS_COMPOSITING guards around -webkit-background-blend mode related code. The same flag guards -webkit-blend-mode code, but those guards are not touched by this change.
+
+        https://bugs.webkit.org/show_bug.cgi?id=117619
+
+        Reviewed by Dean Jackson.
+
+        Test: css3/compositing/effect-background-blend-mode-tiled.html
+
+        * WebCore.exp.in:
+        * bindings/generic/RuntimeEnabledFeatures.h:
+        * css/CSSComputedStyleDeclaration.cpp:
+        (WebCore::ComputedStyleExtractor::propertyValue):
+        * css/CSSParser.cpp:
+        (WebCore::CSSParser::parseValue):
+        (WebCore::CSSParser::parseFillProperty):
+        * css/CSSProperty.cpp:
+        (WebCore::CSSProperty::isInheritedProperty):
+        * css/CSSPropertyNames.in:
+        * css/DeprecatedStyleBuilder.cpp:
+        (WebCore::DeprecatedStyleBuilder::DeprecatedStyleBuilder):
+
 2013-06-26  Christophe Dumez  <ch.du...@sisa.samsung.com>
 
         Rename WorkerContext to WorkerGlobalScope

Modified: trunk/Source/WebCore/WebCore.exp.in (152082 => 152083)


--- trunk/Source/WebCore/WebCore.exp.in	2013-06-27 07:28:05 UTC (rev 152082)
+++ trunk/Source/WebCore/WebCore.exp.in	2013-06-27 07:49:32 UTC (rev 152083)
@@ -731,6 +731,7 @@
 __ZN7WebCore21setPlatformStrategiesEPNS_18PlatformStrategiesE
 __ZN7WebCore22HTMLPlugInImageElement24restartSnapshottedPlugInEv
 __ZN7WebCore22HTMLPlugInImageElement29setIsPrimarySnapshottedPlugInEb
+__ZN7WebCore22RuntimeEnabledFeatures23isCSSCompositingEnabledE
 __ZN7WebCore22RuntimeEnabledFeatures22isCSSExclusionsEnabledE
 __ZN7WebCore22RuntimeEnabledFeatures18isCSSShapesEnabledE
 __ZN7WebCore22RuntimeEnabledFeatures40isLangAttributeAwareFormControlUIEnabledE
@@ -2753,10 +2754,6 @@
 __ZN7WebCore22RuntimeEnabledFeatures19isCSSRegionsEnabledE
 #endif
 
-#if ENABLE(CSS_COMPOSITING)
-__ZN7WebCore22RuntimeEnabledFeatures23isCSSCompositingEnabledE
-#endif
-
 #if ENABLE(IFRAME_SEAMLESS)
 __ZN7WebCore22RuntimeEnabledFeatures25areSeamlessIFramesEnabledE
 #endif

Modified: trunk/Source/WebCore/bindings/generic/RuntimeEnabledFeatures.h (152082 => 152083)


--- trunk/Source/WebCore/bindings/generic/RuntimeEnabledFeatures.h	2013-06-27 07:28:05 UTC (rev 152082)
+++ trunk/Source/WebCore/bindings/generic/RuntimeEnabledFeatures.h	2013-06-27 07:49:32 UTC (rev 152083)
@@ -95,13 +95,8 @@
     static bool cssRegionsEnabled() { return false; }
 #endif
 
-#if ENABLE(CSS_COMPOSITING)
     static void setCSSCompositingEnabled(bool isEnabled) { isCSSCompositingEnabled = isEnabled; }
     static bool cssCompositingEnabled() { return isCSSCompositingEnabled; }
-#else
-    static void setCSSCompositingEnabled(bool) { }
-    static bool cssCompositingEnabled() { return false; }
-#endif
 
 #if ENABLE(FONT_LOAD_EVENTS)
     static void setFontLoadEventsEnabled(bool isEnabled) { isFontLoadEventsEnabled = isEnabled; }

Modified: trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp (152082 => 152083)


--- trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp	2013-06-27 07:28:05 UTC (rev 152082)
+++ trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp	2013-06-27 07:49:32 UTC (rev 152083)
@@ -228,13 +228,13 @@
     CSSPropertyWebkitAnimationTimingFunction,
     CSSPropertyWebkitAppearance,
     CSSPropertyWebkitBackfaceVisibility,
+    CSSPropertyWebkitBackgroundBlendMode,
     CSSPropertyWebkitBackgroundClip,
     CSSPropertyWebkitBackgroundComposite,
     CSSPropertyWebkitBackgroundOrigin,
     CSSPropertyWebkitBackgroundSize,
 #if ENABLE(CSS_COMPOSITING)
     CSSPropertyWebkitBlendMode,
-    CSSPropertyWebkitBackgroundBlendMode,
 #endif
     CSSPropertyWebkitBorderFit,
     CSSPropertyWebkitBorderHorizontalSpacing,
@@ -2726,7 +2726,7 @@
 #if ENABLE(CSS_COMPOSITING)
         case CSSPropertyWebkitBlendMode:
             return cssValuePool().createValue(style->blendMode());
-            
+#endif
         case CSSPropertyWebkitBackgroundBlendMode: {
             const FillLayer* layers = style->backgroundLayers();
             if (!layers->next())
@@ -2738,7 +2738,6 @@
 
             return list.release();
         }
-#endif
         case CSSPropertyBackground:
             return getBackgroundShorthandValue();
         case CSSPropertyBorder: {

Modified: trunk/Source/WebCore/css/CSSParser.cpp (152082 => 152083)


--- trunk/Source/WebCore/css/CSSParser.cpp	2013-06-27 07:28:05 UTC (rev 152082)
+++ trunk/Source/WebCore/css/CSSParser.cpp	2013-06-27 07:49:32 UTC (rev 152083)
@@ -2098,6 +2098,7 @@
 #endif
 
     case CSSPropertyBackgroundAttachment:
+    case CSSPropertyWebkitBackgroundBlendMode:
     case CSSPropertyBackgroundClip:
     case CSSPropertyWebkitBackgroundClip:
     case CSSPropertyWebkitBackgroundComposite:
@@ -2123,9 +2124,6 @@
     case CSSPropertyWebkitMaskRepeat:
     case CSSPropertyWebkitMaskRepeatX:
     case CSSPropertyWebkitMaskRepeatY:
-#if ENABLE(CSS_COMPOSITING)
-    case CSSPropertyWebkitBackgroundBlendMode:
-#endif
     {
         RefPtr<CSSValue> val1;
         RefPtr<CSSValue> val2;
@@ -4379,7 +4377,6 @@
                         m_valueList->next();
                     }
                     break;
-#if ENABLE(CSS_COMPOSITING)
                 case CSSPropertyWebkitBackgroundBlendMode:
                     if (cssCompositingEnabled() && (val->id == CSSValueNormal || val->id == CSSValueMultiply
                         || val->id == CSSValueScreen || val->id == CSSValueOverlay || val->id == CSSValueDarken
@@ -4391,7 +4388,6 @@
                         m_valueList->next();
                     }
                     break;
-#endif
                 case CSSPropertyBackgroundRepeat:
                 case CSSPropertyWebkitMaskRepeat:
                     parseFillRepeat(currValue, currValue2);

Modified: trunk/Source/WebCore/css/CSSProperty.cpp (152082 => 152083)


--- trunk/Source/WebCore/css/CSSProperty.cpp	2013-06-27 07:28:05 UTC (rev 152082)
+++ trunk/Source/WebCore/css/CSSProperty.cpp	2013-06-27 07:49:32 UTC (rev 152083)
@@ -516,6 +516,7 @@
     case CSSPropertyWebkitAnimationTimingFunction:
     case CSSPropertyWebkitAppearance:
     case CSSPropertyWebkitBackfaceVisibility:
+    case CSSPropertyWebkitBackgroundBlendMode:
     case CSSPropertyWebkitBackgroundClip:
     case CSSPropertyWebkitBackgroundComposite:
     case CSSPropertyWebkitBackgroundOrigin:
@@ -571,7 +572,6 @@
 #endif
 #if ENABLE(CSS_COMPOSITING)
     case CSSPropertyWebkitBlendMode:
-    case CSSPropertyWebkitBackgroundBlendMode:
 #endif
     case CSSPropertyWebkitAlignContent:
     case CSSPropertyWebkitAlignItems:

Modified: trunk/Source/WebCore/css/CSSPropertyNames.in (152082 => 152083)


--- trunk/Source/WebCore/css/CSSPropertyNames.in	2013-06-27 07:28:05 UTC (rev 152082)
+++ trunk/Source/WebCore/css/CSSPropertyNames.in	2013-06-27 07:49:32 UTC (rev 152083)
@@ -221,6 +221,7 @@
 -webkit-appearance
 -webkit-aspect-ratio
 -webkit-backface-visibility
+-webkit-background-blend-mode
 -webkit-background-clip
 -webkit-background-composite
 -webkit-background-origin
@@ -282,7 +283,6 @@
 #endif
 #if defined(ENABLE_CSS_COMPOSITING) && ENABLE_CSS_COMPOSITING
 -webkit-blend-mode
--webkit-background-blend-mode
 #endif
 #if defined(ENABLE_CSS_FILTERS) && ENABLE_CSS_FILTERS
 -webkit-filter

Modified: trunk/Source/WebCore/css/DeprecatedStyleBuilder.cpp (152082 => 152083)


--- trunk/Source/WebCore/css/DeprecatedStyleBuilder.cpp	2013-06-27 07:28:05 UTC (rev 152082)
+++ trunk/Source/WebCore/css/DeprecatedStyleBuilder.cpp	2013-06-27 07:49:32 UTC (rev 152083)
@@ -2217,13 +2217,13 @@
     setPropertyHandler(CSSPropertyWebkitAppearance, ApplyPropertyDefault<ControlPart, &RenderStyle::appearance, ControlPart, &RenderStyle::setAppearance, ControlPart, &RenderStyle::initialAppearance>::createHandler());
     setPropertyHandler(CSSPropertyWebkitAspectRatio, ApplyPropertyAspectRatio::createHandler());
     setPropertyHandler(CSSPropertyWebkitBackfaceVisibility, ApplyPropertyDefault<EBackfaceVisibility, &RenderStyle::backfaceVisibility, EBackfaceVisibility, &RenderStyle::setBackfaceVisibility, EBackfaceVisibility, &RenderStyle::initialBackfaceVisibility>::createHandler());
+    setPropertyHandler(CSSPropertyWebkitBackgroundBlendMode, ApplyPropertyFillLayer<BlendMode, CSSPropertyWebkitBackgroundBlendMode, BackgroundFillLayer, &RenderStyle::accessBackgroundLayers, &RenderStyle::backgroundLayers, &FillLayer::isBlendModeSet, &FillLayer::blendMode, &FillLayer::setBlendMode, &FillLayer::clearBlendMode, &FillLayer::initialFillBlendMode, &CSSToStyleMap::mapFillBlendMode>::createHandler());
     setPropertyHandler(CSSPropertyWebkitBackgroundClip, CSSPropertyBackgroundClip);
     setPropertyHandler(CSSPropertyWebkitBackgroundComposite, ApplyPropertyFillLayer<CompositeOperator, CSSPropertyWebkitBackgroundComposite, BackgroundFillLayer, &RenderStyle::accessBackgroundLayers, &RenderStyle::backgroundLayers, &FillLayer::isCompositeSet, &FillLayer::composite, &FillLayer::setComposite, &FillLayer::clearComposite, &FillLayer::initialFillComposite, &CSSToStyleMap::mapFillComposite>::createHandler());
     setPropertyHandler(CSSPropertyWebkitBackgroundOrigin, CSSPropertyBackgroundOrigin);
     setPropertyHandler(CSSPropertyWebkitBackgroundSize, CSSPropertyBackgroundSize);
 #if ENABLE(CSS_COMPOSITING)
     setPropertyHandler(CSSPropertyWebkitBlendMode, ApplyPropertyDefault<BlendMode, &RenderStyle::blendMode, BlendMode, &RenderStyle::setBlendMode, BlendMode, &RenderStyle::initialBlendMode>::createHandler());
-    setPropertyHandler(CSSPropertyWebkitBackgroundBlendMode, ApplyPropertyFillLayer<BlendMode, CSSPropertyWebkitBackgroundBlendMode, BackgroundFillLayer, &RenderStyle::accessBackgroundLayers, &RenderStyle::backgroundLayers, &FillLayer::isBlendModeSet, &FillLayer::blendMode, &FillLayer::setBlendMode, &FillLayer::clearBlendMode, &FillLayer::initialFillBlendMode, &CSSToStyleMap::mapFillBlendMode>::createHandler());
 #endif
     setPropertyHandler(CSSPropertyWebkitBorderFit, ApplyPropertyDefault<EBorderFit, &RenderStyle::borderFit, EBorderFit, &RenderStyle::setBorderFit, EBorderFit, &RenderStyle::initialBorderFit>::createHandler());
     setPropertyHandler(CSSPropertyWebkitBorderHorizontalSpacing, ApplyPropertyComputeLength<short, &RenderStyle::horizontalBorderSpacing, &RenderStyle::setHorizontalBorderSpacing, &RenderStyle::initialHorizontalBorderSpacing>::createHandler());
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to