diff --git a/src/profile-logic/import/chrome.js b/src/profile-logic/import/chrome.js index f9bcc6e735..92b874ae31 100644 --- a/src/profile-logic/import/chrome.js +++ b/src/profile-logic/import/chrome.js @@ -571,9 +571,20 @@ async function processTracingEvents( if (lineNumber === -1) { lineNumber = undefined; } + if (columnNumber === -1) { columnNumber = undefined; } + + // Line and column number are zero-based in chrome profiles, but + // 1-based in the firefox profiler. + if (lineNumber !== undefined) { + lineNumber++; + } + if (columnNumber !== undefined) { + columnNumber++; + } + if (url === '') { url = undefined; } diff --git a/src/test/unit/__snapshots__/profile-conversion.test.js.snap b/src/test/unit/__snapshots__/profile-conversion.test.js.snap index 942d87f59a..9b27ebba3a 100644 --- a/src/test/unit/__snapshots__/profile-conversion.test.js.snap +++ b/src/test/unit/__snapshots__/profile-conversion.test.js.snap @@ -402777,14 +402777,14 @@ Object { ], "column": Array [ null, - 1757, + 1758, null, null, - 1363, - 1783, - 3420, - 463, - 1783, + 1364, + 1784, + 3421, + 464, + 1784, null, null, null, @@ -402793,14 +402793,14 @@ Object { null, null, null, - 4543, - 1326, + 4544, + 1327, null, - 29, + 30, null, - 5197, - 5375, - 296, + 5198, + 5376, + 297, null, ], "func": Array [ @@ -402891,14 +402891,14 @@ Object { "length": 26, "line": Array [ null, - 1, + 2, null, null, - 1, - 25, - 25, - 1, - 25, + 2, + 26, + 26, + 2, + 26, null, null, null, @@ -402907,14 +402907,14 @@ Object { null, null, null, - 25, - 28, + 26, + 29, null, - 1, + 2, null, - 25, - 25, - 1, + 26, + 26, + 2, null, ], "nativeSymbol": Array [ @@ -403005,13 +403005,13 @@ Object { "funcTable": Object { "columnNumber": Array [ null, - 1757, + 1758, null, null, - 1363, - 1783, - 3420, - 463, + 1364, + 1784, + 3421, + 464, null, null, null, @@ -403020,14 +403020,14 @@ Object { null, null, null, - 4543, - 1326, + 4544, + 1327, null, - 29, + 30, null, - 5197, - 5375, - 296, + 5198, + 5376, + 297, null, ], "fileName": Array [ @@ -403087,13 +403087,13 @@ Object { "length": 25, "lineNumber": Array [ null, - 1, + 2, null, null, - 1, - 25, - 25, - 1, + 2, + 26, + 26, + 2, null, null, null, @@ -403102,14 +403102,14 @@ Object { null, null, null, - 25, - 28, + 26, + 29, null, - 1, + 2, null, - 25, - 25, - 1, + 26, + 26, + 2, null, ], "name": Array [ @@ -444269,50 +444269,50 @@ Object { "column": Array [ null, null, - 32, + 33, null, null, null, null, null, - 22, - 22, - 16, + 23, + 23, + 17, null, - 34, - 22, - 20, - 20, - 26, - 52, - 29, + 35, + 23, + 21, + 21, + 27, + 53, + 30, null, null, null, + 26, 25, - 24, null, - 37, - 21, - 32, + 38, + 22, + 33, null, null, null, - 16, + 17, null, null, - 16, - 22, - 20, - 29, - 18, + 17, + 23, + 21, 30, - 18, - 74, - 24, + 19, + 31, + 19, + 75, + 25, null, - 30, - 30, + 31, + 31, null, ], "func": Array [ @@ -444467,50 +444467,50 @@ Object { "line": Array [ null, null, - 3338, + 3339, null, null, null, null, null, - 75674, - 75624, - 65, + 75675, + 75625, + 66, null, - 70, - 73232, - 73084, - 72908, - 74574, - 74061, - 74281, + 71, + 73233, + 73085, + 72909, + 74575, + 74062, + 74282, null, null, null, - 27197, - 27498, + 27198, + 27499, null, - 26821, - 27773, - 28117, + 26822, + 27774, + 28118, null, null, null, - 137, + 138, null, null, - 32, - 73232, - 73084, - 74281, - 74613, - 13073, - 79762, - 74085, - 73822, + 33, + 73233, + 73085, + 74282, + 74614, + 13074, + 79763, + 74086, + 73823, null, - 13073, - 13382, + 13074, + 13383, null, ], "nativeSymbol": Array [ @@ -444665,45 +444665,45 @@ Object { "columnNumber": Array [ null, null, - 32, + 33, null, null, null, null, - 22, - 22, - 16, + 23, + 23, + 17, null, - 34, - 22, - 20, - 20, - 26, - 52, - 29, + 35, + 23, + 21, + 21, + 27, + 53, + 30, null, null, null, null, + 26, 25, - 24, null, - 37, - 21, - 32, + 38, + 22, + 33, null, null, null, - 16, + 17, null, null, - 16, - 18, - 30, - 18, - 74, - 24, - 30, + 17, + 19, + 31, + 19, + 75, + 25, + 31, ], "fileName": Array [ null, @@ -444795,45 +444795,45 @@ Object { "lineNumber": Array [ null, null, - 3338, + 3339, null, null, null, null, - 75674, - 75624, - 65, + 75675, + 75625, + 66, null, - 70, - 73232, - 73084, - 72908, - 74574, - 74061, - 74281, + 71, + 73233, + 73085, + 72909, + 74575, + 74062, + 74282, null, null, null, null, - 27197, - 27498, + 27198, + 27499, null, - 26821, - 27773, - 28117, + 26822, + 27774, + 28118, null, null, null, - 137, + 138, null, null, - 32, - 74613, - 13073, - 79762, - 74085, - 73822, - 13382, + 33, + 74614, + 13074, + 79763, + 74086, + 73823, + 13383, ], "name": Array [ 0, @@ -447302,50 +447302,50 @@ Object { "column": Array [ null, null, - 32, + 33, null, null, null, null, null, - 22, - 22, - 16, + 23, + 23, + 17, null, - 34, - 22, - 20, - 20, - 26, - 52, - 29, + 35, + 23, + 21, + 21, + 27, + 53, + 30, null, null, null, + 26, 25, - 24, null, - 37, - 21, - 32, + 38, + 22, + 33, null, null, null, - 16, + 17, null, null, - 16, - 22, - 20, - 29, - 18, + 17, + 23, + 21, 30, - 18, - 74, - 24, + 19, + 31, + 19, + 75, + 25, null, - 30, - 30, + 31, + 31, null, ], "func": Array [ @@ -447500,50 +447500,50 @@ Object { "line": Array [ null, null, - 3338, + 3339, null, null, null, null, null, - 75674, - 75624, - 65, + 75675, + 75625, + 66, null, - 70, - 73232, - 73084, - 72908, - 74574, - 74061, - 74281, + 71, + 73233, + 73085, + 72909, + 74575, + 74062, + 74282, null, null, null, - 27197, - 27498, + 27198, + 27499, null, - 26821, - 27773, - 28117, + 26822, + 27774, + 28118, null, null, null, - 137, + 138, null, null, - 32, - 73232, - 73084, - 74281, - 74613, - 13073, - 79762, - 74085, - 73822, + 33, + 73233, + 73085, + 74282, + 74614, + 13074, + 79763, + 74086, + 73823, null, - 13073, - 13382, + 13074, + 13383, null, ], "nativeSymbol": Array [ @@ -447698,45 +447698,45 @@ Object { "columnNumber": Array [ null, null, - 32, + 33, null, null, null, null, - 22, - 22, - 16, + 23, + 23, + 17, null, - 34, - 22, - 20, - 20, - 26, - 52, - 29, + 35, + 23, + 21, + 21, + 27, + 53, + 30, null, null, null, null, + 26, 25, - 24, null, - 37, - 21, - 32, + 38, + 22, + 33, null, null, null, - 16, + 17, null, null, - 16, - 18, - 30, - 18, - 74, - 24, - 30, + 17, + 19, + 31, + 19, + 75, + 25, + 31, ], "fileName": Array [ null, @@ -447828,45 +447828,45 @@ Object { "lineNumber": Array [ null, null, - 3338, + 3339, null, null, null, null, - 75674, - 75624, - 65, + 75675, + 75625, + 66, null, - 70, - 73232, - 73084, - 72908, - 74574, - 74061, - 74281, + 71, + 73233, + 73085, + 72909, + 74575, + 74062, + 74282, null, null, null, null, - 27197, - 27498, + 27198, + 27499, null, - 26821, - 27773, - 28117, + 26822, + 27774, + 28118, null, null, null, - 137, + 138, null, null, - 32, - 74613, - 13073, - 79762, - 74085, - 73822, - 13382, + 33, + 74614, + 13074, + 79763, + 74086, + 73823, + 13383, ], "name": Array [ 0,