|this.mInputElt is null| triggered by FireFTP extension.

RESOLVED FIXED in seamonkey2.1a1

Status

SeaMonkey
Autocomplete
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Philip Chee, Assigned: Philip Chee)

Tracking

Trunk
seamonkey2.1a1

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: parity-fx)

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
The XPFE autocomplete textbox binding falls over when bound to a FireFTP autocomplete textbox because it uses a feature in the toolkit autocomplete.xml not back-synced to XPFE.
(Assignee)

Updated

8 years ago
Summary: |this.mInputElt| is null triggered by FireFTP extension. → |this.mInputElt is null| triggered by FireFTP extension.
(Assignee)

Comment 1

8 years ago
Created attachment 419419 [details] [diff] [review]
Trunk Patch v1.0: Minimal backport from toolkit to fix the FireFTP problem
[Checkin: Comment 7]

STR:
1. Install FireFTP mod from:
<http://ftp.osuosl.org/pub/mozdev/xsidebar/mods/fireftp-1.0.7.xpi>
to SeaMonkey 2.1a1pre.

2. Navigate to the FireFTP install location and edit the chrome.manifest commenting out the following line:
#style   chrome://fireftp/content/fireftp.xul chrome://fireftp/content/fireftp.css application={92650c4d-4b8e-4d2a-b7eb-24ecf4f6b63a}

3. Re-Start SeaMonkey. Go Tools->FireFTP.

4. Open the error console. Observe error:
Error: this.mInputElt is null
Source file: chrome://global/content/autocomplete.xml
Attachment #419419 - Flags: superreview?(neil)
Attachment #419419 - Flags: review?(neil)

Comment 2

8 years ago
Comment on attachment 419419 [details] [diff] [review]
Trunk Patch v1.0: Minimal backport from toolkit to fix the FireFTP problem
[Checkin: Comment 7]

>-        <children includes="image|deck|stack">
>+        <children includes="image|deck|stack|box">
Does FireFTP actually use a box child?
(Assignee)

Comment 3

8 years ago
> Does FireFTP actually use a box child?

<textbox id                      = "remotepath"
         value                   = "/"
         flex                    = "1"
         tooltiptext             = "&remotepath.tip;"
         onfocus                 = "onRemotePathFocus(event)"
         onblur                  = "onRemotePathBlur(event)"
         onkeypress              = "if (event.keyCode == 13) remoteDirTree.changeDir(this.value)"
         ontextentered           = "remoteDirTree.changeDir(this.value)"
         type                    = "autocomplete"
         tabscrolling            = "true"
         enablehistory           = "true"
         autocompletepopup       = "PopupAutoComplete"
         autocompletesearch      = "form-history"
         autocompletesearchparam = "remotepath">
  <box id="identity-box" align="center" popup="identity-popup">
    <deck id="page-proxy-deck">
      <image id="page-proxy-button"/>
      <image id="page-proxy-favicon" validate="never"
             onload="this.parentNode.selectedIndex = 1; event.stopPropagation();"
             onerror="this.removeAttribute('src'); this.parentNode.selectedIndex = 0;"/>
    </deck>
  </box>
</textbox>
(Assignee)

Comment 4

8 years ago
>          enablehistory           = "true"

Hmm. We don't have this either but fortunately disablehistory defaults to false

Comment 5

8 years ago
Comment on attachment 419419 [details] [diff] [review]
Trunk Patch v1.0: Minimal backport from toolkit to fix the FireFTP problem
[Checkin: Comment 7]

I checked hg blame and gavin tried and failed not to include the box. Oh well.
Attachment #419419 - Flags: superreview?(neil)
Attachment #419419 - Flags: superreview+
Attachment #419419 - Flags: review?(neil)
Attachment #419419 - Flags: review+

Comment 6

8 years ago
(In reply to comment #4)
>>          enablehistory           = "true"
>Hmm. We don't have this either but fortunately disablehistory defaults to false
Incompatible implementation anyway.
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
Comment on attachment 419419 [details] [diff] [review]
Trunk Patch v1.0: Minimal backport from toolkit to fix the FireFTP problem
[Checkin: Comment 7]


http://hg.mozilla.org/mozilla-central/rev/ca1c43fda0a9
Attachment #419419 - Attachment description: Trunk Patch v1.0: Minimal backport from toolkit to fix the FireFTP problem. → Trunk Patch v1.0: Minimal backport from toolkit to fix the FireFTP problem [Checkin: Comment 7]
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a1
Version: SeaMonkey 2.0 Branch → Trunk
You need to log in before you can comment on or make changes to this bug.