DevTools: similarly to the way we match scripts ignoring sourceurl comments, ignore framework-speci…

third_party/WebKit/Source/devtools/front_end/bindings/ResourceScriptMapping.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 302 matching lines @@
      * @return {boolean}
      */
     _isDiverged: function()
     {
         if (this._uiSourceCode.isDirty())
             return true;
         if (!this._script)
             return false;
         if (typeof this._scriptSource === "undefined")
             return false;
-        if (!this._uiSourceCode.workingCopy().startsWith(this._scriptSource.trimRight()))
-            return true;
-        var suffix = this._uiSourceCode.workingCopy().substr(this._scriptSource.length);
-        return !!suffix.length && !suffix.match(WebInspector.Script.sourceURLRegex);
+        var workingCopy = this._uiSourceCode.workingCopy();
+
+        // Match ignoring sourceURL.
+        if (workingCopy.startsWith(this._scriptSource.trimRight())) {
+            var suffix = this._uiSourceCode.workingCopy().substr(this._scriptSource.length);

[dgozman, 2016/06/20 16:17:42]

Should we strip |this._scriptSource.trimRight().length| symbols to be consistent
with previous line?

[pfeldman, 2016/06/21 08:50:46]

They produce 'opposite' checks, so i'd rather not.

+            return !!suffix.length && !suffix.match(WebInspector.Script.sourceURLRegex);
+        }
+
+        // Match ignoring Node wrapper.
+        var nodePrefix = "(function (exports, require, module, __filename, __dirname) { ";
+        var nodeSuffix = "\n});";
+        if (workingCopy == nodePrefix + this._scriptSource + nodeSuffix)

[dgozman, 2016/06/20 16:17:42]

===

[pfeldman, 2016/06/21 08:50:46]

Done.

+            return false;
+
+        return true;
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _workingCopyChanged: function(event)
     {
         this._update();
     },
 
@@ skipping 135 matching lines @@
     /**
      * @return {boolean}
      */
     hasSourceMapURL: function()
     {
         return this._script && !!this._script.sourceMapURL;
     },
 
     __proto__: WebInspector.Object.prototype
 }