1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
|
// This Source Code Form is subject to the terms of the Mozilla Public
// License, v. 2.0. If a copy of the MPL was not distributed with this
// file, You can obtain one at https://mozilla.org/MPL/2.0/.
use inherent::inherent;
use super::{CommonMetricData, MetricId};
use crate::ipc::need_ipc;
/// Developer-facing API for recording URL metrics.
///
/// Instances of this class type are automatically generated by the parsers
/// at build time, allowing developers to record values that were previously
/// registered in the metrics.yaml file.
#[derive(Clone)]
pub enum UrlMetric {
Parent(glean::private::UrlMetric),
Child(UrlMetricIpc),
}
#[derive(Clone, Debug)]
pub struct UrlMetricIpc;
impl UrlMetric {
/// Create a new Url metric.
pub fn new(_id: MetricId, meta: CommonMetricData) -> Self {
if need_ipc() {
UrlMetric::Child(UrlMetricIpc)
} else {
UrlMetric::Parent(glean::private::UrlMetric::new(meta))
}
}
#[cfg(test)]
pub(crate) fn child_metric(&self) -> Self {
match self {
UrlMetric::Parent(_) => UrlMetric::Child(UrlMetricIpc),
UrlMetric::Child(_) => panic!("Can't get a child metric from a child metric"),
}
}
}
#[inherent]
impl glean::traits::Url for UrlMetric {
pub fn set<S: Into<std::string::String>>(&self, value: S) {
match self {
UrlMetric::Parent(p) => p.set(value),
UrlMetric::Child(_) => {
log::error!(
"Unable to set Url metric in non-main process. This operation will be ignored."
);
// If we're in automation we can panic so the instrumentor knows they've gone wrong.
// This is a deliberate violation of Glean's "metric APIs must not throw" design.
assert!(!crate::ipc::is_in_automation(), "Attempted to set URL metric in non-main process, which is forbidden. This panics in automation.");
// TODO: Record an error.
}
};
}
pub fn test_get_value<'a, S: Into<Option<&'a str>>>(
&self,
ping_name: S,
) -> Option<std::string::String> {
let ping_name = ping_name.into().map(|s| s.to_string());
match self {
UrlMetric::Parent(p) => p.test_get_value(ping_name),
UrlMetric::Child(_) => {
panic!("Cannot get test value for Url metric in non-main process!")
}
}
}
pub fn test_get_num_recorded_errors(&self, error: glean::ErrorType) -> i32 {
match self {
UrlMetric::Parent(p) => p.test_get_num_recorded_errors(error),
UrlMetric::Child(_) => panic!(
"Cannot get the number of recorded errors for Url metric in non-main process!"
),
}
}
}
#[cfg(test)]
mod test {
use crate::{common_test::*, ipc, metrics};
#[test]
fn sets_url_value() {
let _lock = lock_test();
let metric = &metrics::test_only_ipc::a_url;
metric.set("https://example.com");
assert_eq!(
"https://example.com",
metric.test_get_value("store1").unwrap()
);
}
#[test]
fn url_ipc() {
// UrlMetric doesn't support IPC.
let _lock = lock_test();
let parent_metric = &metrics::test_only_ipc::a_url;
parent_metric.set("https://example.com/parent");
{
let child_metric = parent_metric.child_metric();
let _raii = ipc::test_set_need_ipc(true);
// Instrumentation calls do not panic.
child_metric.set("https://example.com/child");
// (They also shouldn't do anything,
// but that's not something we can inspect in this test)
}
assert!(ipc::replay_from_buf(&ipc::take_buf().unwrap()).is_ok());
assert!(
"https://example.com/parent" == parent_metric.test_get_value("store1").unwrap(),
"Url metrics should only work in the parent process"
);
}
}
|
