Skip to content

Commit b0c6db1

Browse files
committed
fix(benchpress): support tdur in events
1 parent cde8ffd commit b0c6db1

2 files changed

Lines changed: 16 additions & 1 deletion

File tree

modules/benchpress/src/webdriver/chrome_driver_extension.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,11 @@ function normalizeEvent(chromeEvent, data) {
116116
'ts': chromeEvent['ts'] / 1000
117117
};
118118
if (chromeEvent['ph'] === 'X') {
119-
result['dur'] = chromeEvent['dur'] / 1000;
119+
var dur = chromeEvent['dur'];
120+
if (isBlank(dur)) {
121+
dur = chromeEvent['tdur'];
122+
}
123+
result['dur'] = isBlank(dur) ? 0.0 : dur / 1000;
120124
}
121125
StringMapWrapper.forEach(data, (value, prop) => {
122126
result[prop] = value;

modules/benchpress/test/webdriver/chrome_driver_extension_spec.js

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,17 @@ export function main() {
8383
});
8484
});
8585

86+
it('should normalize "tdur" to "dur"', (done) => {
87+
var event = chromeTimelineEvents.create('X', 'FunctionCall', 1100, null);
88+
event['tdur'] = 5500;
89+
createExtension([event]).readPerfLog().then( (events) => {
90+
expect(events).toEqual([
91+
normEvents.complete('script', 1.1, 5.5, null),
92+
]);
93+
done();
94+
});
95+
});
96+
8697
it('should report FunctionCall events as "script"', (done) => {
8798
createExtension([
8899
chromeTimelineEvents.start('FunctionCall', 0)

0 commit comments

Comments
 (0)