Navigate between individual search matches in DevTools console

Source/devtools/front_end/ui/UIUtils.js

Index: Source/devtools/front_end/ui/UIUtils.js
diff --git a/Source/devtools/front_end/ui/UIUtils.js b/Source/devtools/front_end/ui/UIUtils.js
index bc5de73f4cbd958523cb4a20db3a3c83d1c88f6b..318e04059950d5275d741a0595ccb73988b266ec 100644
--- a/Source/devtools/front_end/ui/UIUtils.js
+++ b/Source/devtools/front_end/ui/UIUtils.js
@@ -784,11 +784,13 @@ WebInspector.highlightSearchResult = function(element, offset, length, domChange
  * @param {!Element} element
  * @param {!Array.<!WebInspector.SourceRange>} resultRanges
  * @param {!Array.<!Object>=} changes
+ * @param {?Boolean=} isCurrent

[robwu, 2014/10/25 23:13:00]

Booleans are primitive types, use a lowercase "boolean". ? denotes nullable and
{...=} denotes an optional value, you can therefore remove the "=".

(side note, the ! and = at the previous line doesn't make sense either, it says
non-nullable AND optional. That doesn't fit together).

[aknudsen, 2014/10/26 11:22:40]

I made it boolean= instead since the argument is in fact optional. Anyway, the
Closure compiler won't accept ?boolean, since there's an optional argument
before it.

  * @return {!Array.<!Element>}
  */
-WebInspector.highlightSearchResults = function(element, resultRanges, changes)
+WebInspector.highlightSearchResults = function(element, resultRanges, changes, isCurrent)
 {
-    return WebInspector.highlightRangesWithStyleClass(element, resultRanges, "highlighted-search-result", changes);
+    var style = isCurrent ? "current-search-result" : "highlighted-search-result";

[robwu, 2014/10/25 23:13:00]

Rename "style" to className.

[aknudsen, 2014/10/26 11:22:41]

Done.

+    return WebInspector.highlightRangesWithStyleClass(element, resultRanges, style, changes);
 }
 
 /**