Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(812)

Unified Diff: test/mjsunit/harmony/block-for-sloppy.js

Issue 1361403003: Implement ES6 completion semantics (--harmony-completion). (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/mjsunit/harmony/block-for-sloppy.js
diff --git a/test/mjsunit/harmony/block-for-sloppy.js b/test/mjsunit/harmony/block-for-sloppy.js
index eee8e0b5cdc73198914debb77e9ab2b395b8c58f..d316bd9083293df4b10d3f737c38bb4ee87fdf17 100644
--- a/test/mjsunit/harmony/block-for-sloppy.js
+++ b/test/mjsunit/harmony/block-for-sloppy.js
@@ -25,7 +25,7 @@
// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-// Flags: --no-legacy-const --harmony-sloppy --harmony-sloppy-let
+// Flags: --no-legacy-const --harmony-sloppy --harmony-sloppy-let --harmony-completion
rossberg 2015/10/05 11:33:00 Nit: line too long. You can have several Flags lin
neis 2015/10/07 11:58:40 Done.
function props(x) {
var array = [];
@@ -190,10 +190,10 @@ assertEquals(undefined, eval("for (let i = 0; i < 10; i++) { continue; i; }"));
assertEquals(0, eval("for (let i = 0; true;) { i; break; }"));
assertEquals(0, eval("for (const i = 0; true;) { i; break; }"));
assertEquals(9, eval("for (let i = 0; i < 10; i++) { i; continue; }"));
-assertEquals(3, eval("for (let i = 0; true; i++) { i; if (i >= 3) break; }"));
-assertEquals(2, eval("for (let i = 0; true; i++) { if (i >= 3) break; i; }"));
+assertEquals(undefined, eval("for (let i = 0; true; i++) { i; if (i >= 3) break; }"));
rossberg 2015/10/05 11:33:00 Nit: line length
neis 2015/10/07 11:58:40 Done.
+assertEquals(undefined, eval("for (let i = 0; true; i++) { if (i >= 3) break; i; }"));
rossberg 2015/10/05 11:33:00 Dito
neis 2015/10/07 11:58:40 Done.
assertEquals(
- 2, eval("for (let i = 0; i < 10; i++) { if (i >= 3) continue; i; }"));
+ undefined, eval("for (let i = 0; i < 10; i++) { if (i >= 3) continue; i; }"));
assertEquals(undefined, eval("foo: for (let i = 0; true;) { break foo; }"));
assertEquals(undefined, eval("foo: for (const i = 0; true;) { break foo; }"));
assertEquals(3, eval("foo: for (let i = 3; true;) { i; break foo; }"));

Powered by Google App Engine
This is Rietveld 408576698