Closed Bug 1159938 Opened 7 years ago Closed 7 years ago

`this._inspector.change is not a function` error after saving a text node value in markup view

Categories

(DevTools :: Inspector, defect)

defect
Not set
normal

Tracking

(firefox39 affected, firefox40 affected, firefox41 fixed)

RESOLVED FIXED
Firefox 41
Tracking Status
firefox39 --- affected
firefox40 --- affected
firefox41 --- fixed

People

(Reporter: bgrins, Assigned: bgrins)

Details

Attachments

(1 file)

After editing a text node I'm seeing an error in the console.  I'm seeing it in a 39 build also.

Full Stack: MarkupView.prototype.nodeChanged@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/markupview/markup-view.js:1213:7
TextEditor/<.done/</</</<@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/markupview/markup-view.js:2277:15
Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:867:23
this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:746:7
this.PromiseWalker.scheduleWalkerLoop/<@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:688:37
Promise*this.PromiseWalker.scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:688:5
this.PromiseWalker.schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:709:7
this.PromiseWalker.completePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:671:7
Front<.onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1217:7
DebuggerClient.prototype.onPacket@resource://gre/modules/devtools/dbg-client.jsm:953:7
LocalDebuggerTransport.prototype.send/<@resource://gre/modules/devtools/dbg-client.jsm -> resource://gre/modules/devtools/transport/transport.js:561:11
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:82:14
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:82:14
Oh that's right, I saw that a few times recently too, and I remember looking for the change function in the code and couldn't find it. Going through change history, I couldn't find it there either.
I think this has been dead code for a while, but maybe some recent change made it be called again or something.
Remove dead nodeChanged() function.  Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=970394220e0e
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Attachment #8602952 - Flags: review?(pbrosset)
Attachment #8602952 - Flags: review?(pbrosset) → review+
(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #3)
> Maybe worth uplifting?

Yeah, maybe.  It's not affecting the user (other than spewing to the console), but it's also a simple patch that should apply cleanly.
Keywords: checkin-needed
remote:   https://hg.mozilla.org/integration/fx-team/rev/7278397ba6dc
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/7278397ba6dc
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 41
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.