Bug 887320 - Modify deprecated yield without value to yield undefined. r=mak
authorSushant Hiray <hiraysushant@gmail.com>
Tue, 02 Jul 2013 15:49:06 -0400
changeset 137198 c361c2af8910f187eee8714981e2aed99bca9e7c
parent 137197 f568666066f0917e9795da23859ee5f088da922f
child 137199 982d07833360c264e90d4a6e34f8b7d31bb2620c
push id30431
push userryanvm@gmail.com
push dateTue, 02 Jul 2013 19:49:14 +0000
treeherdermozilla-inbound@982d07833360 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmak
bugs887320
milestone25.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 887320 - Modify deprecated yield without value to yield undefined. r=mak
browser/components/places/content/controller.js
toolkit/components/places/tests/browser/browser_colorAnalyzer.js
toolkit/components/places/tests/unit/test_isURIVisited.js
--- a/browser/components/places/content/controller.js
+++ b/browser/components/places/content/controller.js
@@ -932,17 +932,17 @@ PlacesController.prototype = {
       while (aURIs.length) {
         let URIslice = aURIs.splice(0, REMOVE_PAGES_CHUNKLEN);
         PlacesUtils.bhistory.removePages(URIslice, URIslice.length);
         Services.tm.mainThread.dispatch(function() {
           try {
             gen.next();
           } catch (ex if ex instanceof StopIteration) {}
         }, Ci.nsIThread.DISPATCH_NORMAL); 
-        yield;
+        yield undefined;
       }
     }
     let gen = pagesChunkGenerator(URIs);
     gen.next();
   },
 
   /**
    * Removes history visits for an history container node.
--- a/toolkit/components/places/tests/browser/browser_colorAnalyzer.js
+++ b/toolkit/components/places/tests/browser/browser_colorAnalyzer.js
@@ -16,17 +16,17 @@ const hiddenWindowDoc = Cc["@mozilla.org
 const XHTML_NS = "http://www.w3.org/1999/xhtml";
 
 // to make async tests easier, push them into here and call nextStep
 // when a test finishes
 let tests = [];
 function generatorTest() {
   while (tests.length > 0) {
     tests.shift()();
-    yield;
+    yield undefined;
   }
 }
 
 /**
  * Passes the given uri to findRepresentativeColor.
  * If expected is null, you expect it to fail.
  * If expected is a function, it will call that function.
  * If expected is a color, you expect that color to be returned.
--- a/toolkit/components/places/tests/unit/test_isURIVisited.js
+++ b/toolkit/components/places/tests/unit/test_isURIVisited.js
@@ -82,14 +82,14 @@ function step()
         };
 
         history.updatePlaces({ uri:    uri
                              , visits: [ { transitionType: transition
                                          , visitDate:      Date.now() * 1000
                                          } ]
                              }, callback);
       });
-      yield;
+      yield undefined;
     }
   }
 
   do_test_finished();
 }