Commit de901269 authored by Yuki Yamada's avatar Yuki Yamada Committed by Commit Bot

Do not clear TraceWrapperV8Reference in blink::V8AbstractEventListener destructor

This CL removed some code that clearing TraceWrapperV8Reference object
in its destructor.
|listener_| is cleared in the destructor of TraceWrapperV8Reference, so
we do not have to clear it in blink::V8AbstractEventListener.

Bug: 872138
Change-Id: I4f8e8e67074ee8b2081735053b5b0f7604afadff
Reviewed-on: https://chromium-review.googlesource.com/1173195
Commit-Queue: Yuki Yamada <yukiy@google.com>
Reviewed-by: default avatarHitoshi Yoshida <peria@chromium.org>
Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
Reviewed-by: default avatarYuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#582819}
parent d98017a0
...@@ -70,8 +70,6 @@ V8AbstractEventListener::~V8AbstractEventListener() { ...@@ -70,8 +70,6 @@ V8AbstractEventListener::~V8AbstractEventListener() {
DCHECK(listener_.IsEmpty()); DCHECK(listener_.IsEmpty());
InstanceCounters::DecrementCounter( InstanceCounters::DecrementCounter(
InstanceCounters::kJSEventListenerCounter); InstanceCounters::kJSEventListenerCounter);
} else {
listener_.Clear();
} }
} }
......
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