Breaking: Remove incorrect hwb() syntax support from normalize-color

Summary:
The implementation of `hwb()` color functions added in #34600 is pretty flawed.

`hwb()` color functions do not allow comma delimited values. So most of the examples in the unit test here will fail to parse on web. Like `hsl()`, these should also allow numeric non-hue components (instead of just %), and angle values for hue (instead of just numbers), and this is also missing support for alpha values, though these are less dangerous compared to allowing and encouraging incorrect delimiters.

https://www.w3.org/TR/css-color-4/#the-hwb-notation

These were added for web compat, and the examples fail to parse on web, so I'm opting to just remove this incorrect support before implementing this more correctly in the Fabric CSS parser in next diff. I did not attempt to fix the other issues I discovered with the PR implementation in the last couple diffs, around mixing and matching syntax allowed in legacy/modern, along with allowing inconsistent delimiters. 

Changelog:
[General][Breaking] - Remove incorrect hwb() syntax support from normalize-color

Differential Revision: D68591172

Author: NickGerleman

packages/normalize-color/__tests__/normalizeColor-test.js

@@ -127,14 +127,15 @@ it('handles hsla properly', () => {
 });
 
 it('handles hwb properly', () => {
-  expect(normalizeColor('hwb(0, 0%, 100%)')).toBe(0x000000ff);
-  expect(normalizeColor('hwb(0, 100%, 0%)')).toBe(0xffffffff);
-  expect(normalizeColor('hwb(0, 0%, 0%)')).toBe(0xff0000ff);
-  expect(normalizeColor('hwb(70, 50%, 0%)')).toBe(0xeaff80ff);
-  expect(normalizeColor('hwb(0, 50%, 50%)')).toBe(0x808080ff);
-  expect(normalizeColor('hwb(360, 100%, 100%)')).toBe(0x808080ff);
+  expect(normalizeColor('hwb(0 0% 100%)')).toBe(0x000000ff);
+  expect(normalizeColor('hwb(0 100% 0%)')).toBe(0xffffffff);
   expect(normalizeColor('hwb(0 0% 0%)')).toBe(0xff0000ff);
   expect(normalizeColor('hwb(70 50% 0%)')).toBe(0xeaff80ff);
+  expect(normalizeColor('hwb(0 50% 50%)')).toBe(0x808080ff);
+  expect(normalizeColor('hwb(360 100% 100%)')).toBe(0x808080ff);
+  expect(normalizeColor('hwb(0 0% 0%)')).toBe(0xff0000ff);
+  expect(normalizeColor('hwb(70 50% 0%)')).toBe(0xeaff80ff);
+  expect(normalizeColor('hwb(0, 0%, 100%)')).toBe(null);
 });
 
 it('handles named colors properly', () => {

packages/normalize-color/index.js

@@ -216,6 +216,10 @@ function call(...args) {
   return '\\(\\s*(' + args.join(')\\s*,?\\s*(') + ')\\s*\\)';
 }
 
+function callModern(...args) {
+  return '\\(\\s*(' + args.join(')\\s*(') + ')\\s*\\)';
+}
+
 function callWithSlashSeparator(...args) {
   return (
     '\\(\\s*(' +
@@ -251,7 +255,7 @@ function getMatchers() {
           callWithSlashSeparator(NUMBER, PERCENTAGE, PERCENTAGE, NUMBER) +
           ')',
       ),
-      hwb: new RegExp('hwb' + call(NUMBER, PERCENTAGE, PERCENTAGE)),
+      hwb: new RegExp('hwb' + callModern(NUMBER, PERCENTAGE, PERCENTAGE)),
       hex3: /^#([0-9a-fA-F]{1})([0-9a-fA-F]{1})([0-9a-fA-F]{1})$/,
       hex4: /^#([0-9a-fA-F]{1})([0-9a-fA-F]{1})([0-9a-fA-F]{1})([0-9a-fA-F]{1})$/,
       hex6: /^#([0-9a-fA-F]{6})$/,