Fix the merging of threadnames with multiple digits on about:profiler.

chrome/browser/resources/profiler.js

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 var g_browserBridge;
 var g_mainView;
 
 // TODO(eroman): The handling of "max" across snapshots is not correct.
 // For starters the browser needs to be aware to generate new maximums.
 // Secondly, we need to take into account the "max" of intermediary snapshots,
@@ skipping 860 matching lines @@
     // Define a translation function for each property. Normally we copy over
     // properties as-is, but if we have been asked to "merge similar threads" we
     // we will remap the thread names that end in a numeric suffix.
     var propertyGetterFunc;
 
     if (mergeSimilarThreads) {
       propertyGetterFunc = function(row, key) {
         var value = row[key];
         // If the property is a thread name, try to remap it.
         if (key == KEY_BIRTH_THREAD || key == KEY_DEATH_THREAD) {
-          var m = /^(.*)(\d+)$/.exec(value);
+          var m = /^(.*)[^\d](\d+)$/.exec(value);

[jar (doing other things), 2011/11/23 16:45:48]

I think you discard the last character of the non-numeric prefix.  I guess you
might assume it is a separator, like a "-" or a space.... but if you processed:
Thread23

you'd end up with
Threa*

(after you concatenation in line 883).

I think it might be better to say:

var m = /^(.*[^\d])(\d+)$/.exec(value);

With the paren pushed after the non-digit single-match.

           if (m)
             value = m[1] + '*';
         }
         return value;
       }
     } else {
       propertyGetterFunc = function(row, key) { return row[key]; };
     }
 
     // Determine which sets of properties a row needs to match on to be
@@ skipping 1161 matching lines @@
           groupKey.push(entry);
         }
 
         return JSON.stringify(groupKey);
       };
     },
   };
 
   return MainView;
 })();