[SCM] WebKit Debian packaging branch, webkit-1.1, updated. upstream/1.1.15.1-1414-gc69ee75
bdakin at apple.com
bdakin at apple.com
Thu Oct 29 20:37:53 UTC 2009
The following commit has been merged in the webkit-1.1 branch:
commit c20916e75bb346991a426595de40c595a4e46eaf
Author: bdakin at apple.com <bdakin at apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Thu Oct 1 23:00:22 2009 +0000
WebCore: Fix for <rdar://problem/7264725> Re-add a vendor prefix to box-
shadow (29927)
-and corresponding-
https://bugs.webkit.org/show_bug.cgi?id=29927
Reviewed by Dan Bernstein.
* css/CSSComputedStyleDeclaration.cpp:
(WebCore::):
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
* css/CSSParser.cpp:
(WebCore::CSSParser::parseValue):
(WebCore::ShadowParseContext::ShadowParseContext):
(WebCore::ShadowParseContext::commitValue):
(WebCore::ShadowParseContext::commitLength):
(WebCore::ShadowParseContext::commitColor):
(WebCore::cssPropertyID):
* css/CSSPropertyNames.in:
* css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::applyProperty):
* page/animation/AnimationBase.cpp:
(WebCore::ensurePropertyMap):
LayoutTests: Updated test results for <rdar://problem/7264725> Re-add a vendor
prefix to box-shadow (29927)
-and corresponding-
https://bugs.webkit.org/show_bug.cgi?id=29927
Reviewed by Dan Bernstein.
* platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/qt/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/win/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* svg/css/getComputedStyle-basic-expected.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@49003 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 2dee074..d6d0a07 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,20 @@
+2009-10-01 Beth Dakin <bdakin at apple.com>
+
+ Reviewed by Dan Bernstein.
+
+ Updated test results for <rdar://problem/7264725> Re-add a vendor
+ prefix to box-shadow (29927)
+ -and corresponding-
+ https://bugs.webkit.org/show_bug.cgi?id=29927
+
+ * platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
+ * platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+ * platform/qt/fast/css/getComputedStyle/computed-style-expected.txt:
+ * platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+ * platform/win/fast/css/getComputedStyle/computed-style-expected.txt:
+ * platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+ * svg/css/getComputedStyle-basic-expected.txt:
+
2009-10-01 Brian Weinstein <bweinstein at apple.com>
Rubber-stamped by Adam Roben.
diff --git a/LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-expected.txt b/LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-expected.txt
index 3a30015..6f126ec 100644
--- a/LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-expected.txt
+++ b/LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-expected.txt
@@ -26,7 +26,6 @@ border-top-right-radius: 0px;
border-top-style: none;
border-top-width: 0px;
bottom: auto;
-box-shadow: none;
caption-side: top;
clear: none;
clip: auto;
@@ -119,6 +118,7 @@ zoom: 1;
-webkit-box-orient: horizontal;
-webkit-box-pack: start;
-webkit-box-reflect: none;
+-webkit-box-shadow: none;
-webkit-box-sizing: content-box;
-webkit-column-break-after: auto;
-webkit-column-break-before: auto;
diff --git a/LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt b/LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
index fdd123e..fecf3ca 100644
--- a/LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
+++ b/LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
@@ -25,7 +25,6 @@ Computed style of an element whose parent's 'display' value is 'none':
border-top-style: solid
border-top-width: 16px
bottom: auto
- box-shadow: none
caption-side: top
clear: none
clip: auto
@@ -118,6 +117,7 @@ Computed style of an element whose parent's 'display' value is 'none':
-webkit-box-orient: horizontal
-webkit-box-pack: start
-webkit-box-reflect: none
+ -webkit-box-shadow: none
-webkit-box-sizing: content-box
-webkit-column-break-after: auto
-webkit-column-break-before: auto
diff --git a/LayoutTests/platform/qt/fast/css/getComputedStyle/computed-style-expected.txt b/LayoutTests/platform/qt/fast/css/getComputedStyle/computed-style-expected.txt
index 998436f..4cdc126 100644
--- a/LayoutTests/platform/qt/fast/css/getComputedStyle/computed-style-expected.txt
+++ b/LayoutTests/platform/qt/fast/css/getComputedStyle/computed-style-expected.txt
@@ -26,7 +26,6 @@ border-top-right-radius: 0px;
border-top-style: none;
border-top-width: 0px;
bottom: auto;
-box-shadow: none;
caption-side: top;
clear: none;
clip: auto;
@@ -118,6 +117,7 @@ zoom: 1;
-webkit-box-orient: horizontal;
-webkit-box-pack: start;
-webkit-box-reflect: none;
+-webkit-box-shadow: none;
-webkit-box-sizing: content-box;
-webkit-column-break-after: auto;
-webkit-column-break-before: auto;
diff --git a/LayoutTests/platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt b/LayoutTests/platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
index 5ff8407..7de2170 100644
--- a/LayoutTests/platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
+++ b/LayoutTests/platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
@@ -25,7 +25,6 @@ Computed style of an element whose parent's 'display' value is 'none':
border-top-style: solid
border-top-width: 13px
bottom: auto
- box-shadow: none
caption-side: top
clear: none
clip: auto
@@ -117,6 +116,7 @@ Computed style of an element whose parent's 'display' value is 'none':
-webkit-box-orient: horizontal
-webkit-box-pack: start
-webkit-box-reflect: none
+ -webkit-box-shadow: none
-webkit-box-sizing: content-box
-webkit-column-break-after: auto
-webkit-column-break-before: auto
diff --git a/LayoutTests/platform/win/fast/css/getComputedStyle/computed-style-expected.txt b/LayoutTests/platform/win/fast/css/getComputedStyle/computed-style-expected.txt
index 214f2f3..c15d146 100644
--- a/LayoutTests/platform/win/fast/css/getComputedStyle/computed-style-expected.txt
+++ b/LayoutTests/platform/win/fast/css/getComputedStyle/computed-style-expected.txt
@@ -26,7 +26,6 @@ border-top-right-radius: 0px;
border-top-style: none;
border-top-width: 0px;
bottom: auto;
-box-shadow: none;
caption-side: top;
clear: none;
clip: auto;
@@ -119,6 +118,7 @@ zoom: 1;
-webkit-box-orient: horizontal;
-webkit-box-pack: start;
-webkit-box-reflect: none;
+-webkit-box-shadow: none;
-webkit-box-sizing: content-box;
-webkit-column-break-after: auto;
-webkit-column-break-before: auto;
diff --git a/LayoutTests/platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt b/LayoutTests/platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
index 6c12939..7eb249d 100644
--- a/LayoutTests/platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
+++ b/LayoutTests/platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
@@ -25,7 +25,6 @@ Computed style of an element whose parent's 'display' value is 'none':
border-top-style: solid
border-top-width: 16px
bottom: auto
- box-shadow: none
caption-side: top
clear: none
clip: auto
@@ -118,6 +117,7 @@ Computed style of an element whose parent's 'display' value is 'none':
-webkit-box-orient: horizontal
-webkit-box-pack: start
-webkit-box-reflect: none
+ -webkit-box-shadow: none
-webkit-box-sizing: content-box
-webkit-column-break-after: auto
-webkit-column-break-before: auto
diff --git a/LayoutTests/svg/css/getComputedStyle-basic-expected.txt b/LayoutTests/svg/css/getComputedStyle-basic-expected.txt
index f80ac23..e3c77e9 100644
--- a/LayoutTests/svg/css/getComputedStyle-basic-expected.txt
+++ b/LayoutTests/svg/css/getComputedStyle-basic-expected.txt
@@ -50,8 +50,6 @@ rect: style.getPropertyValue(border-top-width) : 0px
rect: style.getPropertyCSSValue(border-top-width) : [object CSSPrimitiveValue]
rect: style.getPropertyValue(bottom) : auto
rect: style.getPropertyCSSValue(bottom) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(box-shadow) : none
-rect: style.getPropertyCSSValue(box-shadow) : [object CSSPrimitiveValue]
rect: style.getPropertyValue(caption-side) : top
rect: style.getPropertyCSSValue(caption-side) : [object CSSPrimitiveValue]
rect: style.getPropertyValue(clear) : none
@@ -236,6 +234,8 @@ rect: style.getPropertyValue(-webkit-box-pack) : start
rect: style.getPropertyCSSValue(-webkit-box-pack) : [object CSSPrimitiveValue]
rect: style.getPropertyValue(-webkit-box-reflect) : none
rect: style.getPropertyCSSValue(-webkit-box-reflect) : [object CSSPrimitiveValue]
+rect: style.getPropertyValue(-webkit-box-shadow) : none
+rect: style.getPropertyCSSValue(-webkit-box-shadow) : [object CSSPrimitiveValue]
rect: style.getPropertyValue(-webkit-box-sizing) : content-box
rect: style.getPropertyCSSValue(-webkit-box-sizing) : [object CSSPrimitiveValue]
rect: style.getPropertyValue(-webkit-column-break-after) : auto
@@ -458,8 +458,6 @@ g: style.getPropertyValue(border-top-width) : 0px
g: style.getPropertyCSSValue(border-top-width) : [object CSSPrimitiveValue]
g: style.getPropertyValue(bottom) : auto
g: style.getPropertyCSSValue(bottom) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(box-shadow) : none
-g: style.getPropertyCSSValue(box-shadow) : [object CSSPrimitiveValue]
g: style.getPropertyValue(caption-side) : top
g: style.getPropertyCSSValue(caption-side) : [object CSSPrimitiveValue]
g: style.getPropertyValue(clear) : none
@@ -644,6 +642,8 @@ g: style.getPropertyValue(-webkit-box-pack) : start
g: style.getPropertyCSSValue(-webkit-box-pack) : [object CSSPrimitiveValue]
g: style.getPropertyValue(-webkit-box-reflect) : none
g: style.getPropertyCSSValue(-webkit-box-reflect) : [object CSSPrimitiveValue]
+g: style.getPropertyValue(-webkit-box-shadow) : none
+g: style.getPropertyCSSValue(-webkit-box-shadow) : [object CSSPrimitiveValue]
g: style.getPropertyValue(-webkit-box-sizing) : content-box
g: style.getPropertyCSSValue(-webkit-box-sizing) : [object CSSPrimitiveValue]
g: style.getPropertyValue(-webkit-column-break-after) : auto
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 4222f51..58776e9 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,5 +1,30 @@
2009-10-01 Beth Dakin <bdakin at apple.com>
+ Reviewed by Dan Bernstein.
+
+ Fix for <rdar://problem/7264725> Re-add a vendor prefix to box-
+ shadow (29927)
+ -and corresponding-
+ https://bugs.webkit.org/show_bug.cgi?id=29927
+
+ * css/CSSComputedStyleDeclaration.cpp:
+ (WebCore::):
+ (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
+ * css/CSSParser.cpp:
+ (WebCore::CSSParser::parseValue):
+ (WebCore::ShadowParseContext::ShadowParseContext):
+ (WebCore::ShadowParseContext::commitValue):
+ (WebCore::ShadowParseContext::commitLength):
+ (WebCore::ShadowParseContext::commitColor):
+ (WebCore::cssPropertyID):
+ * css/CSSPropertyNames.in:
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::applyProperty):
+ * page/animation/AnimationBase.cpp:
+ (WebCore::ensurePropertyMap):
+
+2009-10-01 Beth Dakin <bdakin at apple.com>
+
Rubber-stamped by Sam Weinig.
At bad merge at some point in the development of my patch must have
diff --git a/WebCore/css/CSSComputedStyleDeclaration.cpp b/WebCore/css/CSSComputedStyleDeclaration.cpp
index 5dd9798..07a7581 100644
--- a/WebCore/css/CSSComputedStyleDeclaration.cpp
+++ b/WebCore/css/CSSComputedStyleDeclaration.cpp
@@ -77,7 +77,6 @@ static const int computedProperties[] = {
CSSPropertyBorderTopStyle,
CSSPropertyBorderTopWidth,
CSSPropertyBottom,
- CSSPropertyBoxShadow,
CSSPropertyCaptionSide,
CSSPropertyClear,
CSSPropertyClip,
@@ -171,6 +170,7 @@ static const int computedProperties[] = {
CSSPropertyWebkitBoxOrient,
CSSPropertyWebkitBoxPack,
CSSPropertyWebkitBoxReflect,
+ CSSPropertyWebkitBoxShadow,
CSSPropertyWebkitBoxSizing,
CSSPropertyWebkitColumnBreakAfter,
CSSPropertyWebkitColumnBreakBefore,
@@ -767,7 +767,7 @@ PassRefPtr<CSSValue> CSSComputedStyleDeclaration::getPropertyCSSValue(int proper
}
case CSSPropertyWebkitBoxReflect:
return valueForReflection(style->boxReflect());
- case CSSPropertyBoxShadow:
+ case CSSPropertyWebkitBoxShadow:
return valueForShadow(style->boxShadow(), static_cast<CSSPropertyID>(propertyID));
case CSSPropertyCaptionSide:
return CSSPrimitiveValue::create(style->captionSide());
diff --git a/WebCore/css/CSSParser.cpp b/WebCore/css/CSSParser.cpp
index a8b071f..d768bdb 100644
--- a/WebCore/css/CSSParser.cpp
+++ b/WebCore/css/CSSParser.cpp
@@ -1190,7 +1190,7 @@ bool CSSParser::parseValue(int propId, bool important)
valid_primitive = validUnit(value, FLength, m_strict);
break;
case CSSPropertyTextShadow: // CSS2 property, dropped in CSS2.1, back in CSS3, so treat as CSS3
- case CSSPropertyBoxShadow:
+ case CSSPropertyWebkitBoxShadow:
if (id == CSSValueNone)
valid_primitive = true;
else
@@ -3610,7 +3610,7 @@ struct ShadowParseContext {
, allowBlur(false)
, allowSpread(false)
, allowColor(true)
- , allowStyle(prop == CSSPropertyBoxShadow)
+ , allowStyle(prop == CSSPropertyWebkitBoxShadow)
, allowBreak(true)
{
}
@@ -3642,7 +3642,7 @@ struct ShadowParseContext {
allowY = false;
allowBlur = false;
allowSpread = false;
- allowStyle = property == CSSPropertyBoxShadow;
+ allowStyle = property == CSSPropertyWebkitBoxShadow;
}
void commitLength(CSSParserValue* v)
@@ -3661,12 +3661,12 @@ struct ShadowParseContext {
allowY = false;
allowBlur = true;
allowColor = true;
- allowStyle = property == CSSPropertyBoxShadow;
+ allowStyle = property == CSSPropertyWebkitBoxShadow;
allowBreak = true;
} else if (allowBlur) {
blur = val.release();
allowBlur = false;
- allowSpread = property == CSSPropertyBoxShadow;
+ allowSpread = property == CSSPropertyWebkitBoxShadow;
} else if (allowSpread) {
spread = val.release();
allowSpread = false;
@@ -3683,7 +3683,7 @@ struct ShadowParseContext {
} else {
allowBlur = false;
allowSpread = false;
- allowStyle = property == CSSPropertyBoxShadow;
+ allowStyle = property == CSSPropertyWebkitBoxShadow;
}
}
@@ -5176,11 +5176,6 @@ static int cssPropertyID(const UChar* propertyName, unsigned length)
const char* const opacity = "opacity";
name = opacity;
length = strlen(opacity);
- } else if (strcmp(buffer, "-webkit-box-shadow") == 0) {
- // CSS Backgrounds/Borders. -webkit-box-shadow worked in Safari 4 and earlier.
- const char* const boxShadow = "box-shadow";
- name = boxShadow;
- length = strlen(boxShadow);
} else if (strcmp(buffer, "-webkit-background-size") == 0) {
// CSS Backgrounds/Borders. -webkit-background-size worked in Safari 4 and earlier.
const char* const backgroundSize = "background-size";
diff --git a/WebCore/css/CSSPropertyNames.in b/WebCore/css/CSSPropertyNames.in
index f927419..5ef6605 100644
--- a/WebCore/css/CSSPropertyNames.in
+++ b/WebCore/css/CSSPropertyNames.in
@@ -49,7 +49,6 @@ border-top-style
border-top-width
border-width
bottom
-box-shadow
caption-side
clear
clip
@@ -180,6 +179,7 @@ zoom
-webkit-box-orient
-webkit-box-pack
-webkit-box-reflect
+-webkit-box-shadow
-webkit-box-sizing
-webkit-column-break-after
-webkit-column-break-before
diff --git a/WebCore/css/CSSStyleSelector.cpp b/WebCore/css/CSSStyleSelector.cpp
index d4bc0ab..821cc61 100644
--- a/WebCore/css/CSSStyleSelector.cpp
+++ b/WebCore/css/CSSStyleSelector.cpp
@@ -4650,7 +4650,7 @@ void CSSStyleSelector::applyProperty(int id, CSSValue *value)
return;
}
case CSSPropertyTextShadow:
- case CSSPropertyBoxShadow: {
+ case CSSPropertyWebkitBoxShadow: {
if (isInherit) {
if (id == CSSPropertyTextShadow)
return m_style->setTextShadow(m_parentStyle->textShadow() ? new ShadowData(*m_parentStyle->textShadow()) : 0);
diff --git a/WebCore/page/animation/AnimationBase.cpp b/WebCore/page/animation/AnimationBase.cpp
index dec6f4b..ec0e284 100644
--- a/WebCore/page/animation/AnimationBase.cpp
+++ b/WebCore/page/animation/AnimationBase.cpp
@@ -643,7 +643,7 @@ static void ensurePropertyMap()
gPropertyWrappers->append(new PropertyWrapperMaybeInvalidColor(CSSPropertyOutlineColor, &RenderStyle::outlineColor, &RenderStyle::setOutlineColor));
// These are for shadows
- gPropertyWrappers->append(new PropertyWrapperShadow(CSSPropertyBoxShadow, &RenderStyle::boxShadow, &RenderStyle::setBoxShadow));
+ gPropertyWrappers->append(new PropertyWrapperShadow(CSSPropertyWebkitBoxShadow, &RenderStyle::boxShadow, &RenderStyle::setBoxShadow));
gPropertyWrappers->append(new PropertyWrapperShadow(CSSPropertyTextShadow, &RenderStyle::textShadow, &RenderStyle::setTextShadow));
#if ENABLE(SVG)
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list