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

Unified Diff: Source/devtools/front_end/components/ObjectPropertiesSection.js

Issue 738733006: DevTools: Support harmony variable scopes. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: addressed Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/devtools/front_end/components/ObjectPropertiesSection.js
diff --git a/Source/devtools/front_end/components/ObjectPropertiesSection.js b/Source/devtools/front_end/components/ObjectPropertiesSection.js
index b50493dbfa568ed76f7d4ab19574aabaacd91ee4..5119be84676afb9429c6ef2ac9bd5f1dfa68488e 100644
--- a/Source/devtools/front_end/components/ObjectPropertiesSection.js
+++ b/Source/devtools/front_end/components/ObjectPropertiesSection.js
@@ -659,21 +659,24 @@ WebInspector.FunctionScopeMainTreeElement.prototype = {
for (var i = 0; i < scopeChain.length; ++i) {
var scope = scopeChain[i];
var title = null;
- var isTrueObject;
+ var isTrueObject = false;
switch (scope.type) {
case DebuggerAgent.ScopeType.Local:
// Not really expecting this scope type here.
title = WebInspector.UIString("Local");
- isTrueObject = false;
break;
case DebuggerAgent.ScopeType.Closure:
title = WebInspector.UIString("Closure");
- isTrueObject = false;
break;
case DebuggerAgent.ScopeType.Catch:
title = WebInspector.UIString("Catch");
- isTrueObject = false;
+ break;
+ case DebuggerAgent.ScopeType.Block:
+ title = WebInspector.UIString("Block");
+ break;
+ case DebuggerAgent.ScopeType.Script:
+ title = WebInspector.UIString("Script");
break;
case DebuggerAgent.ScopeType.With:
title = WebInspector.UIString("With Block");
« no previous file with comments | « Source/core/inspector/InjectedScriptSource.js ('k') | Source/devtools/front_end/sources/ScopeChainSidebarPane.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698