Legacy CSS vendor prefixes should only work for Dashboard
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 17 Mar 2013 21:39:31 +0000 (21:39 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 17 Mar 2013 21:39:31 +0000 (21:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=111890

Reviewed by Eric Seidel.

Source/WebCore:

Our experience with the Chromium port is that these legacy CSS vendor
prefixes (-apple- and -khtml-) are not needed for web compatibility.
There is reason to believe, however, that they are needed for
compatibility with Mac OS X Dashboard widgets.

This patch makes the code for these legacy CSS vendor prefixes
enabled at runtime and only enables them when running in Dashboard
compatibility mode. This is the first step towards the plan outlined in
https://lists.webkit.org/pipermail/webkit-dev/2013-March/024085.html.

This patch also removes support for ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
from the V8 bindings because V8 is never used with Dashboard.

* bindings/generic/RuntimeEnabledFeatures.cpp:
(WebCore):
* bindings/generic/RuntimeEnabledFeatures.h:
(RuntimeEnabledFeatures):
(WebCore::RuntimeEnabledFeatures::setLegacyCSSVendorPrefixesEnabled):
(WebCore::RuntimeEnabledFeatures::legacyCSSVendorPrefixesEnabled):
* bindings/js/JSCSSStyleDeclarationCustom.cpp:
(WebCore::getCSSPropertyNamePrefix):
(WebCore::cssPropertyIDForJSCSSPropertyName):
* bindings/v8/custom/V8CSSStyleDeclarationCustom.cpp:
(WebCore::cssPropertyInfo):
* css/CSSParser.cpp:
(WebCore::CSSParser::rewriteSpecifiers):

Source/WebKit/mac:

Enable legacy CSS vendor prefixes when we've been asked to turn on
Dashboard compatibility mode.

* WebView/WebView.mm:
(-[WebView _setDashboardBehavior:to:]):

LayoutTests:

* inspector/styles/vendor-prefixes-expected.txt:
    - Update results to show our new behavior now that -apple- and
      -khtml- are not supported.
* platform/mac/TestExpectations:
    - Skip a test that is testing that we support -apple- prefixes.
* platform/mac/fast/css/dashboard-region-parser.html:
    - Update test to use -webkit- rather than -apple-.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@146025 268f45cc-cd09-0410-ab3c-d52691b4dbfc

31 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/css/apple-prefix.html [deleted file]
LayoutTests/inspector/styles/vendor-prefixes-expected.txt
LayoutTests/platform/chromium-linux/fast/css/apple-prefix-expected.png [deleted file]
LayoutTests/platform/chromium-mac-snowleopard/fast/css/apple-prefix-expected.png [deleted file]
LayoutTests/platform/chromium-mac/fast/css/apple-prefix-expected.png [deleted file]
LayoutTests/platform/chromium-win/fast/css/apple-prefix-expected.png [deleted file]
LayoutTests/platform/chromium-win/fast/css/apple-prefix-expected.txt [deleted file]
LayoutTests/platform/chromium/TestExpectations
LayoutTests/platform/chromium/fast/css/apple-prefix-expected.txt [deleted file]
LayoutTests/platform/efl/TestExpectations
LayoutTests/platform/efl/fast/css/apple-prefix-expected.png [deleted file]
LayoutTests/platform/efl/fast/css/apple-prefix-expected.txt [deleted file]
LayoutTests/platform/gtk/TestExpectations
LayoutTests/platform/gtk/fast/css/apple-prefix-expected.png [deleted file]
LayoutTests/platform/gtk/fast/css/apple-prefix-expected.txt [deleted file]
LayoutTests/platform/mac/fast/css/apple-prefix-expected.png [deleted file]
LayoutTests/platform/mac/fast/css/apple-prefix-expected.txt [deleted file]
LayoutTests/platform/mac/fast/css/dashboard-region-parser.html
LayoutTests/platform/qt/TestExpectations
LayoutTests/platform/qt/fast/css/apple-prefix-expected.png [deleted file]
LayoutTests/platform/qt/fast/css/apple-prefix-expected.txt [deleted file]
Source/WebCore/ChangeLog
Source/WebCore/WebCore.exp.in
Source/WebCore/bindings/generic/RuntimeEnabledFeatures.cpp
Source/WebCore/bindings/generic/RuntimeEnabledFeatures.h
Source/WebCore/bindings/js/JSCSSStyleDeclarationCustom.cpp
Source/WebCore/bindings/v8/custom/V8CSSStyleDeclarationCustom.cpp
Source/WebCore/css/CSSParser.cpp
Source/WebKit/mac/ChangeLog
Source/WebKit/mac/WebView/WebView.mm

index 84f51b2..ac92769 100644 (file)
@@ -1,3 +1,18 @@
+2013-03-17  Adam Barth  <abarth@webkit.org>
+
+        Legacy CSS vendor prefixes should only work for Dashboard
+        https://bugs.webkit.org/show_bug.cgi?id=111890
+
+        Reviewed by Eric Seidel.
+
+        * inspector/styles/vendor-prefixes-expected.txt:
+            - Update results to show our new behavior now that -apple- and
+              -khtml- are not supported.
+        * platform/mac/TestExpectations:
+            - Skip a test that is testing that we support -apple- prefixes.
+        * platform/mac/fast/css/dashboard-region-parser.html:
+            - Update test to use -webkit- rather than -apple-.
+
 2013-03-17  Simon Fraser  <simon.fraser@apple.com>
 
         fast/frames/sandboxed-iframe-navigation-allowed.html sometimes
diff --git a/LayoutTests/fast/css/apple-prefix.html b/LayoutTests/fast/css/apple-prefix.html
deleted file mode 100644 (file)
index de6aaf3..0000000
+++ /dev/null
@@ -1,52 +0,0 @@
-<html>
-<head>
-<script>
-function test()
-{
-    var content = document.getElementById('content');
-    content.style.appleLineClamp = "10%";
-}
-</script>
-<style>
-.content {
-    overflow: hidden;
-    display: -webkit-box;
-    -webkit-box-orient: vertical;
-}
-</style>
-</head>
-<body onload="test()">
-<p>This tests use of the -apple-line-clamp function in a web page, from JavaScript, and using the "apple" prefix, not "khtml".</p>
-<p>If it works properly, the article below should show only the first few lines. Otherwise, it might show many lines.</p>
-<hr> 
-<div class="content" id="content">    
-<div>
-<p>It was just last night, I think, that I was IM&#39;ing with a friend and I said, &quot;Next week Simon starts going to
-school for 3 days extended (until 3pm) and Sophia starts swim camp! I won&#39;t know what to do with my free time! But I&#39;m
-willing to find out.&quot;</p><p>You know the old saying: &quot;Man plans, God laughs.&quot;</p>
-<p>Today I got a call from the preschool telling me that Sophia had broken her arm.</p>
-<p>It happened on the playground. I imagined she had fallen off the monkey bars or something, but no: she had been sitting
-at the Art Table, and as she was getting off the bench she slipped and fell on her arm. She was being very brave when I came
-to get herÑno tears, stoically holding her arm covered in ice packs. She <i>had</i> cried, she told me. All of the teachers
-were quite amazed at how brave she was being. I took her to the doctor and from there to the pediatric orthopedic doctor.
-She started to lose her composure, though, after she had finally gotten the X-rays (&quot;Like Curious George got when he
-went to the hospital&quot;) and we had to wait in the exam room.</p>
-<p>&quot;I want to go home now,&quot; she said. &quot;I wish I&#39;d never hurt my arm.&quot;</p>
-<p>The ortho tech came in to give her a splint (she&#39;ll have a splint for a couple of days and then get a cast on Thursday),
-and can I tell you how worrisome it is to have an ortho tech who is a <b>dead ringer</b> for Ian McShane as Al Swearengen
-wrapping up your daughter in gauze? (I&#39;m totally serious. He didn&#39;t have grease in his hair, and he didn&#39;t have the
-little beard under his lip, but other than that: yup.)</p>
-<p>So now she&#39;s on the couch, watching &quot;Dragontales&quot; with her arm propped up on a pillow (to keep it above the heart).
-She&#39;s eating a couple of cut-up apples and cheese sticks (&quot;because that&#39;s all I can eat now, Mommy&quot;).</p>
-<p>She&#39;s upset because she&#39;s not going to be able to do all the swimming she was planning. I&#39;m upset because my
-little girl got hurt. I assume that after a day or so we&#39;ll adjust to the new regimen. Of course, she can&#39;t go to her last
-day of preschool tomorrow, because they&#39;re having &quot;Water Days,&quot; and she can&#39;t get wet.</p>
-<p>The weather&#39;s finally gotten extremely hot around here, and she can&#39;t get her splint (and in a few days, her cast) wet. <i>Neat.</i></p>
-<p>I&#39;ve never broken anything (knock on nearest wood sprite)Ñdoes it affect your outlook on life?
-She has an adventuresome spirit and I hope she&#39;s just as energetic after this little incident as she was before.
-At the moment she&#39;s even lost her sense of humor.</p>
-</div>
-<hr>
-</div>
-</body>
-</html>
index dd1ee21..baad0c0 100644 (file)
@@ -4,9 +4,8 @@ Text
 [expanded] 
 display: block;
     div - block user agent stylesheet
-opacity: 0.75;
-    #inspected - 0.75 vendor-prefixes.html:11
-    /-- overloaded --/ #inspected - 1 vendor-prefixes.html:4
+opacity: 1;
+    #inspected - 1 vendor-prefixes.html:4
 
 [expanded] 
 element.style  { ()
@@ -14,13 +13,13 @@ element.style  { ()
 ======== Matched CSS Rules ========
 [expanded] 
 #inspected  { (vendor-prefixes.html:11)
--apple-opacity: 0.75;
+/-- overloaded --/ -apple-opacity: 0.75;
 
 [expanded] 
 #inspected  { (vendor-prefixes.html:4)
 /-- overloaded --/ -apple-opacity: 0.35;
 /-- overloaded --/ -webkit-opacity: 0.5;
-/-- overloaded --/ opacity: 1;
+opacity: 1;
 /-- overloaded --/ -khtml-opacity: 0.25;
 
 [expanded] 
diff --git a/LayoutTests/platform/chromium-linux/fast/css/apple-prefix-expected.png b/LayoutTests/platform/chromium-linux/fast/css/apple-prefix-expected.png
deleted file mode 100644 (file)
index a4a34fc..0000000
Binary files a/LayoutTests/platform/chromium-linux/fast/css/apple-prefix-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/chromium-mac-snowleopard/fast/css/apple-prefix-expected.png b/LayoutTests/platform/chromium-mac-snowleopard/fast/css/apple-prefix-expected.png
deleted file mode 100644 (file)
index 1cbc037..0000000
Binary files a/LayoutTests/platform/chromium-mac-snowleopard/fast/css/apple-prefix-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/chromium-mac/fast/css/apple-prefix-expected.png b/LayoutTests/platform/chromium-mac/fast/css/apple-prefix-expected.png
deleted file mode 100644 (file)
index efabda9..0000000
Binary files a/LayoutTests/platform/chromium-mac/fast/css/apple-prefix-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/chromium-win/fast/css/apple-prefix-expected.png b/LayoutTests/platform/chromium-win/fast/css/apple-prefix-expected.png
deleted file mode 100644 (file)
index 694b5d7..0000000
Binary files a/LayoutTests/platform/chromium-win/fast/css/apple-prefix-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/chromium-win/fast/css/apple-prefix-expected.txt b/LayoutTests/platform/chromium-win/fast/css/apple-prefix-expected.txt
deleted file mode 100644 (file)
index 7f1dfb3..0000000
+++ /dev/null
@@ -1,72 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x600
-  RenderBlock {HTML} at (0,0) size 800x600
-    RenderBody {BODY} at (8,8) size 784x584
-      RenderBlock {P} at (0,0) size 784x20
-        RenderText {#text} at (0,0) size 720x19
-          text run at (0,0) width 720: "This tests use of the -apple-line-clamp function in a web page, from JavaScript, and using the \"apple\" prefix, not \"khtml\"."
-      RenderBlock {P} at (0,36) size 784x20
-        RenderText {#text} at (0,0) size 663x19
-          text run at (0,0) width 663: "If it works properly, the article below should show only the first few lines. Otherwise, it might show many lines."
-      RenderBlock {HR} at (0,72) size 784x2 [border: (1px inset #000000)]
-layer at (8,90) size 784x57 scrollHeight 604
-  RenderDeprecatedFlexibleBox {DIV} at (0,82) size 784x57
-    RenderBlock {DIV} at (0,0) size 784x39
-      RenderBlock {P} at (0,0) size 784x40
-        RenderText {#text} at (0,0) size 737x39
-          text run at (0,0) width 733: "It was just last night, I think, that I was IM'ing with a friend and I said, \"Next week Simon starts going to school for 3 days"
-          text run at (0,20) width 737: "extended (until 3pm) and Sophia starts swim camp! I won't know what to do with my free time! But I'm willing to find out.\""
-      RenderBlock {P} at (0,56) size 784x20
-        RenderText {#text} at (0,0) size 310x19
-          text run at (0,0) width 310: "You know the old saying: \"Man plans, God laughs.\""
-      RenderBlock {P} at (0,92) size 784x20
-        RenderText {#text} at (0,0) size 482x19
-          text run at (0,0) width 482: "Today I got a call from the preschool telling me that Sophia had broken her arm."
-      RenderBlock {P} at (0,128) size 784x120
-        RenderText {#text} at (0,0) size 777x59
-          text run at (0,0) width 775: "It happened on the playground. I imagined she had fallen off the monkey bars or something, but no: she had been sitting at the Art"
-          text run at (0,20) width 777: "Table, and as she was getting off the bench she slipped and fell on her arm. She was being very brave when I came to get her\x{D1}no"
-          text run at (0,40) width 345: "tears, stoically holding her arm covered in ice packs. She "
-        RenderInline {I} at (0,0) size 24x19
-          RenderText {#text} at (345,40) size 24x19
-            text run at (345,40) width 24: "had"
-        RenderText {#text} at (369,40) size 770x79
-          text run at (369,40) width 390: " cried, she told me. All of the teachers were quite amazed at how"
-          text run at (0,60) width 714: "brave she was being. I took her to the doctor and from there to the pediatric orthopedic doctor. She started to lose her"
-          text run at (0,80) width 770: "composure, though, after she had finally gotten the X-rays (\"Like Curious George got when he went to the hospital\") and we had"
-          text run at (0,100) width 153: "to wait in the exam room."
-      RenderBlock {P} at (0,264) size 784x20
-        RenderText {#text} at (0,0) size 397x19
-          text run at (0,0) width 397: "\"I want to go home now,\" she said. \"I wish I'd never hurt my arm.\""
-      RenderBlock {P} at (0,300) size 784x80
-        RenderText {#text} at (0,0) size 776x39
-          text run at (0,0) width 776: "The ortho tech came in to give her a splint (she'll have a splint for a couple of days and then get a cast on Thursday), and can I tell"
-          text run at (0,20) width 333: "you how worrisome it is to have an ortho tech who is a "
-        RenderInline {B} at (0,0) size 76x19
-          RenderText {#text} at (333,20) size 76x19
-            text run at (333,20) width 76: "dead ringer"
-        RenderText {#text} at (409,20) size 767x59
-          text run at (409,20) width 329: " for Ian McShane as Al Swearengen wrapping up your"
-          text run at (0,40) width 767: "daughter in gauze? (I'm totally serious. He didn't have grease in his hair, and he didn't have the little beard under his lip, but other"
-          text run at (0,60) width 89: "than that: yup.)"
-      RenderBlock {P} at (0,396) size 784x40
-        RenderText {#text} at (0,0) size 779x39
-          text run at (0,0) width 779: "So now she's on the couch, watching \"Dragontales\" with her arm propped up on a pillow (to keep it above the heart). She's eating"
-          text run at (0,20) width 536: "a couple of cut-up apples and cheese sticks (\"because that's all I can eat now, Mommy\")."
-      RenderBlock {P} at (0,452) size 784x60
-        RenderText {#text} at (0,0) size 771x59
-          text run at (0,0) width 771: "She's upset because she's not going to be able to do all the swimming she was planning. I'm upset because my little girl got hurt. I"
-          text run at (0,20) width 745: "assume that after a day or so we'll adjust to the new regimen. Of course, she can't go to her last day of preschool tomorrow,"
-          text run at (0,40) width 360: "because they're having \"Water Days,\" and she can't get wet."
-      RenderBlock {P} at (0,528) size 784x20
-        RenderText {#text} at (0,0) size 698x19
-          text run at (0,0) width 698: "The weather's finally gotten extremely hot around here, and she can't get her splint (and in a few days, her cast) wet. "
-        RenderInline {I} at (0,0) size 35x19
-          RenderText {#text} at (698,0) size 35x19
-            text run at (698,0) width 35: "Neat."
-      RenderBlock {P} at (0,564) size 784x40
-        RenderText {#text} at (0,0) size 767x39
-          text run at (0,0) width 767: "I've never broken anything (knock on nearest wood sprite)\x{D1}does it affect your outlook on life? She has an adventuresome spirit"
-          text run at (0,20) width 752: "and I hope she's just as energetic after this little incident as she was before. At the moment she's even lost her sense of humor."
-    RenderBlock {HR} at (0,47) size 784x2 [border: (1px inset #000000)]
index aa37c42..0fcd5c4 100644 (file)
@@ -253,7 +253,6 @@ webkit.org/b/98212 fast/dom/Geolocation/coordinates-interface-attributes.html [
 
 # These tests are for -apple- and -khtml- prefixes, which we don't support
 inspector/styles/vendor-prefixes.html [ WontFix ]
-fast/css/apple-prefix.html [ WontFix ]
 
 # This test is specific to JSC; the V8 equivalent is
 # platform/chromium/fast/storage/serialized-script-value.html
diff --git a/LayoutTests/platform/chromium/fast/css/apple-prefix-expected.txt b/LayoutTests/platform/chromium/fast/css/apple-prefix-expected.txt
deleted file mode 100644 (file)
index 2c6940e..0000000
+++ /dev/null
@@ -1,73 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x600
-  RenderBlock {HTML} at (0,0) size 800x600
-    RenderBody {BODY} at (8,8) size 784x584
-      RenderBlock {P} at (0,0) size 784x18
-        RenderText {#text} at (0,0) size 764x18
-          text run at (0,0) width 764: "This tests use of the -apple-line-clamp function in a web page, from JavaScript, and using the \"apple\" prefix, not \"khtml\"."
-      RenderBlock {P} at (0,34) size 784x18
-        RenderText {#text} at (0,0) size 705x18
-          text run at (0,0) width 705: "If it works properly, the article below should show only the first few lines. Otherwise, it might show many lines."
-      RenderBlock {HR} at (0,68) size 784x2 [border: (1px inset #000000)]
-layer at (8,86) size 784x54 scrollHeight 576
-  RenderDeprecatedFlexibleBox {DIV} at (0,78) size 784x54
-    RenderBlock {DIV} at (0,0) size 784x36
-      RenderBlock {P} at (0,0) size 784x36
-        RenderText {#text} at (0,0) size 781x36
-          text run at (0,0) width 771: "It was just last night, I think, that I was IM'ing with a friend and I said, \"Next week Simon starts going to school for 3 days"
-          text run at (0,18) width 781: "extended (until 3pm) and Sophia starts swim camp! I won't know what to do with my free time! But I'm willing to find out.\""
-      RenderBlock {P} at (0,52) size 784x18
-        RenderText {#text} at (0,0) size 333x18
-          text run at (0,0) width 333: "You know the old saying: \"Man plans, God laughs.\""
-      RenderBlock {P} at (0,86) size 784x18
-        RenderText {#text} at (0,0) size 505x18
-          text run at (0,0) width 505: "Today I got a call from the preschool telling me that Sophia had broken her arm."
-      RenderBlock {P} at (0,120) size 784x108
-        RenderText {#text} at (0,0) size 773x54
-          text run at (0,0) width 770: "It happened on the playground. I imagined she had fallen off the monkey bars or something, but no: she had been sitting at"
-          text run at (0,18) width 773: "the Art Table, and as she was getting off the bench she slipped and fell on her arm. She was being very brave when I came"
-          text run at (0,36) width 452: "to get her\x{D1}no tears, stoically holding her arm covered in ice packs. She "
-        RenderInline {I} at (0,0) size 24x18
-          RenderText {#text} at (452,36) size 24x18
-            text run at (452,36) width 24: "had"
-        RenderText {#text} at (476,36) size 784x72
-          text run at (476,36) width 308: " cried, she told me. All of the teachers were quite"
-          text run at (0,54) width 780: "amazed at how brave she was being. I took her to the doctor and from there to the pediatric orthopedic doctor. She started to"
-          text run at (0,72) width 725: "lose her composure, though, after she had finally gotten the X-rays (\"Like Curious George got when he went to the"
-          text run at (0,90) width 303: "hospital\") and we had to wait in the exam room."
-      RenderBlock {P} at (0,244) size 784x18
-        RenderText {#text} at (0,0) size 424x18
-          text run at (0,0) width 424: "\"I want to go home now,\" she said. \"I wish I'd never hurt my arm.\""
-      RenderBlock {P} at (0,278) size 784x72
-        RenderText {#text} at (0,0) size 784x36
-          text run at (0,0) width 784: "The ortho tech came in to give her a splint (she'll have a splint for a couple of days and then get a cast on Thursday), and can"
-          text run at (0,18) width 382: "I tell you how worrisome it is to have an ortho tech who is a "
-        RenderInline {B} at (0,0) size 79x18
-          RenderText {#text} at (382,18) size 79x18
-            text run at (382,18) width 79: "dead ringer"
-        RenderText {#text} at (461,18) size 784x54
-          text run at (461,18) width 313: " for Ian McShane as Al Swearengen wrapping up"
-          text run at (0,36) width 784: "your daughter in gauze? (I'm totally serious. He didn't have grease in his hair, and he didn't have the little beard under his lip,"
-          text run at (0,54) width 155: "but other than that: yup.)"
-      RenderBlock {P} at (0,366) size 784x36
-        RenderText {#text} at (0,0) size 776x36
-          text run at (0,0) width 776: "So now she's on the couch, watching \"Dragontales\" with her arm propped up on a pillow (to keep it above the heart). She's"
-          text run at (0,18) width 600: "eating a couple of cut-up apples and cheese sticks (\"because that's all I can eat now, Mommy\")."
-      RenderBlock {P} at (0,418) size 784x54
-        RenderText {#text} at (0,0) size 771x54
-          text run at (0,0) width 771: "She's upset because she's not going to be able to do all the swimming she was planning. I'm upset because my little girl got"
-          text run at (0,18) width 746: "hurt. I assume that after a day or so we'll adjust to the new regimen. Of course, she can't go to her last day of preschool"
-          text run at (0,36) width 449: "tomorrow, because they're having \"Water Days,\" and she can't get wet."
-      RenderBlock {P} at (0,488) size 784x18
-        RenderText {#text} at (0,0) size 734x18
-          text run at (0,0) width 734: "The weather's finally gotten extremely hot around here, and she can't get her splint (and in a few days, her cast) wet. "
-        RenderInline {I} at (0,0) size 34x18
-          RenderText {#text} at (734,0) size 34x18
-            text run at (734,0) width 34: "Neat."
-      RenderBlock {P} at (0,522) size 784x54
-        RenderText {#text} at (0,0) size 773x54
-          text run at (0,0) width 767: "I've never broken anything (knock on nearest wood sprite)\x{D1}does it affect your outlook on life? She has an adventuresome"
-          text run at (0,18) width 773: "spirit and I hope she's just as energetic after this little incident as she was before. At the moment she's even lost her sense of"
-          text run at (0,36) width 45: "humor."
-    RenderBlock {HR} at (0,44) size 784x2 [border: (1px inset #000000)]
index 13b66f8..2905a17 100644 (file)
@@ -1275,9 +1275,6 @@ webkit.org/b/73544 http/tests/notifications [ Skip ]
 # The test is too slow on Debug bot and randomly fails
 webkit.org/b/61932 [ Debug ] jquery/manipulation.html [ Failure Pass Timeout ]
 
-# These test -apple- prefixed CSS properties, which we don't support.
-fast/css/apple-prefix.html [ WontFix ]
-
 webkit.org/b/75747 fast/css/color-correction-on-background-image.html [ Missing Failure ]
 webkit.org/b/75747 fast/css/color-correction-on-backgrounds.html [ Missing Failure ]
 webkit.org/b/75747 fast/css/color-correction-on-box-shadow.html [ Missing Failure ]
diff --git a/LayoutTests/platform/efl/fast/css/apple-prefix-expected.png b/LayoutTests/platform/efl/fast/css/apple-prefix-expected.png
deleted file mode 100644 (file)
index 783b4b1..0000000
Binary files a/LayoutTests/platform/efl/fast/css/apple-prefix-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/efl/fast/css/apple-prefix-expected.txt b/LayoutTests/platform/efl/fast/css/apple-prefix-expected.txt
deleted file mode 100644 (file)
index 05a3a99..0000000
+++ /dev/null
@@ -1,73 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x600
-  RenderBlock {HTML} at (0,0) size 800x600
-    RenderBody {BODY} at (8,8) size 784x584
-      RenderBlock {P} at (0,0) size 784x18
-        RenderText {#text} at (0,0) size 764x19
-          text run at (0,0) width 764: "This tests use of the -apple-line-clamp function in a web page, from JavaScript, and using the \"apple\" prefix, not \"khtml\"."
-      RenderBlock {P} at (0,34) size 784x18
-        RenderText {#text} at (0,0) size 705x19
-          text run at (0,0) width 705: "If it works properly, the article below should show only the first few lines. Otherwise, it might show many lines."
-      RenderBlock {HR} at (0,68) size 784x2 [border: (1px inset #000000)]
-layer at (8,86) size 784x55 scrollHeight 577
-  RenderDeprecatedFlexibleBox {DIV} at (0,78) size 784x55
-    RenderBlock {DIV} at (0,0) size 784x37
-      RenderBlock {P} at (0,0) size 784x36
-        RenderText {#text} at (0,0) size 781x37
-          text run at (0,0) width 771: "It was just last night, I think, that I was IM'ing with a friend and I said, \"Next week Simon starts going to school for 3 days"
-          text run at (0,18) width 781: "extended (until 3pm) and Sophia starts swim camp! I won't know what to do with my free time! But I'm willing to find out.\""
-      RenderBlock {P} at (0,52) size 784x18
-        RenderText {#text} at (0,0) size 333x19
-          text run at (0,0) width 333: "You know the old saying: \"Man plans, God laughs.\""
-      RenderBlock {P} at (0,86) size 784x18
-        RenderText {#text} at (0,0) size 505x19
-          text run at (0,0) width 505: "Today I got a call from the preschool telling me that Sophia had broken her arm."
-      RenderBlock {P} at (0,120) size 784x108
-        RenderText {#text} at (0,0) size 773x55
-          text run at (0,0) width 770: "It happened on the playground. I imagined she had fallen off the monkey bars or something, but no: she had been sitting at"
-          text run at (0,18) width 773: "the Art Table, and as she was getting off the bench she slipped and fell on her arm. She was being very brave when I came"
-          text run at (0,36) width 452: "to get her\x{D1}no tears, stoically holding her arm covered in ice packs. She "
-        RenderInline {I} at (0,0) size 24x19
-          RenderText {#text} at (452,36) size 24x19
-            text run at (452,36) width 24: "had"
-        RenderText {#text} at (476,36) size 784x73
-          text run at (476,36) width 308: " cried, she told me. All of the teachers were quite"
-          text run at (0,54) width 780: "amazed at how brave she was being. I took her to the doctor and from there to the pediatric orthopedic doctor. She started to"
-          text run at (0,72) width 725: "lose her composure, though, after she had finally gotten the X-rays (\"Like Curious George got when he went to the"
-          text run at (0,90) width 303: "hospital\") and we had to wait in the exam room."
-      RenderBlock {P} at (0,244) size 784x18
-        RenderText {#text} at (0,0) size 424x19
-          text run at (0,0) width 424: "\"I want to go home now,\" she said. \"I wish I'd never hurt my arm.\""
-      RenderBlock {P} at (0,278) size 784x72
-        RenderText {#text} at (0,0) size 784x37
-          text run at (0,0) width 784: "The ortho tech came in to give her a splint (she'll have a splint for a couple of days and then get a cast on Thursday), and can"
-          text run at (0,18) width 382: "I tell you how worrisome it is to have an ortho tech who is a "
-        RenderInline {B} at (0,0) size 79x19
-          RenderText {#text} at (382,18) size 79x19
-            text run at (382,18) width 79: "dead ringer"
-        RenderText {#text} at (461,18) size 784x55
-          text run at (461,18) width 313: " for Ian McShane as Al Swearengen wrapping up"
-          text run at (0,36) width 784: "your daughter in gauze? (I'm totally serious. He didn't have grease in his hair, and he didn't have the little beard under his lip,"
-          text run at (0,54) width 155: "but other than that: yup.)"
-      RenderBlock {P} at (0,366) size 784x36
-        RenderText {#text} at (0,0) size 776x37
-          text run at (0,0) width 776: "So now she's on the couch, watching \"Dragontales\" with her arm propped up on a pillow (to keep it above the heart). She's"
-          text run at (0,18) width 600: "eating a couple of cut-up apples and cheese sticks (\"because that's all I can eat now, Mommy\")."
-      RenderBlock {P} at (0,418) size 784x54
-        RenderText {#text} at (0,0) size 771x55
-          text run at (0,0) width 771: "She's upset because she's not going to be able to do all the swimming she was planning. I'm upset because my little girl got"
-          text run at (0,18) width 746: "hurt. I assume that after a day or so we'll adjust to the new regimen. Of course, she can't go to her last day of preschool"
-          text run at (0,36) width 449: "tomorrow, because they're having \"Water Days,\" and she can't get wet."
-      RenderBlock {P} at (0,488) size 784x18
-        RenderText {#text} at (0,0) size 734x19
-          text run at (0,0) width 734: "The weather's finally gotten extremely hot around here, and she can't get her splint (and in a few days, her cast) wet. "
-        RenderInline {I} at (0,0) size 34x19
-          RenderText {#text} at (734,0) size 34x19
-            text run at (734,0) width 34: "Neat."
-      RenderBlock {P} at (0,522) size 784x54
-        RenderText {#text} at (0,0) size 773x55
-          text run at (0,0) width 767: "I've never broken anything (knock on nearest wood sprite)\x{D1}does it affect your outlook on life? She has an adventuresome"
-          text run at (0,18) width 773: "spirit and I hope she's just as energetic after this little incident as she was before. At the moment she's even lost her sense of"
-          text run at (0,36) width 45: "humor."
-    RenderBlock {HR} at (0,45) size 784x2 [border: (1px inset #000000)]
index 7339986..c3359cd 100644 (file)
@@ -80,7 +80,6 @@ webkit.org/b/71851 http/tests/security/contentTypeOptions [ Skip ]
 
 # These test -apple- and -khtml- prefixed CSS properties, which we don't support.
 inspector/styles/vendor-prefixes.html [ WontFix Failure ]
-fast/css/apple-prefix.html [ WontFix Failure ]
 
 # This port doesn't support detecting slow unload handlers.
 fast/dom/Window/slow-unload-handler.html [ WontFix Timeout Failure ]
diff --git a/LayoutTests/platform/gtk/fast/css/apple-prefix-expected.png b/LayoutTests/platform/gtk/fast/css/apple-prefix-expected.png
deleted file mode 100644 (file)
index e533123..0000000
Binary files a/LayoutTests/platform/gtk/fast/css/apple-prefix-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/gtk/fast/css/apple-prefix-expected.txt b/LayoutTests/platform/gtk/fast/css/apple-prefix-expected.txt
deleted file mode 100644 (file)
index 1b5f1ab..0000000
+++ /dev/null
@@ -1,73 +0,0 @@
-layer at (0,0) size 785x706
-  RenderView at (0,0) size 785x600
-layer at (0,0) size 785x706
-  RenderBlock {HTML} at (0,0) size 785x706
-    RenderBody {BODY} at (8,8) size 769x690
-      RenderBlock {P} at (0,0) size 769x18
-        RenderText {#text} at (0,0) size 764x19
-          text run at (0,0) width 764: "This tests use of the -apple-line-clamp function in a web page, from JavaScript, and using the \"apple\" prefix, not \"khtml\"."
-      RenderBlock {P} at (0,34) size 769x18
-        RenderText {#text} at (0,0) size 705x19
-          text run at (0,0) width 705: "If it works properly, the article below should show only the first few lines. Otherwise, it might show many lines."
-      RenderBlock {HR} at (0,68) size 769x2 [border: (1px inset #000000)]
-      RenderDeprecatedFlexibleBox {DIV} at (0,78) size 769x612
-        RenderBlock {DIV} at (0,0) size 769x594
-          RenderBlock {P} at (0,0) size 769x54
-            RenderText {#text} at (0,0) size 750x55
-              text run at (0,0) width 738: "It was just last night, I think, that I was IM'ing with a friend and I said, \"Next week Simon starts going to school for 3"
-              text run at (0,18) width 750: "days extended (until 3pm) and Sophia starts swim camp! I won't know what to do with my free time! But I'm willing to"
-              text run at (0,36) width 60: "find out.\""
-          RenderBlock {P} at (0,70) size 769x18
-            RenderText {#text} at (0,0) size 333x19
-              text run at (0,0) width 333: "You know the old saying: \"Man plans, God laughs.\""
-          RenderBlock {P} at (0,104) size 769x18
-            RenderText {#text} at (0,0) size 505x19
-              text run at (0,0) width 505: "Today I got a call from the preschool telling me that Sophia had broken her arm."
-          RenderBlock {P} at (0,138) size 769x108
-            RenderText {#text} at (0,0) size 755x55
-              text run at (0,0) width 755: "It happened on the playground. I imagined she had fallen off the monkey bars or something, but no: she had been sitting"
-              text run at (0,18) width 751: "at the Art Table, and as she was getting off the bench she slipped and fell on her arm. She was being very brave when I"
-              text run at (0,36) width 489: "came to get her\x{D1}no tears, stoically holding her arm covered in ice packs. She "
-            RenderInline {I} at (0,0) size 24x19
-              RenderText {#text} at (489,36) size 24x19
-                text run at (489,36) width 24: "had"
-            RenderText {#text} at (513,36) size 761x73
-              text run at (513,36) width 238: " cried, she told me. All of the teachers"
-              text run at (0,54) width 761: "were quite amazed at how brave she was being. I took her to the doctor and from there to the pediatric orthopedic doctor."
-              text run at (0,72) width 740: "She started to lose her composure, though, after she had finally gotten the X-rays (\"Like Curious George got when he"
-              text run at (0,90) width 377: "went to the hospital\") and we had to wait in the exam room."
-          RenderBlock {P} at (0,262) size 769x18
-            RenderText {#text} at (0,0) size 424x19
-              text run at (0,0) width 424: "\"I want to go home now,\" she said. \"I wish I'd never hurt my arm.\""
-          RenderBlock {P} at (0,296) size 769x72
-            RenderText {#text} at (0,0) size 758x37
-              text run at (0,0) width 758: "The ortho tech came in to give her a splint (she'll have a splint for a couple of days and then get a cast on Thursday), and"
-              text run at (0,18) width 408: "can I tell you how worrisome it is to have an ortho tech who is a "
-            RenderInline {B} at (0,0) size 79x19
-              RenderText {#text} at (408,18) size 79x19
-                text run at (408,18) width 79: "dead ringer"
-            RenderText {#text} at (487,18) size 743x55
-              text run at (487,18) width 229: " for Ian McShane as Al Swearengen"
-              text run at (0,36) width 743: "wrapping up your daughter in gauze? (I'm totally serious. He didn't have grease in his hair, and he didn't have the little"
-              text run at (0,54) width 280: "beard under his lip, but other than that: yup.)"
-          RenderBlock {P} at (0,384) size 769x36
-            RenderText {#text} at (0,0) size 739x37
-              text run at (0,0) width 739: "So now she's on the couch, watching \"Dragontales\" with her arm propped up on a pillow (to keep it above the heart)."
-              text run at (0,18) width 637: "She's eating a couple of cut-up apples and cheese sticks (\"because that's all I can eat now, Mommy\")."
-          RenderBlock {P} at (0,436) size 769x54
-            RenderText {#text} at (0,0) size 747x55
-              text run at (0,0) width 747: "She's upset because she's not going to be able to do all the swimming she was planning. I'm upset because my little girl"
-              text run at (0,18) width 705: "got hurt. I assume that after a day or so we'll adjust to the new regimen. Of course, she can't go to her last day of"
-              text run at (0,36) width 514: "preschool tomorrow, because they're having \"Water Days,\" and she can't get wet."
-          RenderBlock {P} at (0,506) size 769x18
-            RenderText {#text} at (0,0) size 734x19
-              text run at (0,0) width 734: "The weather's finally gotten extremely hot around here, and she can't get her splint (and in a few days, her cast) wet. "
-            RenderInline {I} at (0,0) size 34x19
-              RenderText {#text} at (734,0) size 34x19
-                text run at (734,0) width 34: "Neat."
-          RenderBlock {P} at (0,540) size 769x54
-            RenderText {#text} at (0,0) size 767x55
-              text run at (0,0) width 767: "I've never broken anything (knock on nearest wood sprite)\x{D1}does it affect your outlook on life? She has an adventuresome"
-              text run at (0,18) width 756: "spirit and I hope she's just as energetic after this little incident as she was before. At the moment she's even lost her sense"
-              text run at (0,36) width 62: "of humor."
-        RenderBlock {HR} at (0,602) size 769x2 [border: (1px inset #000000)]
diff --git a/LayoutTests/platform/mac/fast/css/apple-prefix-expected.png b/LayoutTests/platform/mac/fast/css/apple-prefix-expected.png
deleted file mode 100644 (file)
index 85ba8eb..0000000
Binary files a/LayoutTests/platform/mac/fast/css/apple-prefix-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/mac/fast/css/apple-prefix-expected.txt b/LayoutTests/platform/mac/fast/css/apple-prefix-expected.txt
deleted file mode 100644 (file)
index 2c6940e..0000000
+++ /dev/null
@@ -1,73 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x600
-  RenderBlock {HTML} at (0,0) size 800x600
-    RenderBody {BODY} at (8,8) size 784x584
-      RenderBlock {P} at (0,0) size 784x18
-        RenderText {#text} at (0,0) size 764x18
-          text run at (0,0) width 764: "This tests use of the -apple-line-clamp function in a web page, from JavaScript, and using the \"apple\" prefix, not \"khtml\"."
-      RenderBlock {P} at (0,34) size 784x18
-        RenderText {#text} at (0,0) size 705x18
-          text run at (0,0) width 705: "If it works properly, the article below should show only the first few lines. Otherwise, it might show many lines."
-      RenderBlock {HR} at (0,68) size 784x2 [border: (1px inset #000000)]
-layer at (8,86) size 784x54 scrollHeight 576
-  RenderDeprecatedFlexibleBox {DIV} at (0,78) size 784x54
-    RenderBlock {DIV} at (0,0) size 784x36
-      RenderBlock {P} at (0,0) size 784x36
-        RenderText {#text} at (0,0) size 781x36
-          text run at (0,0) width 771: "It was just last night, I think, that I was IM'ing with a friend and I said, \"Next week Simon starts going to school for 3 days"
-          text run at (0,18) width 781: "extended (until 3pm) and Sophia starts swim camp! I won't know what to do with my free time! But I'm willing to find out.\""
-      RenderBlock {P} at (0,52) size 784x18
-        RenderText {#text} at (0,0) size 333x18
-          text run at (0,0) width 333: "You know the old saying: \"Man plans, God laughs.\""
-      RenderBlock {P} at (0,86) size 784x18
-        RenderText {#text} at (0,0) size 505x18
-          text run at (0,0) width 505: "Today I got a call from the preschool telling me that Sophia had broken her arm."
-      RenderBlock {P} at (0,120) size 784x108
-        RenderText {#text} at (0,0) size 773x54
-          text run at (0,0) width 770: "It happened on the playground. I imagined she had fallen off the monkey bars or something, but no: she had been sitting at"
-          text run at (0,18) width 773: "the Art Table, and as she was getting off the bench she slipped and fell on her arm. She was being very brave when I came"
-          text run at (0,36) width 452: "to get her\x{D1}no tears, stoically holding her arm covered in ice packs. She "
-        RenderInline {I} at (0,0) size 24x18
-          RenderText {#text} at (452,36) size 24x18
-            text run at (452,36) width 24: "had"
-        RenderText {#text} at (476,36) size 784x72
-          text run at (476,36) width 308: " cried, she told me. All of the teachers were quite"
-          text run at (0,54) width 780: "amazed at how brave she was being. I took her to the doctor and from there to the pediatric orthopedic doctor. She started to"
-          text run at (0,72) width 725: "lose her composure, though, after she had finally gotten the X-rays (\"Like Curious George got when he went to the"
-          text run at (0,90) width 303: "hospital\") and we had to wait in the exam room."
-      RenderBlock {P} at (0,244) size 784x18
-        RenderText {#text} at (0,0) size 424x18
-          text run at (0,0) width 424: "\"I want to go home now,\" she said. \"I wish I'd never hurt my arm.\""
-      RenderBlock {P} at (0,278) size 784x72
-        RenderText {#text} at (0,0) size 784x36
-          text run at (0,0) width 784: "The ortho tech came in to give her a splint (she'll have a splint for a couple of days and then get a cast on Thursday), and can"
-          text run at (0,18) width 382: "I tell you how worrisome it is to have an ortho tech who is a "
-        RenderInline {B} at (0,0) size 79x18
-          RenderText {#text} at (382,18) size 79x18
-            text run at (382,18) width 79: "dead ringer"
-        RenderText {#text} at (461,18) size 784x54
-          text run at (461,18) width 313: " for Ian McShane as Al Swearengen wrapping up"
-          text run at (0,36) width 784: "your daughter in gauze? (I'm totally serious. He didn't have grease in his hair, and he didn't have the little beard under his lip,"
-          text run at (0,54) width 155: "but other than that: yup.)"
-      RenderBlock {P} at (0,366) size 784x36
-        RenderText {#text} at (0,0) size 776x36
-          text run at (0,0) width 776: "So now she's on the couch, watching \"Dragontales\" with her arm propped up on a pillow (to keep it above the heart). She's"
-          text run at (0,18) width 600: "eating a couple of cut-up apples and cheese sticks (\"because that's all I can eat now, Mommy\")."
-      RenderBlock {P} at (0,418) size 784x54
-        RenderText {#text} at (0,0) size 771x54
-          text run at (0,0) width 771: "She's upset because she's not going to be able to do all the swimming she was planning. I'm upset because my little girl got"
-          text run at (0,18) width 746: "hurt. I assume that after a day or so we'll adjust to the new regimen. Of course, she can't go to her last day of preschool"
-          text run at (0,36) width 449: "tomorrow, because they're having \"Water Days,\" and she can't get wet."
-      RenderBlock {P} at (0,488) size 784x18
-        RenderText {#text} at (0,0) size 734x18
-          text run at (0,0) width 734: "The weather's finally gotten extremely hot around here, and she can't get her splint (and in a few days, her cast) wet. "
-        RenderInline {I} at (0,0) size 34x18
-          RenderText {#text} at (734,0) size 34x18
-            text run at (734,0) width 34: "Neat."
-      RenderBlock {P} at (0,522) size 784x54
-        RenderText {#text} at (0,0) size 773x54
-          text run at (0,0) width 767: "I've never broken anything (knock on nearest wood sprite)\x{D1}does it affect your outlook on life? She has an adventuresome"
-          text run at (0,18) width 773: "spirit and I hope she's just as energetic after this little incident as she was before. At the moment she's even lost her sense of"
-          text run at (0,36) width 45: "humor."
-    RenderBlock {HR} at (0,44) size 784x2 [border: (1px inset #000000)]
index 5763bc5..6831955 100644 (file)
@@ -14,10 +14,10 @@ var style = document.getElementById('description').style;
 function parseDashboardRegion(string)
 {
     try {
-        style.removeProperty("-apple-dashboard-region");
-        style.setProperty("-apple-dashboard-region", string, "");
-        var result = style.getPropertyValue("-apple-dashboard-region");
-        style.removeProperty("-apple-dashboard-region");
+        style.removeProperty("-webkit-dashboard-region");
+        style.setProperty("-webkit-dashboard-region", string, "");
+        var result = style.getPropertyValue("-webkit-dashboard-region");
+        style.removeProperty("-webkit-dashboard-region");
         return result;
     } catch (e) {
         return "" + e;
index d3c10db..d70aed0 100644 (file)
@@ -2159,7 +2159,6 @@ fast/loader/reload-zero-byte-plugin.html
 
 # These test -apple- and -khtml- prefixed CSS properties, which we don't support.
 inspector/styles/vendor-prefixes.html
-fast/css/apple-prefix.html
 
 # [Qt] fast/dom/HTMLDocument/hasFocus.html fails
 # https://bugs.webkit.org/show_bug.cgi?id=66076
diff --git a/LayoutTests/platform/qt/fast/css/apple-prefix-expected.png b/LayoutTests/platform/qt/fast/css/apple-prefix-expected.png
deleted file mode 100644 (file)
index 73ec065..0000000
Binary files a/LayoutTests/platform/qt/fast/css/apple-prefix-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/qt/fast/css/apple-prefix-expected.txt b/LayoutTests/platform/qt/fast/css/apple-prefix-expected.txt
deleted file mode 100644 (file)
index dcee3fc..0000000
+++ /dev/null
@@ -1,76 +0,0 @@
-layer at (0,0) size 784x871
-  RenderView at (0,0) size 784x600
-layer at (0,0) size 784x871
-  RenderBlock {HTML} at (0,0) size 784x871
-    RenderBody {BODY} at (8,8) size 768x855
-      RenderBlock {P} at (0,0) size 768x42
-        RenderText {#text} at (0,0) size 758x42
-          text run at (0,0) width 758: "This tests use of the -apple-line-clamp function in a web page, from JavaScript, and using the \"apple\" prefix,"
-          text run at (0,21) width 80: "not \"khtml\"."
-      RenderBlock {P} at (0,58) size 768x21
-        RenderText {#text} at (0,0) size 768x21
-          text run at (0,0) width 768: "If it works properly, the article below should show only the first few lines. Otherwise, it might show many lines."
-      RenderBlock {HR} at (0,95) size 768x2 [border: (1px inset #000000)]
-      RenderDeprecatedFlexibleBox {DIV} at (0,105) size 768x750
-        RenderBlock {DIV} at (0,0) size 768x732
-          RenderBlock {P} at (0,0) size 768x63
-            RenderText {#text} at (0,0) size 765x63
-              text run at (0,0) width 765: "It was just last night, I think, that I was IM'ing with a friend and I said, \"Next week Simon starts going to school"
-              text run at (0,21) width 757: "for 3 days extended (until 3pm) and Sophia starts swim camp! I won't know what to do with my free time! But"
-              text run at (0,42) width 153: "I'm willing to find out.\""
-          RenderBlock {P} at (0,79) size 768x21
-            RenderText {#text} at (0,0) size 362x21
-              text run at (0,0) width 362: "You know the old saying: \"Man plans, God laughs.\""
-          RenderBlock {P} at (0,116) size 768x21
-            RenderText {#text} at (0,0) size 560x21
-              text run at (0,0) width 560: "Today I got a call from the preschool telling me that Sophia had broken her arm."
-          RenderBlock {P} at (0,153) size 768x147
-            RenderText {#text} at (0,0) size 757x63
-              text run at (0,0) width 757: "It happened on the playground. I imagined she had fallen off the monkey bars or something, but no: she had"
-              text run at (0,21) width 752: "been sitting at the Art Table, and as she was getting off the bench she slipped and fell on her arm. She was"
-              text run at (0,42) width 722: "being very brave when I came to get her\x{D1}no tears, stoically holding her arm covered in ice packs. She "
-            RenderInline {I} at (0,0) size 27x21
-              RenderText {#text} at (722,42) size 27x21
-                text run at (722,42) width 27: "had"
-            RenderText {#text} at (0,63) size 765x84
-              text run at (0,63) width 765: "cried, she told me. All of the teachers were quite amazed at how brave she was being. I took her to the doctor"
-              text run at (0,84) width 744: "and from there to the pediatric orthopedic doctor. She started to lose her composure, though, after she had"
-              text run at (0,105) width 751: "finally gotten the X-rays (\"Like Curious George got when he went to the hospital\") and we had to wait in the"
-              text run at (0,126) width 83: "exam room."
-          RenderBlock {P} at (0,316) size 768x21
-            RenderText {#text} at (0,0) size 455x21
-              text run at (0,0) width 455: "\"I want to go home now,\" she said. \"I wish I'd never hurt my arm.\""
-          RenderBlock {P} at (0,353) size 768x84
-            RenderText {#text} at (0,0) size 737x42
-              text run at (0,0) width 737: "The ortho tech came in to give her a splint (she'll have a splint for a couple of days and then get a cast on"
-              text run at (0,21) width 559: "Thursday), and can I tell you how worrisome it is to have an ortho tech who is a "
-            RenderInline {B} at (0,0) size 87x21
-              RenderText {#text} at (559,21) size 87x21
-                text run at (559,21) width 87: "dead ringer"
-            RenderText {#text} at (646,21) size 766x63
-              text run at (646,21) width 120: " for Ian McShane"
-              text run at (0,42) width 735: "as Al Swearengen wrapping up your daughter in gauze? (I'm totally serious. He didn't have grease in his"
-              text run at (0,63) width 538: "hair, and he didn't have the little beard under his lip, but other than that: yup.)"
-          RenderBlock {P} at (0,453) size 768x63
-            RenderText {#text} at (0,0) size 743x63
-              text run at (0,0) width 743: "So now she's on the couch, watching \"Dragontales\" with her arm propped up on a pillow (to keep it above"
-              text run at (0,21) width 716: "the heart). She's eating a couple of cut-up apples and cheese sticks (\"because that's all I can eat now,"
-              text run at (0,42) width 71: "Mommy\")."
-          RenderBlock {P} at (0,532) size 768x63
-            RenderText {#text} at (0,0) size 764x63
-              text run at (0,0) width 760: "She's upset because she's not going to be able to do all the swimming she was planning. I'm upset because"
-              text run at (0,21) width 764: "my little girl got hurt. I assume that after a day or so we'll adjust to the new regimen. Of course, she can't go to"
-              text run at (0,42) width 675: "her last day of preschool tomorrow, because they're having \"Water Days,\" and she can't get wet."
-          RenderBlock {P} at (0,611) size 768x42
-            RenderText {#text} at (0,0) size 744x42
-              text run at (0,0) width 744: "The weather's finally gotten extremely hot around here, and she can't get her splint (and in a few days, her"
-              text run at (0,21) width 71: "cast) wet. "
-            RenderInline {I} at (0,0) size 38x21
-              RenderText {#text} at (71,21) size 38x21
-                text run at (71,21) width 38: "Neat."
-          RenderBlock {P} at (0,669) size 768x63
-            RenderText {#text} at (0,0) size 740x63
-              text run at (0,0) width 740: "I've never broken anything (knock on nearest wood sprite)\x{D1}does it affect your outlook on life? She has an"
-              text run at (0,21) width 727: "adventuresome spirit and I hope she's just as energetic after this little incident as she was before. At the"
-              text run at (0,42) width 310: "moment she's even lost her sense of humor."
-        RenderBlock {HR} at (0,740) size 768x2 [border: (1px inset #000000)]
index 1f0f1f2..6ff9e8b 100644 (file)
@@ -1,3 +1,37 @@
+2013-03-17  Adam Barth  <abarth@webkit.org>
+
+        Legacy CSS vendor prefixes should only work for Dashboard
+        https://bugs.webkit.org/show_bug.cgi?id=111890
+
+        Reviewed by Eric Seidel.
+
+        Our experience with the Chromium port is that these legacy CSS vendor
+        prefixes (-apple- and -khtml-) are not needed for web compatibility.
+        There is reason to believe, however, that they are needed for
+        compatibility with Mac OS X Dashboard widgets.
+
+        This patch makes the code for these legacy CSS vendor prefixes
+        enabled at runtime and only enables them when running in Dashboard
+        compatibility mode. This is the first step towards the plan outlined in
+        https://lists.webkit.org/pipermail/webkit-dev/2013-March/024085.html.
+
+        This patch also removes support for ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
+        from the V8 bindings because V8 is never used with Dashboard.
+
+        * bindings/generic/RuntimeEnabledFeatures.cpp:
+        (WebCore):
+        * bindings/generic/RuntimeEnabledFeatures.h:
+        (RuntimeEnabledFeatures):
+        (WebCore::RuntimeEnabledFeatures::setLegacyCSSVendorPrefixesEnabled):
+        (WebCore::RuntimeEnabledFeatures::legacyCSSVendorPrefixesEnabled):
+        * bindings/js/JSCSSStyleDeclarationCustom.cpp:
+        (WebCore::getCSSPropertyNamePrefix):
+        (WebCore::cssPropertyIDForJSCSSPropertyName):
+        * bindings/v8/custom/V8CSSStyleDeclarationCustom.cpp:
+        (WebCore::cssPropertyInfo):
+        * css/CSSParser.cpp:
+        (WebCore::CSSParser::rewriteSpecifiers):
+
 2013-03-17  David Kilzer  <ddkilzer@apple.com>
 
         BUILD FIX (r145592): AutodrainedPool.h moved to WTF
index 1fb7d68..0446fad 100644 (file)
@@ -2841,6 +2841,10 @@ __ZN7WebCore22RuntimeEnabledFeatures23isCSSCompositingEnabledE
 __ZN7WebCore22RuntimeEnabledFeatures25areSeamlessIFramesEnabledE
 #endif
 
+#if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
+__ZN7WebCore22RuntimeEnabledFeatures32isLegacyCSSVendorPrefixesEnabledE
+#endif
+
 #if ENABLE(ENCRYPTED_MEDIA_V2)
 __ZN7WebCore3CDM18registerCDMFactoryEPFN3WTF10PassOwnPtrINS_19CDMPrivateInterfaceEEEPS0_EPFbRKNS1_6StringEE
 #endif
index 71c3864..0702176 100644 (file)
@@ -69,6 +69,10 @@ bool RuntimeEnabledFeatures::isMediaStreamEnabled = true;
 bool RuntimeEnabledFeatures::isPeerConnectionEnabled = true;
 #endif
 
+#if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
+bool RuntimeEnabledFeatures::isLegacyCSSVendorPrefixesEnabled = false;
+#endif
+
 #if ENABLE(GAMEPAD)
 bool RuntimeEnabledFeatures::isGamepadEnabled = false;
 #endif
index f74994f..977bbb2 100644 (file)
@@ -201,6 +201,11 @@ public:
     static bool webkitRTCPeerConnectionEnabled() { return peerConnectionEnabled(); }
 #endif
 
+#if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
+    static void setLegacyCSSVendorPrefixesEnabled(bool isEnabled) { isLegacyCSSVendorPrefixesEnabled = isEnabled; }
+    static bool legacyCSSVendorPrefixesEnabled() { return isLegacyCSSVendorPrefixesEnabled; }
+#endif
+
 #if ENABLE(GAMEPAD)
     static void setWebkitGetGamepadsEnabled(bool isEnabled) { isGamepadEnabled = isEnabled; }
     static bool webkitGetGamepadsEnabled() { return isGamepadEnabled; }
@@ -339,6 +344,10 @@ private:
     static bool isGamepadEnabled;
 #endif
 
+#if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
+    static bool isLegacyCSSVendorPrefixesEnabled;
+#endif
+
 #if ENABLE(QUOTA)
     static bool isQuotaEnabled;
 #endif
index 646d71c..dcb9649 100644 (file)
@@ -33,6 +33,7 @@
 #include "HashTools.h"
 #include "JSCSSValue.h"
 #include "JSNode.h"
+#include "RuntimeEnabledFeatures.h"
 #include "Settings.h"
 #include "StylePropertySet.h"
 #include <runtime/StringPrototype.h>
@@ -119,7 +120,7 @@ static PropertyNamePrefix getCSSPropertyNamePrefix(const StringImpl& propertyNam
     switch (firstChar) {
 #if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
     case 'a':
-        if (matchesCSSPropertyNamePrefix(propertyName, "apple"))
+        if (RuntimeEnabledFeatures::legacyCSSVendorPrefixesEnabled() && matchesCSSPropertyNamePrefix(propertyName, "apple"))
             return PropertyNamePrefixApple;
         break;
 #endif
@@ -129,7 +130,7 @@ static PropertyNamePrefix getCSSPropertyNamePrefix(const StringImpl& propertyNam
         break;
 #if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
     case 'k':
-        if (matchesCSSPropertyNamePrefix(propertyName, "khtml"))
+        if (RuntimeEnabledFeatures::legacyCSSVendorPrefixesEnabled() && matchesCSSPropertyNamePrefix(propertyName, "khtml"))
             return PropertyNamePrefixKHTML;
         break;
 #endif
@@ -222,6 +223,7 @@ static CSSPropertyInfo cssPropertyIDForJSCSSPropertyName(PropertyName propertyNa
 #if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
     case PropertyNamePrefixApple:
     case PropertyNamePrefixKHTML:
+        ASSERT(RuntimeEnabledFeatures::legacyCSSVendorPrefixesEnabled());
         writeWebKitPrefix(bufferPtr);
         i += 5;
         break;
index 702dd4b..82b2f9e 100644 (file)
@@ -124,12 +124,7 @@ static CSSPropertyInfo* cssPropertyInfo(v8::Handle<v8::String>v8PropertyName)
         } else if (hasCSSPropertyNamePrefix(propertyName, "pos")) {
             i += 3;
             hadPixelOrPosPrefix = true;
-        } else if (hasCSSPropertyNamePrefix(propertyName, "webkit")
-#if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
-                || hasCSSPropertyNamePrefix(propertyName, "khtml")
-                || hasCSSPropertyNamePrefix(propertyName, "apple")
-#endif
-                  )
+        } else if (hasCSSPropertyNamePrefix(propertyName, "webkit"))
             builder.append('-');
         else if (isASCIIUpper(propertyName[0]))
             return 0;
index 3d8816a..1023539 100644 (file)
@@ -11822,7 +11822,8 @@ static CSSPropertyID cssPropertyID(const CharacterType* propertyName, unsigned l
 #if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
         // If the prefix is -apple- or -khtml-, change it to -webkit-.
         // This makes the string one character longer.
-        if (hasPrefix(buffer, length, "-apple-") || hasPrefix(buffer, length, "-khtml-")) {
+        if (RuntimeEnabledFeatures::legacyCSSVendorPrefixesEnabled()
+            && (hasPrefix(buffer, length, "-apple-") || hasPrefix(buffer, length, "-khtml-"))) {
             memmove(buffer + 7, buffer + 6, length + 1 - 6);
             memcpy(buffer, "-webkit", 7);
             ++length;
index e457a69..3a540c7 100644 (file)
@@ -1,3 +1,16 @@
+2013-03-17  Adam Barth  <abarth@webkit.org>
+
+        Legacy CSS vendor prefixes should only work for Dashboard
+        https://bugs.webkit.org/show_bug.cgi?id=111890
+
+        Reviewed by Eric Seidel.
+
+        Enable legacy CSS vendor prefixes when we've been asked to turn on
+        Dashboard compatibility mode.
+
+        * WebView/WebView.mm:
+        (-[WebView _setDashboardBehavior:to:]):
+
 2013-03-16  Pratik Solanki  <psolanki@apple.com>
 
         Disable High DPI Canvas on iOS
index d7542bb..22f1806 100644 (file)
@@ -2203,6 +2203,9 @@ static inline IMP getMethod(id o, SEL s)
         case WebDashboardBehaviorUseBackwardCompatibilityMode: {
             if (_private->page)
                 _private->page->settings()->setUsesDashboardBackwardCompatibilityMode(flag);
+#if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)
+            RuntimeEnabledFeatures::setLegacyCSSVendorPrefixesEnabled(flag);
+#endif
             break;
         }
     }