To better understand what features are valuable to developers, we should record in UMA a histogram that shows, for the Feature-Policy header, as well as the <iframe> allow attribute, which features are actually named.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a3895348d360755b3acd30ce44d51d27067b5f5a commit a3895348d360755b3acd30ce44d51d27067b5f5a Author: Paul Meyer <paulmeyer@chromium.org> Date: Thu Sep 06 20:32:49 2018 Add histogram to track use of feature policies in http response header. Change-Id: I617662f80f0bcdc723607337d8a2e153f537469b Bug: 879692 TBR: jbroman@chromium.org Reviewed-on: https://chromium-review.googlesource.com/1200149 Commit-Queue: Paul Meyer <paulmeyer@chromium.org> Reviewed-by: Ken Buchanan <kenrb@chromium.org> Reviewed-by: Ian Clelland <iclelland@chromium.org> Reviewed-by: Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#589277} [modify] https://crrev.com/a3895348d360755b3acd30ce44d51d27067b5f5a/third_party/blink/public/mojom/feature_policy/feature_policy.mojom [modify] https://crrev.com/a3895348d360755b3acd30ce44d51d27067b5f5a/third_party/blink/renderer/platform/feature_policy/feature_policy.cc [modify] https://crrev.com/a3895348d360755b3acd30ce44d51d27067b5f5a/third_party/blink/renderer/platform/feature_policy/feature_policy_test.cc [modify] https://crrev.com/a3895348d360755b3acd30ce44d51d27067b5f5a/tools/metrics/histograms/enums.xml [modify] https://crrev.com/a3895348d360755b3acd30ce44d51d27067b5f5a/tools/metrics/histograms/histograms.xml [add] https://crrev.com/a3895348d360755b3acd30ce44d51d27067b5f5a/tools/metrics/histograms/update_feature_policy_enum.py
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/920403301bcfbb663e2d7ea85ffb945f8dc5a696 commit 920403301bcfbb663e2d7ea85ffb945f8dc5a696 Author: Ehsan Karamad <ekaramad@chromium.org> Date: Wed Sep 12 15:52:34 2018 Presubmit script for feature_policy.mojom This CL adds a short pre-submit script to warn against changes to feature_policy.mojom. Bug: 879692 Change-Id: Ib55912b978737c62d5a01f472456ba2df8f9d059 Reviewed-on: https://chromium-review.googlesource.com/1219929 Reviewed-by: Ehsan Karamad <ekaramad@chromium.org> Reviewed-by: Ilya Sherman <isherman@chromium.org> Reviewed-by: Will Harris <wfh@chromium.org> Reviewed-by: Ian Clelland <iclelland@chromium.org> Commit-Queue: Ehsan Karamad <ekaramad@chromium.org> Cr-Commit-Position: refs/heads/master@{#590701} [add] https://crrev.com/920403301bcfbb663e2d7ea85ffb945f8dc5a696/third_party/blink/public/mojom/feature_policy/PRESUBMIT.py
Comment 1 by iclell...@chromium.org
, Sep 6