Show v8.parseOnBackground in DevTools Timeline

third_party/WebKit/Source/devtools/front_end/components/Linkifier.js

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 538 matching lines @@
  * @param {number=} lineNumber
  * @param {number=} columnNumber
  * @param {string=} classes
  * @param {string=} tooltipText
  * @param {string=} urlDisplayName
  * @return {!Element}
  */
 WebInspector.linkifyResourceAsNode = function(url, lineNumber, columnNumber, classes, tooltipText, urlDisplayName)
 {
     var linkText = urlDisplayName ? urlDisplayName : url ? WebInspector.displayNameForURL(url) : WebInspector.UIString("(program)");
-    if (typeof lineNumber === "number")
+    if (typeof lineNumber === "number" && !isNaN(lineNumber))

[caseq, 2016/03/25 02:00:59]

this looks quite unfortunate, how do we get a NaN here? is this because we
merged switch cases in TimelineUIUtils? If so, I'd rather have separate cases.

[horo, 2016/03/25 04:51:45]

Done.

linkifyLocation() calls linkifier.linkifyScriptLocation with "lineNumber - 1"
which is NaN because lineNumber is undefined.

         linkText += ":" + (lineNumber + 1);
     var anchor = WebInspector.linkifyURLAsNode(url, linkText, classes, false, tooltipText);
     anchor.lineNumber = lineNumber;
     anchor.columnNumber = columnNumber;
     return anchor;
 }
 
 /**
  * @param {!WebInspector.NetworkRequest} request
  * @return {!Element}
  */
 WebInspector.linkifyRequestAsNode = function(request)
 {
     var anchor = WebInspector.linkifyURLAsNode(request.url);
     anchor.requestId = request.requestId;
     return anchor;
 }