[DevTools] Removed deprecatedCommentWasUsed flag from protocol scriptParsed

third_party/WebKit/Source/devtools/front_end/sdk/Script.js

 /*
  * Copyright (C) 2008 Apple 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 71 matching lines @@
     }
     var sourceURLLineIndex = source.lastIndexOf("\n", sourceURLIndex);
     if (sourceURLLineIndex === -1)
         return source;
     var sourceURLLine = source.substr(sourceURLLineIndex + 1).split("\n", 1)[0];
     if (sourceURLLine.search(WebInspector.Script.sourceURLRegex) === -1)
         return source;
     return source.substr(0, sourceURLLineIndex) + source.substr(sourceURLLineIndex + sourceURLLine.length + 1);
 }
 
+/**
+ * @param {!WebInspector.Script} script
+ * @param {string} source
+ */
+WebInspector.Script._reportDeprecatedCommentIfNeeded = function(script, source)
+{
+    var consoleModel = script.target().consoleModel;
+    if (!consoleModel)
+        return;
+    var linesToCheck = 5;
+    var offset = source.lastIndexOf("\n");
+    while (linesToCheck && offset !== -1)
+        offset = source.lastIndexOf("\n", offset - 1);

[dgozman, 2016/08/12 19:06:04]

linesToCheck--

[kozy, 2016/08/12 19:24:36]

Done.

+    offset = offset !== -1 ? offset : 0;
+    var sourceTail = source.substr(offset);
+    if (sourceTail.search(/^[\040\t]*\/\/@ source(mapping)?url=/mi) === -1)

[dgozman, 2016/08/12 19:06:04]

Let's not do anything if sourceTail is longer than 5000 characters.

[kozy, 2016/08/12 19:24:36]

Done.

+        return;
+    var text = WebInspector.UIString("'//@ sourceURL' and '//@ sourceMappingURL' are deprecated, please use '//# sourceURL=' and '//# sourceMappingURL=' instead.");
+    var msg = new WebInspector.ConsoleMessage(script.target(), WebInspector.ConsoleMessage.MessageSource.JS, WebInspector.ConsoleMessage.MessageLevel.Warning, text, undefined, undefined, undefined, undefined, undefined, undefined, undefined, undefined, undefined, script.scriptId);

[dgozman, 2016/08/12 19:06:04]

nit: can we have lineNumber and columnNumber so that anchor shows the correct
place?

[kozy, 2016/08/12 19:24:36]

I'd like to avoid full scan of documents for new lines.

+    consoleModel.addMessage(msg);
+}
 
 WebInspector.Script.prototype = {
     /**
      * @return {boolean}
      */
     isContentScript: function()
     {
         return this._isContentScript;
     },
 
@@ skipping 55 matching lines @@
         this.target().debuggerAgent().getScriptSource(this.scriptId, didGetScriptSource.bind(this));
         return promise;
 
         /**
          * @this {WebInspector.Script}
          * @param {?Protocol.Error} error
          * @param {string} source
          */
         function didGetScriptSource(error, source)
         {
+            WebInspector.Script._reportDeprecatedCommentIfNeeded(this, source);
             this._source = WebInspector.Script._trimSourceURLComment(error ? "" : source);
             callback(this._source);
         }
     },
 
     /**
      * @override
      * @param {string} query
      * @param {boolean} caseSensitive
      * @param {boolean} isRegex
@@ skipping 140 matching lines @@
             {
                 if (error)
                     console.error(error);
                 fulfill(!error);
             }
         }
     },
 
     __proto__: WebInspector.SDKObject.prototype
 }