Closed
Bug 1050000
Opened 10 years ago
Closed 10 years ago
Handler function JSPropertyProvider threw an exception: TypeError: result is undefined
Categories
(DevTools :: Console, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 36
People
(Reporter: fitzgen, Assigned: tromey)
References
Details
Attachments
(1 file)
1.05 KB,
patch
|
past
:
review+
|
Details | Diff | Splinter Review |
Got this when playing with the console when chrome debugging: Handler function JSPropertyProvider threw an exception: TypeError: result is undefined Stack: DebuggerEnvironmentSupport.getProperty@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/toolkit/webconsole/utils.js:1168:1 getExactMatch_impl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/toolkit/webconsole/utils.js:1098:16 getVariableInEnvironment@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/toolkit/webconsole/utils.js:1003:10 JSPropertyProvider@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/toolkit/webconsole/utils.js:895:11 makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14 WCA_onAutocomplete@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/webconsole.js:805:18 DSC_onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/main.js:1225:15 DebuggerTransport.prototype._onJSONObjectReady/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/transport/transport.js:461:9 makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14 makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14 EventLoop.prototype.enter@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:410:5 ThreadActor.prototype._pushThreadPause@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:609:5 ThreadActor.prototype._pauseAndRespond@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:783:7 ThreadActor.prototype._makeSteppingHooks/steppingHookState.pauseAndRespond@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:915:16 ThreadActor.prototype._makeOnStep/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:896:1 AbstractCanvasGraph.prototype.refresh@resource:///modules/devtools/Graphs.jsm:611:7 setNamedTimeout/<@resource:///modules/devtools/ViewHelpers.jsm:65:39 setTimeout_timer@resource://gre/modules/Timer.jsm:30:5 Line: 1168, column: 0
See Also: → 1068448
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → ttromey
Assignee | ||
Comment 1•10 years ago
|
||
Well, I took this, thinking I had a patch to fix it. (A few days ago I could reproduce this by typing something like "content." into the browser console) However, now when I backed out my patch I can no longer reproduce the bug. I suspect it was fixed some other way.
Assignee | ||
Comment 2•10 years ago
|
||
Today I happened to encounter the error again. I could only make it happen in a certain scenario (typing "this." while stopped at a breakpoint in timeline.js). However this was enough to verify that this patch removes the exception.
Assignee | ||
Updated•10 years ago
|
Attachment #8505539 -
Flags: review?(past)
Comment 3•10 years ago
|
||
Comment on attachment 8505539 [details] [diff] [review] check "result" against undefined before use Review of attachment 8505539 [details] [diff] [review]: ----------------------------------------------------------------- Good catch!
Attachment #8505539 -
Flags: review?(past) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 4•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/683d12c284ff
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Comment 5•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/683d12c284ff
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 36
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•