I was implementing the WPT facing parts of GPC for Chrome (https://crrev.com/c/8186637) and noticed a few documentation errors (web-platform-tests/wpt#61761) as well as an inconsistency in the way this spec defines set_global_privacy_control and TestDriver implements it.
WPTs expect that set_global_privacy_control returns the updated GPC status, which seems reasonable: https://github.com/web-platform-tests/wpt/blob/f9a50c1cea3305dbea06dcf55cce4fbb01839fe2/gpc/global_privacy_control.testdriver.html#L14
The spec defines this method as returning null on success: https://www.w3.org/TR/gpc/#set-global-privacy-control
I think the spec should be moved to align with TestDriver, but before putting up a PR for that I wanted to get feedback.
I was implementing the WPT facing parts of GPC for Chrome (https://crrev.com/c/8186637) and noticed a few documentation errors (web-platform-tests/wpt#61761) as well as an inconsistency in the way this spec defines set_global_privacy_control and TestDriver implements it.
WPTs expect that set_global_privacy_control returns the updated GPC status, which seems reasonable: https://github.com/web-platform-tests/wpt/blob/f9a50c1cea3305dbea06dcf55cce4fbb01839fe2/gpc/global_privacy_control.testdriver.html#L14
The spec defines this method as returning null on success: https://www.w3.org/TR/gpc/#set-global-privacy-control
I think the spec should be moved to align with TestDriver, but before putting up a PR for that I wanted to get feedback.