Closed Bug 1276465 Opened 4 years ago Closed 4 years ago

" autocompleteMap.get(...) is undefined" error in jsconsole when creating a new stylesheet in the StyleEditor

Categories

(DevTools :: Style Editor, defect, P2)

49 Branch
defect

Tracking

(firefox49 fixed)

RESOLVED FIXED
Firefox 49
Tracking Status
firefox49 --- fixed

People

(Reporter: nchevobbe, Assigned: nchevobbe)

Details

Attachments

(1 file)

There is no impact on the tool itself, everything works fine, but the jsconsole is cluttered with error messages.

STR: 
1. Open `data:text/html,<p>hello` url
2. Open the devtools
3. Go to the style editor tab
4. Click on the "New" button in the left panel

Expected Result:
No error in the jsconsole

Actual Result:
There is an error in the jsconsole

Full error : 
Full Message: TypeError: autocompleteMap.get(...) is undefined
Full Stack: getInfoAt@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/sourceeditor/autocomplete.js:384:19
StyleSheetEditor.prototype._highlightSelectorAt<@resource://devtools/client/styleeditor/StyleSheetEditor.jsm:587:16
TaskImpl.prototype._run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:310:39
TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:272:3
createAsyncFunction/asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:246:14
StyleSheetEditor.prototype._onMouseMove/this.mouseMoveTimeout<@resource://devtools/client/styleeditor/StyleSheetEditor.jsm:574:7
setTimeout handler*StyleSheetEditor.prototype._onMouseMove@resource://devtools/client/styleeditor/StyleSheetEditor.jsm:573:29
EventListener.handleEvent*StyleSheetEditor.prototype.load/<@resource://devtools/client/styleeditor/StyleSheetEditor.jsm:464:9
Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:937:23
this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:816:7
Promise*this.PromiseWalker.scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:747:11
this.PromiseWalker.schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:779:7
this.PromiseWalker.completePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:714:7
Editor.prototype.appendTo/onLoad@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/sourceeditor/editor.js:456:7
EventListener.handleEvent*Editor.prototype.appendTo@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/sourceeditor/editor.js:459:5
StyleSheetEditor.prototype.load@resource://devtools/client/styleeditor/StyleSheetEditor.jsm:446:12
StyleEditorUI.prototype._sourceLoaded/<.onShow</<@resource://devtools/client/styleeditor/StyleEditorUI.jsm:601:19
TaskImpl.prototype._run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:310:39
TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:272:3
createAsyncFunction/asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:246:14
this.Task.spawn@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:160:12
StyleEditorUI.prototype._sourceLoaded/<.onShow<@resource://devtools/client/styleeditor/StyleEditorUI.jsm:596:9
set activeSummary@resource://devtools/client/shared/SplitView.jsm:147:7
StyleEditorUI.prototype._selectEditor/summaryPromise<@resource://devtools/client/styleeditor/StyleEditorUI.jsm:702:7
Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:937:23
this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:816:7
Promise*this.PromiseWalker.scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:747:11
this.PromiseWalker.schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:779:7
this.PromiseWalker.completePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:714:7
Front<.onPacket/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1281:9
DevTools RDP*Front<.request@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1223:7
generateRequestMethods/</frontProto[name]@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1382:14
StyleSheetEditor.prototype._getSourceTextAndPrettify@resource://devtools/client/styleeditor/StyleSheetEditor.jsm:270:12
StyleSheetEditor.prototype.fetchSource@resource://devtools/client/styleeditor/StyleSheetEditor.jsm:290:12
StyleEditorUI.prototype._addStyleSheetEditor<@resource://devtools/client/styleeditor/StyleEditorUI.jsm:348:11
TaskImpl.prototype._run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:310:39
TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:272:3
createAsyncFunction/asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:246:14
StyleEditorUI.prototype._onStyleSheetCreated@resource://devtools/client/styleeditor/StyleEditorUI.jsm:398:5
Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:937:23
this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:816:7
Promise*this.PromiseWalker.scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:747:11
this.PromiseWalker.schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:779:7
this.PromiseWalker.completePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:714:7
Front<.onPacket/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1281:9
DevTools RDP*Front<.request@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1223:7
generateRequestMethods/</frontProto[name]@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1382:14
StyleEditorUI.prototype.createUI/<@resource://devtools/client/styleeditor/StyleEditorUI.jsm:161:7
EventListener.handleEvent*wire/<@resource://devtools/client/styleeditor/StyleEditorUtil.jsm:161:7
forEach@resource://devtools/client/styleeditor/StyleEditorUtil.jsm:106:7
Assignee: nobody → chevobbe.nicolas
Comment on attachment 8757641 [details]
MozReview Request: Bug 1276465 - Check if autompleter map item exists before using it. r=bgrins

https://reviewboard.mozilla.org/r/56012/#review53676
Attachment #8757641 - Flags: review?(bgrinstead) → review+
TRY was green on dt* except one known intermittent
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/fx-team/rev/8d11755302e8
Check if autompleter map item exists before using it. r=bgrins
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/8d11755302e8
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.