Skip to content

Commit f2e0a62

Browse files
committed
fix
1 parent ddb422b commit f2e0a62

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

docs/en/api/browser-protocol.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ Input:
5555
"pagePath": "/index.html",
5656
"fmpTime": 10, // Deprecated since skywalking-client-js 1.0. This is no longer recommended. Please use the `useWebVitals` instead.
5757
/* NOTE, Safari does not support all core web vitals, and Firefox does not support `CLS`. */
58-
"clsTime": 0.1,
58+
"cls": 0.1,
5959
"lcpTime": 10,
6060
}
6161
```

oap-server/server-receiver-plugin/skywalking-browser-receiver-plugin/src/main/java/org/apache/skywalking/oap/server/receiver/browser/provider/parser/performance/listener/BrowserWebVitalsPerfDataAnalysisListener.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,8 @@ public void parse(BrowserWebVitalsPerfDataDecorator decorator) {
4949
browserAppWebVitalsPerf.setServiceName(namingControl.formatServiceName(decorator.getService()));
5050
browserAppWebVitalsPerf.setPath(namingControl.formatEndpointName(browserAppWebVitalsPerf.getServiceName(), decorator.getPagePath()));
5151
browserAppWebVitalsPerf.setFmpTime(decorator.getFmpTime());
52-
// Make CLS multiply 1000 to convert to int from double
53-
// MQE should divide it by 1000 while querying
52+
// CLS values are typically between 0 and 1. Multiplying by 1000 allows storage as an integer
53+
// while preserving 3 decimal places of precision. When querying, divide by 1000 to restore the original value.
5454
browserAppWebVitalsPerf.setCls((int) Math.round(decorator.getCls() * 1000));
5555
browserAppWebVitalsPerf.setLcpTime(decorator.getLcpTime());
5656
}

0 commit comments

Comments
 (0)