Clean up references to nextValueOrThrow and StopIteration from the ES4 Iteration protocol.

RELNOTES: n/a

PiperOrigin-RevId: 436486224
Change-Id: I8c3bac4ed17277a6ddead69b91d87309e6904d76

Author: 12wrigja

closure/goog/date/daterange.js

@@ -423,14 +423,3 @@ goog.date.DateRange.Iterator.prototype.next = function() {
   this.nextDate_.add(new goog.date.Interval(goog.date.Interval.DAYS, 1));
   return goog.iter.createEs6IteratorYield(rv);
 };
-
-
-/**
- * TODO(user): Please do not remove - this will be cleaned up centrally.
- * @override @see {!goog.iter.Iterator}
- * @return {!goog.date.Date}
- */
-goog.date.DateRange.Iterator.prototype.nextValueOrThrow = function() {
-  return goog.iter.toEs4IteratorNext(
-      goog.date.DateRange.Iterator.prototype.next.call(this));
-};

closure/goog/dom/controlrange.js

@@ -522,18 +522,6 @@ goog.dom.ControlRangeIterator.prototype.next = function() {
 };
 
 
-/**
- * TODO(user): Please do not remove - this will be cleaned up centrally.
- * @override @see {!goog.iter.Iterator}
- * @return {!Node}
- */
-goog.dom.ControlRangeIterator.prototype.nextValueOrThrow = function() {
-  return goog.iter.toEs4IteratorNext(
-      goog.dom.ControlRangeIterator.prototype.next.call(this));
-};
-
-
-
 /** @override */
 goog.dom.ControlRangeIterator.prototype.copyFrom = function(other) {
   'use strict';

closure/goog/dom/iter.js

@@ -66,18 +66,6 @@ goog.dom.iter.SiblingIterator.prototype.next = function() {
 };
 
 
-/**
- * TODO(user): Please do not remove - this will be cleaned up centrally.
- * @override @see {!goog.iter.Iterator}
- * @return {!Node}
- */
-goog.dom.iter.SiblingIterator.prototype.nextValueOrThrow = function() {
-  return goog.iter.toEs4IteratorNext(
-      goog.dom.iter.SiblingIterator.prototype.next.call(this));
-};
-
-
-
 /**
  * Iterator over an Element's children.
  * @param {Element} element The element to iterate over.
@@ -139,15 +127,4 @@ goog.dom.iter.AncestorIterator.prototype.next = function() {
   }
   this.node_ = node.parentNode;
   return goog.iter.createEs6IteratorYield(node);
-};
-
-
-/**
- * TODO(user): Please do not remove - this will be cleaned up centrally.
- * @override @see {!goog.iter.Iterator}
- * @return {!Node}
- */
-goog.dom.iter.AncestorIterator.prototype.nextValueOrThrow = function() {
-  return goog.iter.toEs4IteratorNext(
-      goog.dom.iter.AncestorIterator.prototype.next.call(this));
-};
+};

closure/goog/dom/multirange.js

@@ -555,18 +555,6 @@ goog.dom.MultiRangeIterator.prototype.next = function() {
 };
 
 
-/**
- * TODO(user): Please do not remove - this will be cleaned up centrally.
- * @override @see {!goog.iter.Iterator}
- * @return {!Node}
- */
-goog.dom.MultiRangeIterator.prototype.nextValueOrThrow = function() {
-  return goog.iter.toEs4IteratorNext(
-      goog.dom.MultiRangeIterator.prototype.next.call(this));
-};
-
-
-
 /** @override */
 goog.dom.MultiRangeIterator.prototype.copyFrom = function(other) {
   'use strict';

closure/goog/dom/nodeiterator.js

@@ -79,14 +79,3 @@ goog.dom.NodeIterator.prototype.next = function() {
 
   return goog.iter.createEs6IteratorYield(/** @type {!Node} */ (this.node));
 };
-
-
-/**
- * TODO(user): Please do not remove - this will be cleaned up centrally.
- * @override @see {!goog.iter.Iterator}
- * @return {!Node}
- */
-goog.dom.NodeIterator.prototype.nextValueOrThrow = function() {
-  return goog.iter.toEs4IteratorNext(
-      goog.dom.NodeIterator.prototype.next.call(this));
-};

closure/goog/dom/tagiterator.js

@@ -285,18 +285,6 @@ goog.dom.TagIterator.prototype.next = function() {
 };
 
 
-/**
- * TODO(user): Please do not remove - this will be cleaned up centrally.
- * @override @see {!goog.iter.Iterator}
- * @return {!Node}
- */
-goog.dom.TagIterator.prototype.nextValueOrThrow = function() {
-  return goog.iter.toEs4IteratorNext(
-      goog.dom.TagIterator.prototype.next.call(this));
-};
-
-
-
 /**
  * @return {boolean} Whether next has ever been called on this iterator.
  * @protected

closure/goog/dom/textrangeiterator.js

@@ -196,8 +196,8 @@ goog.dom.TextRangeIterator.prototype.isLast = function() {
 };
 
 /**
- * Returns true if the iterator is on the last step before StopIteration is
- * thrown, otherwise false.
+ * Returns true if the iterator is on the last step before iteration finishes,
+ * false otherwise.
  * @return {boolean}
  * @private
  */
@@ -218,8 +218,8 @@ goog.dom.TextRangeIterator.prototype.isLastTag_ = function() {
 };
 
 /**
- * Move to the next position in the selection.
- * Throws `goog.iter.StopIteration` when it passes the end of the range.
+ * Move to the next position in the selection. Returns `{done: true}` when it
+ * passes the end of the range.
  * @return {!IIterableResult<!Node>} The node at the next position.
  * @override
  */
@@ -234,17 +234,6 @@ goog.dom.TextRangeIterator.prototype.next = function() {
 };
 
 
-/**
- * TODO(user): Please do not remove - this will be cleaned up centrally.
- * @override @see {!goog.iter.Iterator}
- * @return {!Node}
- */
-goog.dom.TextRangeIterator.prototype.nextValueOrThrow = function() {
-  return goog.iter.toEs4IteratorNext(
-      goog.dom.TextRangeIterator.prototype.next.call(this));
-};
-
-
 /**
  * Get the last node the iterator will hit.
  * @return {?Node} The last node the iterator will hit.

closure/goog/iter/es6.js

@@ -13,7 +13,6 @@ goog.module.declareLegacyNamespace();
 
 const GoogIterable = goog.require('goog.iter.Iterable');
 const GoogIterator = goog.require('goog.iter.Iterator');
-const StopIteration = goog.require('goog.iter.StopIteration');
 
 
 /**
@@ -121,12 +120,6 @@ class ShimGoogIterator extends GoogIterator {
     this.iter_ = iter;
   }
 
-  /** @override */
-  nextValueOrThrow() {
-    const result = this.iter_.next();
-    if (result.done) throw StopIteration;
-    return result.value;
-  }
   /**
    * @override @see {!goog.iter.Iterator}
    * @return {!IIterableResult<VALUE>}