Commit 5251bd61 authored by Tim Volodine's avatar Tim Volodine Committed by Commit Bot

[RAPPOR] Remove deprecated cross origin sensor metrics

Remove cross origin sensor metrics, due to RAPPOR framework
being turned down.

Cross origin metrics measure the Device Motion/Orientation
API usage across frames. These metrics will be added using
the new UKM framework as a follow-up if necessary.

BUG=1025720

Change-Id: Ie9506fcdc9d0b04088e49af91f7a5a7b093a881b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2062332Reviewed-by: default avatarRobert Kaplow <rkaplow@chromium.org>
Commit-Queue: Tim Volodine <timvolodine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743084}
parent 476bf35d
......@@ -62,11 +62,6 @@ void DeviceMotionController::DidAddEventListener(
UseCounter::Count(GetDocument(), WebFeature::kDeviceMotionSecureOrigin);
if (!has_event_listener_) {
if (!IsSameSecurityOriginAsMainFrame()) {
Platform::Current()->RecordRapporURL(
"DeviceSensors.DeviceMotionCrossOrigin", WebURL(GetDocument().Url()));
}
if (!CheckPolicyFeatures(
{mojom::blink::FeaturePolicyFeature::kAccelerometer,
mojom::blink::FeaturePolicyFeature::kGyroscope})) {
......
......@@ -73,12 +73,6 @@ void DeviceOrientationController::DidAddEventListener(
UseCounter::Count(GetDocument(), WebFeature::kDeviceOrientationSecureOrigin);
if (!has_event_listener_) {
if (!IsSameSecurityOriginAsMainFrame()) {
Platform::Current()->RecordRapporURL(
"DeviceSensors.DeviceOrientationCrossOrigin",
WebURL(GetDocument().Url()));
}
if (!CheckPolicyFeatures(
{mojom::blink::FeaturePolicyFeature::kAccelerometer,
mojom::blink::FeaturePolicyFeature::kGyroscope})) {
......
......@@ -119,24 +119,6 @@ components/rappor/rappor_parameters.h.
</summary>
</rappor-metric>
<rappor-metric name="DeviceSensors.DeviceMotionCrossOrigin"
type="ETLD_PLUS_ONE">
<owner>timvolodine@chromium.org</owner>
<summary>
The domain and registry of the URL that is using Device Motion API from a
frame with an origin different to that of the main frame.
</summary>
</rappor-metric>
<rappor-metric name="DeviceSensors.DeviceOrientationCrossOrigin"
type="ETLD_PLUS_ONE">
<owner>timvolodine@chromium.org</owner>
<summary>
The domain and registry of the URL that is using Device Orientation API from
a frame with an origin different to that of the main frame.
</summary>
</rappor-metric>
<rappor-metric name="NTP.ExplicitUserAction.PageNavigation.NTPTileClick"
type="UMA_RAPPOR_TYPE">
<owner>knn@chromium.org</owner>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment