Commit 3774cf17 authored by Reilly Grant's avatar Reilly Grant Committed by Commit Bot

[RAPPOR] Remove redundant DeviceSensors metrics

The DeviceSensors.DeviceMotion and DeviceSensors.DeviceOrientation
RAPPOR metrics can be replaced by the existing UKM-enabled Blink use
counters for this feature.

Bug: 1016906
Change-Id: I50cf7ac9ed26ac9b57f47cb8f74b74bb5f133612
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898297
Auto-Submit: Reilly Grant <reillyg@chromium.org>
Commit-Queue: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: default avatarRobert Kaplow <rkaplow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#713214}
parent a72d6b89
......@@ -62,9 +62,6 @@ void DeviceMotionController::DidAddEventListener(
UseCounter::Count(GetDocument(), WebFeature::kDeviceMotionSecureOrigin);
if (!has_event_listener_) {
Platform::Current()->RecordRapporURL("DeviceSensors.DeviceMotion",
WebURL(GetDocument().Url()));
if (!IsSameSecurityOriginAsMainFrame()) {
Platform::Current()->RecordRapporURL(
"DeviceSensors.DeviceMotionCrossOrigin", WebURL(GetDocument().Url()));
......
......@@ -72,9 +72,6 @@ void DeviceOrientationController::DidAddEventListener(
UseCounter::Count(GetDocument(), WebFeature::kDeviceOrientationSecureOrigin);
if (!has_event_listener_) {
Platform::Current()->RecordRapporURL("DeviceSensors.DeviceOrientation",
WebURL(GetDocument().Url()));
if (!IsSameSecurityOriginAsMainFrame()) {
Platform::Current()->RecordRapporURL(
"DeviceSensors.DeviceOrientationCrossOrigin",
......
......@@ -137,13 +137,6 @@ components/rappor/rappor_parameters.h.
</summary>
</rappor-metric>
<rappor-metric name="DeviceSensors.DeviceMotion" type="ETLD_PLUS_ONE">
<owner>timvolodine@chromium.org</owner>
<summary>
The domain and registry of the URL that is using Device Motion API.
</summary>
</rappor-metric>
<rappor-metric name="DeviceSensors.DeviceMotionCrossOrigin"
type="ETLD_PLUS_ONE">
<owner>timvolodine@chromium.org</owner>
......@@ -153,13 +146,6 @@ components/rappor/rappor_parameters.h.
</summary>
</rappor-metric>
<rappor-metric name="DeviceSensors.DeviceOrientation" type="ETLD_PLUS_ONE">
<owner>timvolodine@chromium.org</owner>
<summary>
The domain and registry of the URL that is using Device Orientation API.
</summary>
</rappor-metric>
<rappor-metric name="DeviceSensors.DeviceOrientationCrossOrigin"
type="ETLD_PLUS_ONE">
<owner>timvolodine@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