335e169b6c
This is a reland of a2ed05144c
Original change's description:
> [debug] Add test for promise finally
>
> As of v8:6536, we no longer have to mark builtins explicitly.
>
> Also remove test whitelist for promise finally
> builtins.
>
> Bug: v8:6088, v8:5967
> Change-Id: I7f98dfe7708678653e944ac76ba9938205490b16
> Reviewed-on: https://chromium-review.googlesource.com/654067
> Commit-Queue: Sathya Gunasekaran <gsathya@chromium.org>
> Reviewed-by: Jakob Gruber <jgruber@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#47896}
TBR=jgruber@chromium.org
Bug: v8:6088, v8:5967
Change-Id: I25a1820e04596a44769fc8ded80678f3663bbcd5
Reviewed-on: https://chromium-review.googlesource.com/655740
Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org>
Commit-Queue: Sathya Gunasekaran <gsathya@chromium.org>
Cr-Commit-Position: refs/heads/master@{#47906}
47 lines
999 B
JavaScript
47 lines
999 B
JavaScript
// Copyright 2017 the V8 project authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
// Flags: --harmony-promise-finally
|
|
|
|
Debug = debug.Debug
|
|
|
|
var exception = null;
|
|
var step = 0;
|
|
|
|
function listener(event, exec_state, event_data, data) {
|
|
if (event != Debug.DebugEvent.Exception) return;
|
|
try {
|
|
var line = exec_state.frame(0).sourceLineText();
|
|
var match = /Exception/.exec(line);
|
|
assertNotNull(match);
|
|
step++;
|
|
} catch (e) {
|
|
exception = e;
|
|
}
|
|
}
|
|
|
|
// Caught throw, events on any exception.
|
|
Debug.setListener(listener);
|
|
Debug.setBreakOnException();
|
|
|
|
var thenable = {
|
|
get then() {
|
|
throw new Error('err'); // Exception
|
|
}
|
|
};
|
|
|
|
var caughtException = null;
|
|
|
|
Promise.resolve()
|
|
.finally(() => thenable)
|
|
.catch(e => caughtException = e);
|
|
|
|
%RunMicrotasks();
|
|
|
|
Debug.setListener(null);
|
|
Debug.clearBreakOnException();
|
|
assertNull(exception);
|
|
assertNotNull(caughtException);
|
|
assertEquals(1, step);
|