Merge 97975 - Web Inspector: Advanced search results should keep working after pretty print toggled.

Source/WebCore/inspector/front-end/AdvancedSearchController.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:
  *
  * 1. Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *
@@ skipping 278 matching lines @@
 
 WebInspector.SearchScope.prototype = {
     /**
      * @param {WebInspector.SearchConfig} searchConfig
      */
     performSearch: function(searchConfig, searchResultCallback, searchFinishedCallback) { },
 
     stopSearch: function() { },
     
     /**
+     * @param {WebInspector.SearchConfig} searchConfig
      * @return WebInspector.SearchResultsPane}
      */
-    createSearchResultsPane: function() { }
+    createSearchResultsPane: function(searchConfig) { }
 }
 
 /**
  * @constructor
  * @param {WebInspector.SearchConfig} searchConfig
  */
 WebInspector.SearchResultsPane = function(searchConfig)
 {
     this._searchConfig = searchConfig;
     this.element = document.createElement("div");
@@ skipping 30 matching lines @@
     this._treeOutlineElement = document.createElement("ol");
     this._treeOutlineElement.className = "outline-disclosure";
     this.element.appendChild(this._treeOutlineElement);
     this._treeOutline = new TreeOutline(this._treeOutlineElement);
 }
 
 WebInspector.FileBasedSearchResultsPane.prototype = {
     /**
      * @param {Object} file
      * @param {number} lineNumber
+     * @param {number} columnNumber
      * @return {Element}
      */
-    createAnchor: function(file, lineNumber) { },
+    createAnchor: function(file, lineNumber, columnNumber) { },
     
     /**
      * @param {Object} file
      * @return {string}
      */
     fileName: function(file) { },
     
     /**
      * @param {Object} searchResult
      */
     addSearchResult: function(searchResult)
     {
         this._searchResults.push(searchResult);
         var file = searchResult.file;
         var fileName = this.fileName(file);
         var searchMatches = searchResult.searchMatches;
             
         // Expand first file with matches only.
         var fileTreeElement = this._addFileTreeElement(fileName, searchMatches.length, this._searchResults.length === 1);
         
-        var regexObject = createSearchRegex(this._searchConfig.query, !this._searchConfig.ignoreCase, this._searchConfig.isRegex);
+        var regex = createSearchRegex(this._searchConfig.query, !this._searchConfig.ignoreCase, this._searchConfig.isRegex);
         for (var i = 0; i < searchMatches.length; i++) {
             var lineNumber = searchMatches[i].lineNumber;
+            var lineContent = searchMatches[i].lineContent;
+            var matchRanges = this._regexMatchRanges(lineContent, regex);
             
-            var anchor = this.createAnchor(file, lineNumber);
+            var anchor = this.createAnchor(file, lineNumber, matchRanges[0].offset);
             
             var numberString = numberToStringWithSpacesPadding(lineNumber + 1, 4);
             var lineNumberSpan = document.createElement("span");
             lineNumberSpan.addStyleClass("webkit-line-number");
             lineNumberSpan.addStyleClass("search-match-line-number");
             lineNumberSpan.textContent = numberString;
             anchor.appendChild(lineNumberSpan);
             
-            var contentSpan = this._createContentSpan(searchMatches[i].lineContent, regexObject);
+            var contentSpan = this._createContentSpan(lineContent, matchRanges);
             anchor.appendChild(contentSpan);
             
             var searchMatchElement = new TreeElement("", null, false);
             fileTreeElement.appendChild(searchMatchElement);
             searchMatchElement.listItemElement.className = "search-match";
             searchMatchElement.listItemElement.appendChild(anchor);
         }
     },
         
     /**
@@ skipping 23 matching lines @@
         else
             matchesCountSpan.textContent = WebInspector.UIString("(%d matches)", searchMatchesCount);
         
         fileTreeElement.listItemElement.appendChild(matchesCountSpan);
         
         return fileTreeElement; 
     },
 
     /**
      * @param {string} lineContent
-     * @param {RegExp} regexObject
+     * @param {RegExp} regex
+     * @return {Array.<Object>}
      */
-    _createContentSpan: function(lineContent, regexObject)
+    _regexMatchRanges: function(lineContent, regex)
+    {
+        regex.lastIndex = 0;
+        var match;
+        var offset = 0;
+        var matchRanges = [];
+        while (match = regex.exec(lineContent))
+            matchRanges.push({ offset: match.index, length: match[0].length });
+        
+        return matchRanges;
+    },
+    
+    /**
+     * @param {string} lineContent
+     * @param {Array.<Object>} matchRanges
+     */
+    _createContentSpan: function(lineContent, matchRanges)
     {
         var contentSpan = document.createElement("span");
         contentSpan.className = "search-match-content";
         contentSpan.textContent = lineContent;
-        
-        regexObject.lastIndex = 0;
-        var match = regexObject.exec(lineContent);
-        var offset = 0;
-        var matchRanges = [];
-        while (match) {
-            matchRanges.push({ offset: match.index, length: match[0].length });
-            match = regexObject.exec(lineContent);
-        }
         highlightRangesWithStyleClass(contentSpan, matchRanges, "highlighted-match");
-        
         return contentSpan;
     }
 }
 
 WebInspector.FileBasedSearchResultsPane.prototype.__proto__ = WebInspector.SearchResultsPane.prototype;
 
 /**
  * @constructor
  * @param {Object} file
  * @param {Array.<Object>} searchMatches
  */
 WebInspector.FileBasedSearchResultsPane.SearchResult = function(file, searchMatches) {
     this.file = file;
     this.searchMatches = searchMatches;
 }