browser-compat-data icon indicating copy to clipboard operation
browser-compat-data copied to clipboard

Clean up data for Permissions Policy header

Open queengooborg opened this issue 1 year ago • 3 comments

This PR cleans up a lot of data for the Permissions Policy HTTP header. The alternative name was copied down to all subfeatures, which is improper.

Additionally, this marks the header as unsupported in Firefox and Safari, as the header is not supported, and support was only indicated because the values are supported as values to <iframe allow="">. The features were copied to #23487 to untangle this data.

This fixes #15987.

queengooborg avatar Jun 20 '24 14:06 queengooborg

This pull request has merge conflicts that must be resolved before it can be merged.

github-actions[bot] avatar Sep 10 '24 11:09 github-actions[bot]

Sent from my iPadOn Oct 3, 2024, at 10:38, Claas Augner @.***> wrote: @caugner commented on this pull request.

In http/headers/Permissions-Policy.json:

  •          "safari": {
    
  •            "version_added": false
    
  •          },
    
  •          "safari_ios": "mirror",
    
  •          "samsunginternet_android": "mirror",
    
  •          "webview_android": "mirror",
    
  •          "webview_ios": "mirror"
    
  •        },
    
  •        "status": {
    
  •          "experimental": false,
    
  •          "standard_track": true,
    
  •          "deprecated": false
    
  •        }
    
  •      }
    
  •    },
    
  •    "battery": {
    

Mind adding battery in a separate PR?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

alexjalonso7777 avatar Oct 04 '24 01:10 alexjalonso7777

The changes regarding the removal of "Feature-Policy" were stated in the description, in that the alternative name was removed from all subfeatures.

The failing lint is actually not because of that change, though, but rather because gamepad and speaker-selection were incorrectly stated as supported on the header when they're only supported on the allow iframe attribute (see the notes that were removed). It looks like the data was change for those two features for Chrome/Safari in between which set them to false, so now all browser engines say the feature is unsupported, and thus they should be removed. Doing so now.

queengooborg avatar Oct 19 '24 07:10 queengooborg

This pull request has merge conflicts that must be resolved before it can be merged.

github-actions[bot] avatar Oct 29 '24 15:10 github-actions[bot]