Bug 1815460 - [devtools] Netmonitor waterfall sort should fallback to…

… resource ids r=ochameau,bomsy,devtools-reviewers

resource ids are channel ids and should be incrementally generated. The current id is actually an actor id, meaning it is a string which will lead to incorrect comparisons (eg "Actor10" will be considered "lower" than "Actor2") and also fully depends on the order in which we create the actors.

On some platform / configurations we get identical timestamps for batches of requests, so the fallback mechanism should be reliable

Differential Revision: https://phabricator.services.mozilla.com/D169361

devtools/client/netmonitor/src/utils/sort-predicates.js

@@ -41,7 +41,7 @@ function compareValues(first, second) {
 
 function waterfall(first, second) {
   const result = compareValues(first.startedMs, second.startedMs);
-  return result || compareValues(first.id, second.id);
+  return result || compareValues(first.resourceId, second.resourceId);
 }
 
 function priority(first, second) {