Chromium Code Reviews| Index: tools/metrics/histograms/enums.xml |
| diff --git a/tools/metrics/histograms/enums.xml b/tools/metrics/histograms/enums.xml |
| index fda1ce096b0d23fe10823453f76d7f27f851b9e5..5f2d3d8db9ce8c82cd2aa69911dcce5cd7b1e03c 100644 |
| --- a/tools/metrics/histograms/enums.xml |
| +++ b/tools/metrics/histograms/enums.xml |
| @@ -7053,6 +7053,21 @@ uploading your change for review. These are checked by presubmit scripts. |
| <int value="3" label="Enabled from second screen"/> |
| </enum> |
| +<enum name="DataReductionProxyProtocolAcceptTransformEvent"> |
| + <int value="0" label="AcceptTransform lite-page sent"/> |
| + <int value="1" label="ContentTransform lite-page received"/> |
| + <int value="2" label="Page policy for empty-image received"/> |
| + <int value="3" label="AcceptTransform empty-image sent"/> |
| + <int value="4" label="ContentTransform empty-image received"/> |
| + <int value="5" label="AcceptTransform compressed-video sent"/> |
|
bengr
2017/06/26 18:11:47
Is there a corresponding CT value for this one?
dougarnett
2017/06/26 21:19:16
No, Flywheel will respond with a 302 redirect to t
|
| +</enum> |
| + |
| +<enum name="DataReductionProxyProtocolNotAcceptingTransformReason"> |
| + <int value="0" label="Not accepting proxy transforms, transforms disabled"/> |
|
bengr
2017/06/26 18:11:48
Disabled by what?
dougarnett
2017/06/26 21:19:16
Eventually, either by finch or user flag disabling
|
| + <int value="1" label="Not accepting proxy transforms, page blacklisted"/> |
|
bengr
2017/06/26 18:11:48
By client side blacklist?
dougarnett
2017/06/26 21:19:16
Done.
|
| + <int value="2" label="Not accepting proxy transforms, cellular-only mode"/> |
|
bengr
2017/06/26 18:11:48
And the network is not cellular? Be explicit.
dougarnett
2017/06/26 21:19:16
Done.
|
| +</enum> |
| + |
| <enum name="DataReductionProxyProxyPrefMigrationResult"> |
| <int value="0" label="Proxy pref not cleared"/> |
| <int value="1" label="Empty proxy pref was cleared"/> |