[Devtools] JSONView implements Searchable interface

third_party/WebKit/Source/devtools/front_end/network/JSONView.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 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 /**
  * @constructor
  * @extends {WebInspector.VBox}
+ * @implements {WebInspector.Searchable}
  * @param {!WebInspector.ParsedJSON} parsedJSON
  */
 WebInspector.JSONView = function(parsedJSON)
 {
     WebInspector.VBox.call(this);
     this._parsedJSON = parsedJSON;
     this.element.classList.add("json-view");
+
+    /** @type {?WebInspector.SearchableView} */
+    this._searchableView;
+    /** @type {!WebInspector.ObjectPropertiesSection} */
+    this._treeOutline;

[allada, 2016/05/05 23:01:35]

The name did not make sense so I renamed it to match what other's named the
outline object.

+    /** @type {number} */
+    this._currentSearchFocusIndex = 0;
+    /** @type {!Array.<!TreeElement>} */
+    this._currentSearchTreeElements = [];
+    /** @type {?RegExp} */
+    this._searchRegex = null;
 }
 
 /**
+ * @param {!WebInspector.ParsedJSON} parsedJSON
+ * @return {!WebInspector.SearchableView}
+ */
+WebInspector.JSONView.createSearchableView = function(parsedJSON)
+{
+    var jsonView = new WebInspector.JSONView(parsedJSON);
+    var searchableView = new WebInspector.SearchableView(jsonView);
+    searchableView.setPlaceholder(WebInspector.UIString("Find"));
+    jsonView.setSearchableView(searchableView);
+    jsonView.show(searchableView.element);
+    jsonView.element.setAttribute("tabIndex", 0);
+    return searchableView;
+}
+
+/**
  * @param {?string} text
  * @return {!Promise<?WebInspector.ParsedJSON>}
  */
 WebInspector.JSONView.parseJSON = function(text)
 {
     var returnObj = null;
     if (text)
         returnObj = WebInspector.JSONView._extractJSON(/** @type {string} */ (text));
     if (!returnObj)
         return Promise.resolve(/** @type {?WebInspector.ParsedJSON} */ (null));
@@ skipping 51 matching lines @@
 {
     var start = text.indexOf(open);
     var end = text.lastIndexOf(close);
     var length = end - start - 1;
     if (start == -1 || end == -1 || end < start)
         length = -1;
     return {start: start, end: end, length: length};
 }
 
 WebInspector.JSONView.prototype = {
+    /**
+     * @param {?WebInspector.SearchableView} view
+     */
+    setSearchableView: function(view)

[lushnikov, 2016/05/06 18:46:25]

Could this be made private?

[allada, 2016/05/06 19:31:50]

Done. But I did it because I was following how SourcesView handles it.

+    {
+        this._searchableView = view;
+    },
+
     wasShown: function()
     {
         this._initialize();
     },
 
     _initialize: function()
     {
         if (this._initialized)
             return;
         this._initialized = true;
 
         var obj = WebInspector.RemoteObject.fromLocalObject(this._parsedJSON.data);
         var title = this._parsedJSON.prefix + obj.description + this._parsedJSON.suffix;
-        var section = new WebInspector.ObjectPropertiesSection(obj, title);
-        section.setEditable(false);
-        section.expand();
-        this.element.appendChild(section.element);
+        this._treeOutline = new WebInspector.ObjectPropertiesSection(obj, title);
+        this._treeOutline.setEditable(false);
+        this._treeOutline.expand();
+        this.element.appendChild(this._treeOutline.element);
+    },
+
+    /**
+     * @param {number} index
+     */
+    _jumpToMatch: function(index)
+    {
+        if (!this._searchRegex)
+            return;
+        var newFocusElement = this._currentSearchTreeElements[index];
+        var previousFocusElement = this._currentSearchTreeElements[this._currentSearchFocusIndex];
+
+        if (!newFocusElement)
+            index = 0;
+
+        if (previousFocusElement)
+            previousFocusElement.setSearchRegex(this._searchRegex);
+
+        this._updateSearchIndex(index);
+        if (newFocusElement) {
+            newFocusElement.setSearchRegex(this._searchRegex, WebInspector.highlightedCurrentSearchResultClassName);
+            newFocusElement.reveal();
+        }

[lushnikov, 2016/05/06 18:46:25]

I think _jumpToMatch function could be structured a little bit more
straightforward:

if (!this._searchRegex)
    return;

var previousFocusElement = ...;
if (previousFocusElement)
    previousFocusElement.setSearchRegex(..);

var newFocusElement = ...;
if (newFocusElement) {
    newFocusElement.setSearchRegex(..)
    newFocusElement.reveal();
    this._updateSearchIndex(index);
} else {
    this._updateSearchIndex(0);
}

[allada, 2016/05/06 19:31:51]

Done.

+    },
+
+    /**
+     * @param {number} count
+     */
+    _updateSearchCount: function(count)
+    {
+        if (!this._searchableView)
+            return;
+        this._searchableView.updateSearchMatchesCount(count);
+    },
+
+    /**
+     * @param {number} index
+     */
+    _updateSearchIndex: function(index)
+    {
+        this._currentSearchFocusIndex = index;
+        if (!this._searchableView)
+            return;
+        this._searchableView.updateCurrentMatchIndex(index);
+    },
+
+    /**
+     * @override
+     */
+    searchCanceled: function()
+    {
+        this._searchRegex = null;
+        this._currentSearchTreeElements = [];
+
+        for (var element = this._treeOutline.rootElement(); element; element = element.traverseNextTreeElement(false)) {
+            if (!(element instanceof WebInspector.ObjectPropertyTreeElement))
+                continue;
+            element.revertHighlightChanges();
+        }
+        this._updateSearchCount(0);
+        this._updateSearchIndex(0);
+    },
+
+    /**
+     * @override
+     * @param {!WebInspector.SearchableView.SearchConfig} searchConfig
+     * @param {boolean} shouldJump
+     * @param {boolean=} jumpBackwards
+     */
+    performSearch: function(searchConfig, shouldJump, jumpBackwards)
+    {
+        var newIndex = this._currentSearchFocusIndex;
+        var previousSearchFocusElement = this._currentSearchTreeElements[newIndex];
+        this.searchCanceled();
+        this._searchRegex = searchConfig.toSearchRegex(true);
+
+        for (var element = this._treeOutline.rootElement(); element; element = element.traverseNextTreeElement(false)) {
+            if (!(element instanceof WebInspector.ObjectPropertyTreeElement))

[lushnikov, 2016/05/06 18:46:25]

which other type could it be?

[allada, 2016/05/06 19:31:51]

Root node has a special type. Also the
WebInspector.ObjectPropertyTreeElement.populateWithProperties (it's part of the
constructor/initializer stack) function may append:
WebInspector.FunctionScopeMainTreeElement and
WebInspector.CollectionEntriesMainTreeElement.

+                continue;
+            var hasMatch = element.setSearchRegex(this._searchRegex);
+            if (hasMatch)
+                this._currentSearchTreeElements.push(element);
+            if (previousSearchFocusElement === element) {
+                var currentIndex = this._currentSearchTreeElements.length - 1;
+                if (hasMatch || jumpBackwards)
+                    newIndex = currentIndex;
+                else
+                    newIndex = currentIndex + 1;
+            }
+        }
+        this._updateSearchCount(this._currentSearchTreeElements.length);
+
+        if (!this._currentSearchTreeElements.length) {
+            this._updateSearchIndex(0);
+            return;
+        }
+        newIndex = mod(newIndex, this._currentSearchTreeElements.length);
+
+        this._jumpToMatch(newIndex);
+    },
+
+    /**
+     * @override
+     */
+    jumpToNextSearchResult: function()
+    {
+        if (!this._currentSearchTreeElements.length)
+            return;
+        var newIndex = mod(this._currentSearchFocusIndex + 1, this._currentSearchTreeElements.length);
+        this._jumpToMatch(newIndex);
+    },
+
+    /**
+     * @override
+     */
+    jumpToPreviousSearchResult: function()
+    {
+        if (!this._currentSearchTreeElements.length)
+            return;
+        var newIndex = mod(this._currentSearchFocusIndex - 1, this._currentSearchTreeElements.length);
+        this._jumpToMatch(newIndex);
+    },
+
+    /**
+     * @override
+     * @return {boolean}
+     */
+    supportsCaseSensitiveSearch: function()
+    {
+        return true;
+    },
+
+    /**
+     * @override
+     * @return {boolean}
+     */
+    supportsRegexSearch: function()
+    {
+        return true;
     },
 
     __proto__: WebInspector.VBox.prototype
 }
 
 /**
  * @constructor
  * @param {*} data
  * @param {string} prefix
  * @param {string} suffix
  */
 WebInspector.ParsedJSON = function(data, prefix, suffix)
 {
     this.data = data;
     this.prefix = prefix;
     this.suffix = suffix;
 }