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

Unified Diff: third_party/WebKit/Source/devtools/front_end/sources/SourceMapNamesResolver.js

Issue 2550563002: [DevTools] Remove obsolete experiment flags. (Closed)
Patch Set: rebased Created 4 years 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
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/security/module.json ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/devtools/front_end/sources/SourceMapNamesResolver.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sources/SourceMapNamesResolver.js b/third_party/WebKit/Source/devtools/front_end/sources/SourceMapNamesResolver.js
index 9a6b7019446c64c67c0ea32a247f7ee4b553159d..058cba20ac81173ab76fdf44b965c7f7c2069b3b 100644
--- a/third_party/WebKit/Source/devtools/front_end/sources/SourceMapNamesResolver.js
+++ b/third_party/WebKit/Source/devtools/front_end/sources/SourceMapNamesResolver.js
@@ -225,7 +225,7 @@ Sources.SourceMapNamesResolver._allVariablesInCallFrame = function(callFrame) {
*/
Sources.SourceMapNamesResolver.resolveExpression = function(
callFrame, originalText, uiSourceCode, lineNumber, startColumnNumber, endColumnNumber) {
- if (!Runtime.experiments.isEnabled('resolveVariableNames') || !uiSourceCode.contentType().isFromSourceMap())
+ if (!uiSourceCode.contentType().isFromSourceMap())
return Promise.resolve('');
return Sources.SourceMapNamesResolver._allVariablesInCallFrame(callFrame).then(findCompiledName);
@@ -301,7 +301,7 @@ Sources.SourceMapNamesResolver._resolveExpression = function(
Sources.SourceMapNamesResolver.resolveThisObject = function(callFrame) {
if (!callFrame)
return Promise.resolve(/** @type {?SDK.RemoteObject} */ (null));
- if (!Runtime.experiments.isEnabled('resolveVariableNames') || !callFrame.scopeChain().length)
+ if (!callFrame.scopeChain().length)
return Promise.resolve(callFrame.thisObject());
return Sources.SourceMapNamesResolver._resolveScope(callFrame.scopeChain()[0]).then(onScopeResolved);
@@ -338,9 +338,6 @@ Sources.SourceMapNamesResolver.resolveThisObject = function(callFrame) {
* @return {!SDK.RemoteObject}
*/
Sources.SourceMapNamesResolver.resolveScopeInObject = function(scope) {
- if (!Runtime.experiments.isEnabled('resolveVariableNames'))
- return scope.object();
-
var startLocation = scope.startLocation();
var endLocation = scope.endLocation();
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/security/module.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698