Chromium Code Reviews
DescriptionRemove outdated interface tests which were hidden by promise_test behaviour.
A recent change to testharness.js (https://codereview.chromium.org/2468053002)
causes promise_test to throw a TypeError exception when the test returns
something other than a promise. Previously, the test could hide test
failures if the asynchronous test failures resolved after the default
promise created by promise_test.
The change to promise_test behaviour caused the budget service's interface.html
test to start failing because it did not return a promise, but it also
exposed the root problem that outdated code should have been failing
already.
This CL removes the interfaces-expected.txt which reported the promise_test
misuse, and also removes the outdated code.
BUG=662172
Committed: https://crrev.com/af2af12b8a17d3176148b15ce17e5236879ae60a
Cr-Commit-Position: refs/heads/master@{#429907}
Patch Set 1 #
Messages
Total messages: 12 (7 generated)
|
||||||||||||||||||||||||||||