2011-09-28 Andrey Kosyakov <caseq@chromium.org>

        Web Inspector: network log view refresh optimizations
        https://bugs.webkit.org/show_bug.cgi?id=69010

        Reviewed by Pavel Feldman.

        * inspector/front-end/NetworkPanel.js:
        (WebInspector.NetworkLogView.prototype.refresh):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@96318 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index eb9249f..3afe219 100644 (file)
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,13 @@
+2011-09-28  Andrey Kosyakov  <caseq@chromium.org>
+
+        Web Inspector: network log view refresh optimizations
+        https://bugs.webkit.org/show_bug.cgi?id=69010
+
+        Reviewed by Pavel Feldman.
+
+        * inspector/front-end/NetworkPanel.js:
+        (WebInspector.NetworkLogView.prototype.refresh):
+
 2011-09-29  Pavel Podivilov  <podivilov@chromium.org>
 
         Web Inspector: split SourceFile.js into RawSourceCode.js and UISourceCodeContentProviders.js.

diff --git a/Source/WebCore/inspector/front-end/NetworkPanel.js b/Source/WebCore/inspector/front-end/NetworkPanel.js
index e9dec7f..5554620 100644 (file)
--- a/Source/WebCore/inspector/front-end/NetworkPanel.js
+++ b/Source/WebCore/inspector/front-end/NetworkPanel.js
@@ -696,9 +696,8 @@ WebInspector.NetworkLogView.prototype = {
         this._staleResources = [];
         this._sortItems();
         this._updateSummaryBar();
-        this._updateOffscreenRows();
         this._dataGrid.updateWidths();
-
+        // FIXME: evaluate performance impact of moving this before a call to sortItems()
         if (wasScrolledToLastRow)
             this._dataGrid.scrollToLastRow();
     },
@@ -1814,6 +1813,8 @@ WebInspector.NetworkDataGridNode = function(parentView, resource)
 WebInspector.NetworkDataGridNode.prototype = {
     createCells: function()
     {
+        // Out of sight, out of mind: create nodes offscreen to save on render tree update times when running updateOffscreenRows()
+        this._element.addStyleClass("offscreen");
         this._nameCell = this._createDivInTD("name");
         this._methodCell = this._createDivInTD("method");
         this._statusCell = this._createDivInTD("status");