[DevTools] Fixed inline scope variables for object property

third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js

 /*
  * Copyright (C) 2011 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 694 matching lines @@
             this._clearValueWidgets();
             return;
         }
 
         var valuesMap = new Map();
         for (var property of properties)
             valuesMap.set(property.name, property.value);
 
         /** @type {!Map.<number, !Set<string>>} */
         var namesPerLine = new Map();
+        var skipNextToken = false;

[lushnikov, 2016/10/04 23:31:31]

let's name this skipObjectProperty so that reader would have at least some idea
why
we do this

[kozy, 2016/10/05 00:10:22]

Done.

         var tokenizer = new WebInspector.CodeMirrorUtils.TokenizerFactory().createTokenizer("text/javascript");
         tokenizer(this.textEditor.line(fromLine).substring(fromColumn), processToken.bind(this, fromLine));
         for (var i = fromLine + 1; i < toLine; ++i)
             tokenizer(this.textEditor.line(i), processToken.bind(this, i));
 
         /**
          * @param {number} lineNumber
          * @param {string} tokenValue
          * @param {?string} tokenType
          * @param {number} column
          * @param {number} newColumn
          * @this {WebInspector.JavaScriptSourceFrame}
          */
         function processToken(lineNumber, tokenValue, tokenType, column, newColumn)
         {
-            if (tokenType && this._isIdentifier(tokenType) && valuesMap.get(tokenValue)) {
+            if (!skipNextToken && tokenType && this._isIdentifier(tokenType) && valuesMap.get(tokenValue)) {
                 var names = namesPerLine.get(lineNumber);
                 if (!names) {
                     names = new Set();
                     namesPerLine.set(lineNumber, names);
                 }
                 names.add(tokenValue);
             }
+            skipNextToken = tokenValue === ".";
         }
         this.textEditor.operation(this._renderDecorations.bind(this, valuesMap, namesPerLine, fromLine, toLine));
     },
 
     /**
      * @param {!Map.<string,!WebInspector.RemoteObject>} valuesMap
      * @param {!Map.<number, !Set<string>>} namesPerLine
      * @param {number} fromLine
      * @param {number} toLine
      */
@@ skipping 319 matching lines @@
         this.uiSourceCode().removeEventListener(WebInspector.UISourceCode.Events.WorkingCopyChanged, this._workingCopyChanged, this);
         this.uiSourceCode().removeEventListener(WebInspector.UISourceCode.Events.WorkingCopyCommitted, this._workingCopyCommitted, this);
         this.uiSourceCode().removeEventListener(WebInspector.UISourceCode.Events.TitleChanged, this._showBlackboxInfobarIfNeeded, this);
         WebInspector.moduleSetting("skipStackFramesPattern").removeChangeListener(this._showBlackboxInfobarIfNeeded, this);
         WebInspector.moduleSetting("skipContentScripts").removeChangeListener(this._showBlackboxInfobarIfNeeded, this);
         WebInspector.UISourceCodeFrame.prototype.dispose.call(this);
     },
 
     __proto__: WebInspector.UISourceCodeFrame.prototype
 }