DevTools: speed up network scrolling and layout.

DevTools: speed up network scrolling and layout.
(still needs a lot of work)

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170024

[alph, 2014-03-25 22:07:24]

lgtm

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
File Source/devtools/front_end/NetworkPanel.js (right):

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
Source/devtools/front_end/NetworkPanel.js:2366: // Out of sight, out of mind:
create nodes offscreen to save on render tree update times when running
updateOffscreenRows()
Nuke it

[eustas, 2014-03-26 04:46:48]

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
File Source/devtools/front_end/DataGrid.js (right):

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
Source/devtools/front_end/DataGrid.js:720: for (var i = 0; i < numColumns - 1;
i++) {
Though this avoids recalc-style waterfall, it doesn't make a huge performance
boost when thousands requests are shown.

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
File Source/devtools/front_end/NetworkPanel.js (right):

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
Source/devtools/front_end/NetworkPanel.js:1150: _updateRows: function()
This method could be eliminated totally by adding gradient stripes back.

[pfeldman, 2014-03-26 05:00:56]

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
File Source/devtools/front_end/DataGrid.js (right):

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
Source/devtools/front_end/DataGrid.js:720: for (var i = 0; i < numColumns - 1;
i++) {
It prevents a dozen of full layouts from happening. Layout time is proportional
to number of rows. How is that not huge?

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
File Source/devtools/front_end/NetworkPanel.js (right):

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
Source/devtools/front_end/NetworkPanel.js:1150: _updateRows: function()
I don't think so - it is responsible for filtering rows as well.

https://codereview.chromium.org/207553010/diff/40001/Source/devtools/front_en...
Source/devtools/front_end/NetworkPanel.js:2366: // Out of sight, out of mind:
create nodes offscreen to save on render tree update times when running
updateOffscreenRows()
On 2014/03/25 22:07:24, alph wrote:
> Nuke it

Done.

[pfeldman, 2014-03-26 05:29:46]

The CQ bit was checked by pfeldman@chromium.org

[commit-bot: I haz the power, 2014-03-26 05:29:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pfeldman@chromium.org/207553010/130001

[commit-bot: I haz the power, 2014-03-26 06:34:17]

Change committed as 170024