515899 - "Empty string passed to getElementById()" warning on TB start-up.

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Valery Kondakoff]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.1.3) Gecko/20090824 Firefox/3.5.3
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.1.4pre) Gecko/20090910 Shredder/3.0b4pre

Every TB start-up I see following error and warning in Error Console (see screenshot):

Error: Components.classes['@mozilla.org/satchel/form-fill-controller;1'] is undefined
Source File: chrome://global/content/bindings/browser.xml
Line: 400

Warning: Empty string passed to getElementById().

Reproducible: Always

Comment 1

[Mark Banner (:standard8)]

(In reply to comment #0)
> Error: Components.classes['@mozilla.org/satchel/form-fill-controller;1'] is
> undefined
> Source File: chrome://global/content/bindings/browser.xml
> Line: 400

This has already been filed at least once and commented on in one bug. This will be fixed in today's nightly as bug 510377 has now landed on 1.9.1 branch.

> Warning: Empty string passed to getElementById().

This is still a bug however.

Comment 2

[Mark Banner (:standard8)]

Attachment: Silence the warning

Silence the getElementById warning. The issue here is that we've created a findbar and not pointed it at the browser, so findbar.xml gets all upset and moans that it can't find anything. As this is just a dummy to copy anyway, we'll just give it some dummy ids.

Comment 3

[Mark Banner (:standard8)]

Comment on attachment 399991 [details] [diff] [review]
Silence the warning

Given this is just a small fix with linking up a couple of ids and getting rid of console noise on start up, I think we can take this for b4.

Comment 4

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/4de57d545330