Bug 1820749 - [devtool] Update compatibility data r=devtools-reviewer…

…s,nchevobbe Differential Revision: https://phabricator.services.mozilla.com/D171850
iaxat · Mar 10, 2023 · 6e07a87 · 6e07a87
1 parent 2a7dbb5
commit 6e07a87
Show file tree

Hide file tree

Showing 7 changed files with 41 additions and 33 deletions.
diff --git a/...inspector/compatibility/test/browser/browser_compatibility_dynamic_js-attribute-change.js b/...inspector/compatibility/test/browser/browser_compatibility_dynamic_js-attribute-change.js
@@ -13,10 +13,10 @@ const {
 
 // Test the behavior rules are dynamically added
 
-const ISSUE_BINDING = {
+const ISSUE_OUTLINE_RADIUS = {
   type: COMPATIBILITY_ISSUE_TYPE.CSS_PROPERTY,
-  property: "-moz-binding",
-  url: "https://developer.mozilla.org/docs/Web/CSS/-moz-binding",
+  property: "-moz-outline-radius",
+  url: "https://developer.mozilla.org/docs/Web/CSS/-moz-outline-radius",
   deprecated: true,
   experimental: false,
 };
@@ -33,7 +33,7 @@ const ISSUE_HYPHENS = {
 const TEST_URI = `
   <style>
     .issue {
-      -moz-binding: none;
+      -moz-outline-radius: 25px;
     }
   </style>
   <body>
@@ -82,8 +82,8 @@ add_task(async function() {
     inspector,
     selectedElementPane,
     allElementsPane,
-    [ISSUE_HYPHENS, ISSUE_BINDING],
-    [ISSUE_HYPHENS, ISSUE_BINDING],
+    [ISSUE_HYPHENS, ISSUE_OUTLINE_RADIUS],
+    [ISSUE_HYPHENS, ISSUE_OUTLINE_RADIUS],
     async function() {
       content.document.querySelector(".test").classList.add("issue");
     }

diff --git a/...lient/inspector/compatibility/test/browser/browser_compatibility_dynamic_js-dom-change.js b/...lient/inspector/compatibility/test/browser/browser_compatibility_dynamic_js-dom-change.js
@@ -14,10 +14,10 @@ const {
 
 // Test the behavior rules are dynamically added
 
-const ISSUE_BINDING = {
+const ISSUE_OUTLINE_RADIUS = {
   type: COMPATIBILITY_ISSUE_TYPE.CSS_PROPERTY,
-  property: "-moz-binding",
-  url: "https://developer.mozilla.org/docs/Web/CSS/-moz-binding",
+  property: "-moz-outline-radius",
+  url: "https://developer.mozilla.org/docs/Web/CSS/-moz-outline-radius",
   deprecated: true,
   experimental: false,
 };
@@ -34,7 +34,7 @@ const ISSUE_HYPHENS = {
 const TEST_URI = `
   <style>
     .child {
-      -moz-binding: none;
+      -moz-outline-radius: 25px;
     }
   </style>
   <body></body>
@@ -64,8 +64,8 @@ add_task(async function() {
     inspector,
     selectedElementPane,
     allElementsPane,
-    [ISSUE_BINDING],
-    [ISSUE_HYPHENS, ISSUE_BINDING],
+    [ISSUE_OUTLINE_RADIUS],
+    [ISSUE_HYPHENS, ISSUE_OUTLINE_RADIUS],
     async function() {
       const doc = content.document;
       const parent = doc.querySelector("body");

diff --git a/...t/inspector/compatibility/test/browser/browser_compatibility_dynamic_markup-dom-change.js b/...t/inspector/compatibility/test/browser/browser_compatibility_dynamic_markup-dom-change.js
@@ -14,10 +14,10 @@ const {
 
 // Test the behavior rules are dynamically added
 
-const ISSUE_BINDING = {
+const ISSUE_OUTLINE_RADIUS = {
   type: COMPATIBILITY_ISSUE_TYPE.CSS_PROPERTY,
-  property: "-moz-binding",
-  url: "https://developer.mozilla.org/docs/Web/CSS/-moz-binding",
+  property: "-moz-outline-radius",
+  url: "https://developer.mozilla.org/docs/Web/CSS/-moz-outline-radius",
   deprecated: true,
   experimental: false,
 };
@@ -34,7 +34,7 @@ const ISSUE_HYPHENS = {
 const TEST_URI = `
   <style>
     div {
-      -moz-binding: none;
+      -moz-outline-radius: 25px;
     }
   </style>
   <body>
@@ -54,16 +54,20 @@ add_task(async function() {
   const { allElementsPane, inspector } = await openCompatibilityView();
 
   info("Check initial issues");
-  await assertIssueList(allElementsPane, [ISSUE_BINDING, ISSUE_HYPHENS]);
+  await assertIssueList(allElementsPane, [ISSUE_OUTLINE_RADIUS, ISSUE_HYPHENS]);
 
   info("Delete node whose child node has CSS compatibility issue");
-  await testNodeRemoval(".parent", inspector, allElementsPane, [ISSUE_BINDING]);
+  await testNodeRemoval(".parent", inspector, allElementsPane, [
+    ISSUE_OUTLINE_RADIUS,
+  ]);
 
   info("Delete node that has CSS compatibility issue");
   await testNodeRemoval("div", inspector, allElementsPane, []);
 
   info("Add node that has CSS compatibility issue");
-  await testNodeAddition("div", inspector, allElementsPane, [ISSUE_BINDING]);
+  await testNodeAddition("div", inspector, allElementsPane, [
+    ISSUE_OUTLINE_RADIUS,
+  ]);
 
   await removeTab(tab);
 });

diff --git a/...tor/compatibility/test/browser/browser_compatibility_dynamic_ruleview-attribute-change.js b/...tor/compatibility/test/browser/browser_compatibility_dynamic_ruleview-attribute-change.js
@@ -13,10 +13,10 @@ const {
 
 // Test the behavior rules are dynamically added
 
-const ISSUE_BINDING = {
+const ISSUE_OUTLINE_RADIUS = {
   type: COMPATIBILITY_ISSUE_TYPE.CSS_PROPERTY,
-  property: "-moz-binding",
-  url: "https://developer.mozilla.org/docs/Web/CSS/-moz-binding",
+  property: "-moz-outline-radius",
+  url: "https://developer.mozilla.org/docs/Web/CSS/-moz-outline-radius",
   deprecated: true,
   experimental: false,
 };
@@ -33,7 +33,7 @@ const ISSUE_HYPHENS = {
 const TEST_URI = `
   <style>
     .issue {
-      -moz-binding: none;
+      -moz-outline-radius: 25px;
     }
   </style>
   <body>
@@ -61,21 +61,25 @@ add_task(async function() {
   await waitForCompatibilityListUpdate;
 
   info("Check initial issues");
-  await checkPanelIssues(selectedElementPane, allElementsPane, [ISSUE_BINDING]);
+  await checkPanelIssues(selectedElementPane, allElementsPane, [
+    ISSUE_OUTLINE_RADIUS,
+  ]);
 
   await addNewRule(
     "hyphens",
     "none",
     inspector,
     selectedElementPane,
     allElementsPane,
-    [ISSUE_BINDING, ISSUE_HYPHENS]
+    [ISSUE_OUTLINE_RADIUS, ISSUE_HYPHENS]
   );
 
   info("Toggle the inline issue rendering it disable");
   await togglePropStatusOnRuleView(inspector, 0, 0);
   info("Check the issues listed in panel");
-  await checkPanelIssues(selectedElementPane, allElementsPane, [ISSUE_BINDING]);
+  await checkPanelIssues(selectedElementPane, allElementsPane, [
+    ISSUE_OUTLINE_RADIUS,
+  ]);
 
   info("Toggle the class rule rendering it disabled");
   await togglePropStatusOnRuleView(inspector, 1, 0);

diff --git a/devtools/client/inspector/compatibility/test/browser/browser_compatibility_settings.js b/devtools/client/inspector/compatibility/test/browser/browser_compatibility_settings.js
@@ -8,10 +8,10 @@
 const TEST_URI = `
   <style>
   body {
-    -moz-binding: none;
+    -moz-outline-radius: 25px;
   }
   div {
-    -moz-binding: none;
+    -moz-outline-radius: 25px;
   }
   </style>
   <body><div></div></body>

diff --git a/devtools/server/actors/compatibility/lib/test/xpcshell/test_mdn-compatibility.js b/devtools/server/actors/compatibility/lib/test/xpcshell/test_mdn-compatibility.js
@@ -151,16 +151,16 @@ const TEST_DATA = [
   },
   {
     description: "Test for a property defined with prefix",
-    declarations: [{ name: "-moz-binding" }],
+    declarations: [{ name: "-moz-outline-radius" }],
     browsers: [FIREFOX_1, FIREFOX_60, FIREFOX_69],
     expectedIssues: [
       {
         type: COMPATIBILITY_ISSUE_TYPE.CSS_PROPERTY,
-        property: "-moz-binding",
-        url: "https://developer.mozilla.org/docs/Web/CSS/-moz-binding",
+        property: "-moz-outline-radius",
+        url: "https://developer.mozilla.org/docs/Web/CSS/-moz-outline-radius",
         deprecated: true,
         experimental: false,
-        unsupportedBrowsers: [FIREFOX_69],
+        unsupportedBrowsers: [],
       },
     ],
   },

diff --git a/devtools/shared/compatibility/dataset/css-properties.json b/devtools/shared/compatibility/dataset/css-properties.json