Closed
Bug 799230
Opened 12 years ago
Closed 11 years ago
[Metro] Error message in JS console during startup: chrome://browser/content/bindings/autocomplete.xml: this.input is null
Categories
(Firefox for Metro Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ally, Assigned: mbrubeck)
References
Details
(Whiteboard: [completed-elm])
Attachments
(1 file)
1.06 KB,
patch
|
sfoster
:
review+
|
Details | Diff | Splinter Review |
error to console upon starting metro up JavaScript error: chrome://browser/content/bindings/autocomplete.xml, line 255: this.input is null
Assignee | ||
Comment 1•12 years ago
|
||
If anyone would like to work on this, the code generating this error is here. We probably just need to make sure this code isn't run before this.input is set. http://hg.mozilla.org/projects/elm/file/503aba7760ad/browser/metro/base/content/bindings/autocomplete.xml#l255 For help building and running Metro Firefox, see: https://wiki.mozilla.org/Firefox/Windows_8_Integration
Summary: error starting up metro: chrome://browser/content/bindings/autocomplete.xml → [Metro] Error message in JS console during startup: chrome://browser/content/bindings/autocomplete.xml: this.input is null
Whiteboard: [mentor=mbrubeck][lang=js]
Updated•12 years ago
|
Component: Untriaged → General
Product: Firefox → Firefox for Metro
Version: unspecified → Trunk
Assignee | ||
Comment 2•12 years ago
|
||
This error is probably happening when this constructor calls gridBoundCallback, which calls updateResults: http://hg.mozilla.org/projects/elm/file/503aba7760ad/browser/metro/base/content/bindings/grid.xml#l260 That code was added here in bug 792901. We should make sure not to regress that bug while fixing this one.
Blocks: 792901
Assignee | ||
Comment 3•11 years ago
|
||
This is safe and it silences the warning, but if anyone has a clever idea for how to restructure this code so that updateResults isn't called too early/often, that'd be great too.
Assignee: nobody → mbrubeck
Attachment #702346 -
Flags: review?(sfoster)
Comment 4•11 years ago
|
||
Comment on attachment 702346 [details] [diff] [review] patch Review of attachment 702346 [details] [diff] [review]: ----------------------------------------------------------------- I think this probably is good enough for now. We could have some init / if(this._inited) flag, but actually we might want to revisit this whole thing later once the richgrid / tile group is done so I think even that isn't worth it right now
Attachment #702346 -
Flags: review?(sfoster) → review+
Assignee | ||
Comment 5•11 years ago
|
||
https://hg.mozilla.org/projects/elm/rev/efa80af076d8
Status: NEW → ASSIGNED
Whiteboard: [mentor=mbrubeck][lang=js] → [completed-elm]
Assignee | ||
Comment 6•11 years ago
|
||
Resolving bugs in the Firefox for Metro product that are fixed on the elm branch. Sorry for the bugspam. Search your email for "bugspam-elm" if you want to find and delete all of these messages at once.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
OS: Windows 8 Metro → Windows 8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•