Fix NPE (issue 11881024)

2 views
Skip to first unread message

peter...@gmail.com

unread,
Jan 14, 2013, 10:14:24 AM1/14/13
to apa...@chromium.org, chromedevtoo...@googlegroups.com
Reviewers: apavlov,

Message:
TBR=apa...@chromium.org

Description:
Fix NPE


Please review this at https://codereview.chromium.org/11881024/

SVN Base: https://chromedevtools.googlecode.com/svn/trunk

Affected files:
M
plugins/org.chromium.debug.core/src/org/chromium/debug/core/model/ConnectedTargetData.java


Index:
plugins/org.chromium.debug.core/src/org/chromium/debug/core/model/ConnectedTargetData.java
diff --git
a/plugins/org.chromium.debug.core/src/org/chromium/debug/core/model/ConnectedTargetData.java
b/plugins/org.chromium.debug.core/src/org/chromium/debug/core/model/ConnectedTargetData.java
index
f23d83306ce8a0d1748f66aad607acb8eec31074..6d6fea467110a9e104c40e4062afd1fed4f31bf3
100644
---
a/plugins/org.chromium.debug.core/src/org/chromium/debug/core/model/ConnectedTargetData.java
+++
b/plugins/org.chromium.debug.core/src/org/chromium/debug/core/model/ConnectedTargetData.java
@@ -385,7 +385,11 @@ public class ConnectedTargetData {

@Override
EvaluateContext getEvaluateContext() {
- return getThread().getEvaluateContext();
+ JavascriptThread thread = getThread();
+ if (thread == null) {
+ return null;
+ }
+ return thread.getEvaluateContext();
}

@Override


Reply all
Reply to author
Forward
0 new messages