Always use global indexes for nodes and edges in the heap snapshot representation

Always use global indexes for nodes and edges in the heap snapshot representation

- HeapSnapshotNode, HeapSnapshotEdge and HeapSnapshotRetainerEdge now store only reference to HeapSnapshot and global index of corresponding item.
- all index boundaries were moved out of these classes into corresponding iterators

BUG=None

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

[alph, 2014-03-21 08:30:39]

lgtm

https://codereview.chromium.org/207523004/diff/1/LayoutTests/inspector/profil...
File LayoutTests/inspector/profiler/heap-snapshot.html (right):

https://codereview.chromium.org/207523004/diff/1/LayoutTests/inspector/profil...
LayoutTests/inspector/profiler/heap-snapshot.html:69: for (;
edgeIterator.hasNext(); edgeIterator.next()) {
s/[{}]//

https://codereview.chromium.org/207523004/diff/1/Source/devtools/front_end/He...
File Source/devtools/front_end/HeapSnapshot.js (right):

https://codereview.chromium.org/207523004/diff/1/Source/devtools/front_end/He...
Source/devtools/front_end/HeapSnapshot.js:360: var retainedNodeOrdinal =
retainedNode.nodeIndex / snapshot._nodeFieldCount;
_ordinal() ?

[yurys, 2014-03-21 08:33:20]

https://codereview.chromium.org/207523004/diff/1/LayoutTests/inspector/profil...
File LayoutTests/inspector/profiler/heap-snapshot.html (right):

https://codereview.chromium.org/207523004/diff/1/LayoutTests/inspector/profil...
LayoutTests/inspector/profiler/heap-snapshot.html:69: for (;
edgeIterator.hasNext(); edgeIterator.next()) {
On 2014/03/21 08:30:39, alph wrote:
> s/[{}]//

Done.

https://codereview.chromium.org/207523004/diff/1/Source/devtools/front_end/He...
File Source/devtools/front_end/HeapSnapshot.js (right):

https://codereview.chromium.org/207523004/diff/1/Source/devtools/front_end/He...
Source/devtools/front_end/HeapSnapshot.js:360: var retainedNodeOrdinal =
retainedNode.nodeIndex / snapshot._nodeFieldCount;
On 2014/03/21 08:30:39, alph wrote:
> _ordinal() ?

Done.

[yurys, 2014-03-21 08:33:49]

The CQ bit was checked by yurys@chromium.org

[commit-bot: I haz the power, 2014-03-21 08:33:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yurys@chromium.org/207523004/20001

[commit-bot: I haz the power, 2014-03-21 09:15:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-21 09:15:28]

Try jobs failed on following builders:
  tryserver.blink on win_blink_rel

[yurys, 2014-03-21 09:32:29]

The CQ bit was checked by yurys@chromium.org

[commit-bot: I haz the power, 2014-03-21 09:32:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yurys@chromium.org/207523004/20001

[commit-bot: I haz the power, 2014-03-21 10:09:30]

Change committed as 169732