Remove HeapSnapshotItemIterator.rewind

Source/devtools/front_end/JSHeapSnapshot.js

 /*
  * Copyright (C) 2012 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 123 matching lines @@
             var edge = iter.edge;
             if (!edge.isShortcut())
                 continue;
             var node = edge.node();
             var propNames = {};
             for (var innerIter = node.edges(); innerIter.hasNext(); innerIter.next()) {
                 var globalObjEdge = innerIter.edge;
                 if (globalObjEdge.isShortcut())
                     propNames[globalObjEdge._nameOrIndex()] = true;
             }
-            for (innerIter.rewind(); innerIter.hasNext(); innerIter.next()) {
+            for (var innerIter = node.edges(); innerIter.hasNext(); innerIter.next()) {

[loislo, 2014/03/20 13:16:40]

The change is fine but looks like the code doesn't work at all.

[yurys, 2014/03/20 13:19:04]

Yeah, I agree, let's remove it in a separate CL.

                 var globalObjEdge = innerIter.edge;
                 if (!globalObjEdge.isShortcut()
                     && globalObjEdge.node().isHidden()
                     && globalObjEdge._hasStringName()
                     && (globalObjEdge._nameOrIndex() in propNames))
                     globalObjEdge._edges[globalObjEdge.edgeIndex + this._edgeTypeOffset] = this._edgeInvisibleType;
             }
         }
     },
 
@@ skipping 702 matching lines @@
      * @return {boolean}
      */
     isWeak: function()
     {
         return this._edge().isWeak();
     },
 
     __proto__: WebInspector.HeapSnapshotRetainerEdge.prototype
 }