Title: [235802] branches/safari-606-branch
Revision
235802
Author
[email protected]
Date
2018-09-07 13:08:20 -0700 (Fri, 07 Sep 2018)

Log Message

Cherry-pick r235754. rdar://problem/44210305

    [Apple Pay] Rename the -apple-pay-button-type value "checkout" to "check-out"
    https://bugs.webkit.org/show_bug.cgi?id=189366
    <rdar://problem/44193218>

    Reviewed by Sam Weinig.

    Source/WebCore:

    The button title is "Check out with Apple Pay", so we should use the verb phrase "check out"
    rather than the noun "checkout" for naming this value.

    Updated http/tests/ssl/applepay/ApplePayButtonV4.html.

    * css/CSSPrimitiveValueMappings.h:
    (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
    (WebCore::CSSPrimitiveValue::operator ApplePayButtonType const):
    * css/CSSValueKeywords.in:
    * css/parser/CSSParserFastPaths.cpp:
    (WebCore::CSSParserFastPaths::isValidKeywordPropertyAndValue):
    * rendering/RenderThemeCocoa.mm:
    (WebCore::toPKPaymentButtonType):
    * rendering/style/RenderStyleConstants.h:

    Source/WebInspectorUI:

    * UserInterface/Models/CSSKeywordCompletions.js:

    LayoutTests:

    * http/tests/ssl/applepay/ApplePayButtonV4.html:

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

Modified Paths

Diff

Modified: branches/safari-606-branch/LayoutTests/ChangeLog (235801 => 235802)


--- branches/safari-606-branch/LayoutTests/ChangeLog	2018-09-07 20:08:16 UTC (rev 235801)
+++ branches/safari-606-branch/LayoutTests/ChangeLog	2018-09-07 20:08:20 UTC (rev 235802)
@@ -1,5 +1,52 @@
 2018-09-06  Babak Shafiei  <[email protected]>
 
+        Cherry-pick r235754. rdar://problem/44210305
+
+    [Apple Pay] Rename the -apple-pay-button-type value "checkout" to "check-out"
+    https://bugs.webkit.org/show_bug.cgi?id=189366
+    <rdar://problem/44193218>
+    
+    Reviewed by Sam Weinig.
+    
+    Source/WebCore:
+    
+    The button title is "Check out with Apple Pay", so we should use the verb phrase "check out"
+    rather than the noun "checkout" for naming this value.
+    
+    Updated http/tests/ssl/applepay/ApplePayButtonV4.html.
+    
+    * css/CSSPrimitiveValueMappings.h:
+    (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
+    (WebCore::CSSPrimitiveValue::operator ApplePayButtonType const):
+    * css/CSSValueKeywords.in:
+    * css/parser/CSSParserFastPaths.cpp:
+    (WebCore::CSSParserFastPaths::isValidKeywordPropertyAndValue):
+    * rendering/RenderThemeCocoa.mm:
+    (WebCore::toPKPaymentButtonType):
+    * rendering/style/RenderStyleConstants.h:
+    
+    Source/WebInspectorUI:
+    
+    * UserInterface/Models/CSSKeywordCompletions.js:
+    
+    LayoutTests:
+    
+    * http/tests/ssl/applepay/ApplePayButtonV4.html:
+    
+    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@235754 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+    2018-09-06  Andy Estes  <[email protected]>
+
+            [Apple Pay] Rename the -apple-pay-button-type value "checkout" to "check-out"
+            https://bugs.webkit.org/show_bug.cgi?id=189366
+            <rdar://problem/44193218>
+
+            Reviewed by Sam Weinig.
+
+            * http/tests/ssl/applepay/ApplePayButtonV4.html:
+
+2018-09-06  Babak Shafiei  <[email protected]>
+
         Cherry-pick r235349. rdar://problem/44209834
 
     [Apple Pay] Introduce new values for -apple-pay-button-type

Modified: branches/safari-606-branch/LayoutTests/http/tests/ssl/applepay/ApplePayButtonV4.html (235801 => 235802)


--- branches/safari-606-branch/LayoutTests/http/tests/ssl/applepay/ApplePayButtonV4.html	2018-09-07 20:08:16 UTC (rev 235801)
+++ branches/safari-606-branch/LayoutTests/http/tests/ssl/applepay/ApplePayButtonV4.html	2018-09-07 20:08:20 UTC (rev 235802)
@@ -21,8 +21,8 @@
             -apple-pay-button-style: black;
         }
 
-        .checkout {
-            -apple-pay-button-type: checkout;
+        .check-out {
+            -apple-pay-button-type: check-out;
         }
 
         .book {
@@ -37,7 +37,7 @@
 <body>
     <script>
         for (let style of ["white", "white-outline", "black"]) {
-            for (let type of ["checkout", "book", "subscribe"]) {
+            for (let type of ["check-out", "book", "subscribe"]) {
                 var button = document.createElement("button");
                 button.classList.add("apple-pay-button", style, type);
                 document.body.appendChild(button);

Modified: branches/safari-606-branch/Source/WebCore/ChangeLog (235801 => 235802)


--- branches/safari-606-branch/Source/WebCore/ChangeLog	2018-09-07 20:08:16 UTC (rev 235801)
+++ branches/safari-606-branch/Source/WebCore/ChangeLog	2018-09-07 20:08:20 UTC (rev 235802)
@@ -1,5 +1,65 @@
 2018-09-06  Babak Shafiei  <[email protected]>
 
+        Cherry-pick r235754. rdar://problem/44210305
+
+    [Apple Pay] Rename the -apple-pay-button-type value "checkout" to "check-out"
+    https://bugs.webkit.org/show_bug.cgi?id=189366
+    <rdar://problem/44193218>
+    
+    Reviewed by Sam Weinig.
+    
+    Source/WebCore:
+    
+    The button title is "Check out with Apple Pay", so we should use the verb phrase "check out"
+    rather than the noun "checkout" for naming this value.
+    
+    Updated http/tests/ssl/applepay/ApplePayButtonV4.html.
+    
+    * css/CSSPrimitiveValueMappings.h:
+    (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
+    (WebCore::CSSPrimitiveValue::operator ApplePayButtonType const):
+    * css/CSSValueKeywords.in:
+    * css/parser/CSSParserFastPaths.cpp:
+    (WebCore::CSSParserFastPaths::isValidKeywordPropertyAndValue):
+    * rendering/RenderThemeCocoa.mm:
+    (WebCore::toPKPaymentButtonType):
+    * rendering/style/RenderStyleConstants.h:
+    
+    Source/WebInspectorUI:
+    
+    * UserInterface/Models/CSSKeywordCompletions.js:
+    
+    LayoutTests:
+    
+    * http/tests/ssl/applepay/ApplePayButtonV4.html:
+    
+    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@235754 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+    2018-09-06  Andy Estes  <[email protected]>
+
+            [Apple Pay] Rename the -apple-pay-button-type value "checkout" to "check-out"
+            https://bugs.webkit.org/show_bug.cgi?id=189366
+            <rdar://problem/44193218>
+
+            Reviewed by Sam Weinig.
+
+            The button title is "Check out with Apple Pay", so we should use the verb phrase "check out"
+            rather than the noun "checkout" for naming this value.
+
+            Updated http/tests/ssl/applepay/ApplePayButtonV4.html.
+
+            * css/CSSPrimitiveValueMappings.h:
+            (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
+            (WebCore::CSSPrimitiveValue::operator ApplePayButtonType const):
+            * css/CSSValueKeywords.in:
+            * css/parser/CSSParserFastPaths.cpp:
+            (WebCore::CSSParserFastPaths::isValidKeywordPropertyAndValue):
+            * rendering/RenderThemeCocoa.mm:
+            (WebCore::toPKPaymentButtonType):
+            * rendering/style/RenderStyleConstants.h:
+
+2018-09-06  Babak Shafiei  <[email protected]>
+
         Cherry-pick r235349. rdar://problem/44209834
 
     [Apple Pay] Introduce new values for -apple-pay-button-type

Modified: branches/safari-606-branch/Source/WebCore/css/CSSPrimitiveValueMappings.h (235801 => 235802)


--- branches/safari-606-branch/Source/WebCore/css/CSSPrimitiveValueMappings.h	2018-09-07 20:08:16 UTC (rev 235801)
+++ branches/safari-606-branch/Source/WebCore/css/CSSPrimitiveValueMappings.h	2018-09-07 20:08:20 UTC (rev 235802)
@@ -5472,8 +5472,8 @@
         m_value.valueID = CSSValueDonate;
         break;
 #if ENABLE(APPLE_PAY_SESSION_V4)
-    case ApplePayButtonType::Checkout:
-        m_value.valueID = CSSValueCheckout;
+    case ApplePayButtonType::CheckOut:
+        m_value.valueID = CSSValueCheckOut;
         break;
     case ApplePayButtonType::Book:
         m_value.valueID = CSSValueBook;
@@ -5504,8 +5504,8 @@
     case CSSValueDonate:
         return ApplePayButtonType::Donate;
 #if ENABLE(APPLE_PAY_SESSION_V4)
-    case CSSValueCheckout:
-        return ApplePayButtonType::Checkout;
+    case CSSValueCheckOut:
+        return ApplePayButtonType::CheckOut;
     case CSSValueBook:
         return ApplePayButtonType::Book;
     case CSSValueSubscribe:

Modified: branches/safari-606-branch/Source/WebCore/css/CSSValueKeywords.in (235801 => 235802)


--- branches/safari-606-branch/Source/WebCore/css/CSSValueKeywords.in	2018-09-07 20:08:16 UTC (rev 235801)
+++ branches/safari-606-branch/Source/WebCore/css/CSSValueKeywords.in	2018-09-07 20:08:20 UTC (rev 235802)
@@ -1331,7 +1331,7 @@
 in-store
 donate
 #if defined(ENABLE_APPLE_PAY_SESSION_V4) && ENABLE_APPLE_PAY_SESSION_V4
-checkout
+check-out
 book
 subscribe
 #endif

Modified: branches/safari-606-branch/Source/WebCore/css/parser/CSSParserFastPaths.cpp (235801 => 235802)


--- branches/safari-606-branch/Source/WebCore/css/parser/CSSParserFastPaths.cpp	2018-09-07 20:08:16 UTC (rev 235801)
+++ branches/safari-606-branch/Source/WebCore/css/parser/CSSParserFastPaths.cpp	2018-09-07 20:08:20 UTC (rev 235802)
@@ -769,11 +769,11 @@
 #if ENABLE(APPLE_PAY)
     case CSSPropertyApplePayButtonStyle: // white | white-outline | black
         return valueID == CSSValueWhite || valueID == CSSValueWhiteOutline || valueID == CSSValueBlack;
-    case CSSPropertyApplePayButtonType: // plain | buy | set-up | in-store | donate | checkout | book | subscribe
+    case CSSPropertyApplePayButtonType: // plain | buy | set-up | in-store | donate | check-out | book | subscribe
         if (valueID == CSSValuePlain || valueID == CSSValueBuy || valueID == CSSValueSetUp || valueID == CSSValueInStore || valueID == CSSValueDonate)
             return true;
 #if ENABLE(APPLE_PAY_SESSION_V4)
-        return valueID == CSSValueCheckout || valueID == CSSValueBook || valueID == CSSValueSubscribe;
+        return valueID == CSSValueCheckOut || valueID == CSSValueBook || valueID == CSSValueSubscribe;
 #else
         return false;
 #endif

Modified: branches/safari-606-branch/Source/WebCore/rendering/RenderThemeCocoa.mm (235801 => 235802)


--- branches/safari-606-branch/Source/WebCore/rendering/RenderThemeCocoa.mm	2018-09-07 20:08:16 UTC (rev 235801)
+++ branches/safari-606-branch/Source/WebCore/rendering/RenderThemeCocoa.mm	2018-09-07 20:08:20 UTC (rev 235802)
@@ -92,7 +92,7 @@
     case ApplePayButtonType::Donate:
         return PKPaymentButtonTypeDonate;
 #if ENABLE(APPLE_PAY_SESSION_V4)
-    case ApplePayButtonType::Checkout:
+    case ApplePayButtonType::CheckOut:
         return PKPaymentButtonTypeCheckout;
     case ApplePayButtonType::Book:
         return PKPaymentButtonTypeBook;

Modified: branches/safari-606-branch/Source/WebCore/rendering/style/RenderStyleConstants.h (235801 => 235802)


--- branches/safari-606-branch/Source/WebCore/rendering/style/RenderStyleConstants.h	2018-09-07 20:08:16 UTC (rev 235801)
+++ branches/safari-606-branch/Source/WebCore/rendering/style/RenderStyleConstants.h	2018-09-07 20:08:20 UTC (rev 235802)
@@ -1129,7 +1129,7 @@
     InStore,
     Donate,
 #if ENABLE(APPLE_PAY_SESSION_V4)
-    Checkout,
+    CheckOut,
     Book,
     Subscribe,
 #endif

Modified: branches/safari-606-branch/Source/WebInspectorUI/ChangeLog (235801 => 235802)


--- branches/safari-606-branch/Source/WebInspectorUI/ChangeLog	2018-09-07 20:08:16 UTC (rev 235801)
+++ branches/safari-606-branch/Source/WebInspectorUI/ChangeLog	2018-09-07 20:08:20 UTC (rev 235802)
@@ -1,5 +1,52 @@
 2018-09-06  Babak Shafiei  <[email protected]>
 
+        Cherry-pick r235754. rdar://problem/44210305
+
+    [Apple Pay] Rename the -apple-pay-button-type value "checkout" to "check-out"
+    https://bugs.webkit.org/show_bug.cgi?id=189366
+    <rdar://problem/44193218>
+    
+    Reviewed by Sam Weinig.
+    
+    Source/WebCore:
+    
+    The button title is "Check out with Apple Pay", so we should use the verb phrase "check out"
+    rather than the noun "checkout" for naming this value.
+    
+    Updated http/tests/ssl/applepay/ApplePayButtonV4.html.
+    
+    * css/CSSPrimitiveValueMappings.h:
+    (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
+    (WebCore::CSSPrimitiveValue::operator ApplePayButtonType const):
+    * css/CSSValueKeywords.in:
+    * css/parser/CSSParserFastPaths.cpp:
+    (WebCore::CSSParserFastPaths::isValidKeywordPropertyAndValue):
+    * rendering/RenderThemeCocoa.mm:
+    (WebCore::toPKPaymentButtonType):
+    * rendering/style/RenderStyleConstants.h:
+    
+    Source/WebInspectorUI:
+    
+    * UserInterface/Models/CSSKeywordCompletions.js:
+    
+    LayoutTests:
+    
+    * http/tests/ssl/applepay/ApplePayButtonV4.html:
+    
+    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@235754 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+    2018-09-06  Andy Estes  <[email protected]>
+
+            [Apple Pay] Rename the -apple-pay-button-type value "checkout" to "check-out"
+            https://bugs.webkit.org/show_bug.cgi?id=189366
+            <rdar://problem/44193218>
+
+            Reviewed by Sam Weinig.
+
+            * UserInterface/Models/CSSKeywordCompletions.js:
+
+2018-09-06  Babak Shafiei  <[email protected]>
+
         Cherry-pick r235406. rdar://problem/44210305
 
     Teach Web Inspector how to complete keywords for -apple-pay-button-style and -apple-pay-button-type

Modified: branches/safari-606-branch/Source/WebInspectorUI/UserInterface/Models/CSSKeywordCompletions.js (235801 => 235802)


--- branches/safari-606-branch/Source/WebInspectorUI/UserInterface/Models/CSSKeywordCompletions.js	2018-09-07 20:08:16 UTC (rev 235801)
+++ branches/safari-606-branch/Source/WebInspectorUI/UserInterface/Models/CSSKeywordCompletions.js	2018-09-07 20:08:20 UTC (rev 235802)
@@ -1063,6 +1063,6 @@
         "black", "white", "white-outline"
     ],
     "-apple-pay-button-type": [
-        "book", "buy", "checkout", "donate", "in-store", "plain", "set-up", "subscribe"
+        "book", "buy", "check-out", "donate", "in-store", "plain", "set-up", "subscribe"
     ]
 };
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to