Bug 1967673 - [devtools] Don't omit valid safe getter undefined values in _findSafeGetterValues. r=devtools-reviewers,jdescottes
Differential Revision: https://phabricator.services.mozilla.com/D250613
This commit is contained in:
committed by
nchevobbe@mozilla.com
parent
39032d8048
commit
4fe1cbf23e
@@ -478,7 +478,7 @@ class ObjectActor extends Actor {
|
||||
}
|
||||
|
||||
const getterValue = this._evaluateGetter(desc.get);
|
||||
if (getterValue === undefined) {
|
||||
if (getterValue === this._evaluateGetterNoResult) {
|
||||
continue;
|
||||
}
|
||||
|
||||
@@ -516,6 +516,8 @@ class ObjectActor extends Actor {
|
||||
return safeGetterValues;
|
||||
}
|
||||
|
||||
_evaluateGetterNoResult = Symbol();
|
||||
|
||||
/**
|
||||
* Evaluate the getter function |desc.get|.
|
||||
* @param {Object} getter
|
||||
@@ -523,10 +525,10 @@ class ObjectActor extends Actor {
|
||||
_evaluateGetter(getter) {
|
||||
const result = getter.call(this.obj);
|
||||
if (!result || "throw" in result) {
|
||||
return undefined;
|
||||
return this._evaluateGetterNoResult;
|
||||
}
|
||||
|
||||
let getterValue = undefined;
|
||||
let getterValue = this._evaluateGetterNoResult;
|
||||
if ("return" in result) {
|
||||
getterValue = result.return;
|
||||
} else if ("yield" in result) {
|
||||
|
||||
Reference in New Issue
Block a user