Commit 261e251b authored by Yuki Yamada's avatar Yuki Yamada Committed by Commit Bot

Do not call undefined internals.gc()

Undefined internals.gc() is called in onEventInFrame() and a TypeError
is thrown in current test, but the error is not visible because the
error is dispatched in another frame as mentioned in
https://crbug.com/606900 .
The test will call gc() instead of internals.gc() after and work as
intended after this patch.

Bug: 872138, 606900
Change-Id: I5d059e5ff7bea56b382f0e973282788427f5a306
Reviewed-on: https://chromium-review.googlesource.com/1186287
Commit-Queue: Yuki Yamada <yukiy@google.com>
Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
Reviewed-by: default avatarYuki Shiino <yukishiino@chromium.org>
Reviewed-by: default avatarHitoshi Yoshida <peria@chromium.org>
Cr-Commit-Position: refs/heads/master@{#585441}
parent f448af55
......@@ -16,7 +16,7 @@ function onEventInFrame(e) {
if (e.type == removalEventType) {
debug('Removing iframe');
target.parentNode.removeChild(target);
internals.gc();
gc();
}
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment