Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(212)

Issue 12320023: Disabled context variables in local variable list in the debugger (Closed)

Created:
7 years, 10 months ago by Tom Ball
Modified:
7 years, 10 months ago
Reviewers:
siva, Ivan Posva
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Visibility:
Public.

Description

Disabled context variables in local variable list in the debugger, so developers can still use it while issue 8593 is being fixed. Committed: https://code.google.com/p/dart/source/detail?r=19020

Patch Set 1 #

Patch Set 2 : Return <unknown> for context variables now. #

Total comments: 3

Patch Set 3 : #

Patch Set 4 : #

Patch Set 5 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+322 lines, -310 lines) Patch
M runtime/vm/debugger.cc View 1 2 3 4 2 chunks +5 lines, -1 line 0 comments Download
M runtime/vm/debugger_api_impl_test.cc View 1 2 3 4 1 chunk +317 lines, -309 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Tom Ball
7 years, 10 months ago (2013-02-21 00:17:35 UTC) #1
siva
lgtm
7 years, 10 months ago (2013-02-21 01:11:08 UTC) #2
Ivan Posva
https://codereview.chromium.org/12320023/diff/3/runtime/vm/debugger.cc File runtime/vm/debugger.cc (right): https://codereview.chromium.org/12320023/diff/3/runtime/vm/debugger.cc#newcode520 runtime/vm/debugger.cc:520: *value = String::New("<unknown>"); Symbol::New("<unknown>") https://codereview.chromium.org/12320023/diff/3/runtime/vm/debugger.cc#newcode520 runtime/vm/debugger.cc:520: *value = String::New("<unknown>"); ...
7 years, 10 months ago (2013-02-21 01:26:12 UTC) #3
Tom Ball
https://codereview.chromium.org/12320023/diff/3/runtime/vm/debugger.cc File runtime/vm/debugger.cc (right): https://codereview.chromium.org/12320023/diff/3/runtime/vm/debugger.cc#newcode520 runtime/vm/debugger.cc:520: *value = String::New("<unknown>"); On 2013/02/21 01:26:12, Ivan Posva wrote: ...
7 years, 10 months ago (2013-02-21 17:32:57 UTC) #4
Tom Ball
7 years, 10 months ago (2013-02-25 23:36:35 UTC) #5
Message was sent while issue was closed.
Committed patchset #5 manually as r19020.

Powered by Google App Engine
This is Rietveld 408576698