Don't create debug context if debug listener is not set
authoryurys <yurys@chromium.org>
Tue, 12 May 2015 07:33:21 +0000 (00:33 -0700)
committerCommit bot <commit-bot@chromium.org>
Tue, 12 May 2015 07:33:18 +0000 (07:33 +0000)
commitcf07add2272fee6e2ee8c99da087aa50ff654eff
treeb1278e92f3cb66fb3fe04f2edb2dc5439fdda818
parentc39a0a75ad5066108b5e4dd10e14b3e0cff63b2d
Don't create debug context if debug listener is not set

If there had been no debug listener v8::Debug::GetDebugContext would have created new context and wouln't have kept reference to it. This way we may well end up with several debug contexts and disabled debugger.

As a side effect this change allows to efficiently distinguish debug context from blink contexts by simply comparing handles.

BUG=chromium:482290
LOG=Y

Review URL: https://codereview.chromium.org/1136733002

Cr-Commit-Position: refs/heads/master@{#28356}
include/v8-debug.h
src/api.cc
src/debug.cc
src/runtime/runtime-debug.cc
test/cctest/test-debug.cc
test/mjsunit/regress/regress-3960.js