273910 - typeahead find causes problems if the tab was closed while the findbar was open

Status: RESOLVED FIXED

(Toolkit :: Find Toolbar, defect)

Description

[Dorando]

User-Agent:       Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0
Build Identifier: Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:1.8a6) Gecko/20041208 Firefox/1.0+

.


Reproducible: Always
Steps to Reproduce:
Clean profile with javascript.options.showInConsole == true

1. Open a page in a second tab
2. Click on the current tab to ensure that the tab/label has the focus
3. Start typeahead find ('/') and close the tab immediately
4a. Start typeahead find ('/')
4b. Start Find (Accel+F) and enter something

Actual Results:  
Find fails.

3
  Error: document.commandDispatcher.focusedElement.parentNode has no properties
  Source File: chrome://global/content/findBar.js
  Line: 282
4a
  No reaction
4b
  Error: uncaught exception: [Exception... "Component returned failure code:
0x80004005 (NS_ERROR_FAILURE) [nsITypeAheadFind.find]"  nsresult: "0x80004005
(NS_ERROR_FAILURE)"  location: "JS frame :: chrome://global/content/findBar.js
:: find :: line 470"  data: no]


Expected Results:  
No errors.


Also happens with "Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8a6)
Gecko/20041208 Firefox/1.0+"

This seems to be a toolkit bug, but I can't file it there.

Comment 1

[Michael Clackler]

I can confirm this bug with Firefox 1.0+

The Severity of the bug needs to be changed to "critical" since the bug results
in a browser crash.

Comment 2

[Dorando]

Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:1.8b2) Gecko/20050309 Firefox/1.0+

Only the error on step 3 remains.

Comment 3

[Nickolay_Ponomarev]

see also bug 286885 / bug 273200

Comment 4

[Nickolay_Ponomarev]

confirming with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2)
Gecko/20050408 Firefox/1.0+

It's important to not miss the step two from original report. Also note that
it's different from bug 273200 in that the focus is not in FT (and the JS error
is different).

Comment 5

[Nickolay_Ponomarev]

*** Bug 286885 has been marked as a duplicate of this bug. ***

Comment 6

[Peter van der Woude [:Peter6]]

*** Bug 291010 has been marked as a duplicate of this bug. ***

Comment 7

[Thomas K. (:tom)]

That bug was not a dupe.  I saw this bug and noticed that it was not similar enough.

This bug is about find failing to find text in the tab, bug 291010 was about
failing to close the tab.

Comment 8

[Colin S]

Confirmed Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.8)
Gecko/20050511 Firefox/1.0.4. This is more serious than 'minor', it completely
breaks the Find function in the browser!

Comment 9

[Mike Connor [:mconnor]]

Can't reproduce even the error here on Linux or Windows trunk... minusing.

Comment 10

[Dorando]

Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:1.8b4) Gecko/20050715 Firefox/1.0+

I still get the error on step 3, but I need to close the tab with a middleclick now.

Comment 11

[Mike Connor [:mconnor]]

still can't reproduce, current trunk on multiple Windows machines, using the
reporter's latest set of instructions.  Gavin can't do it either.

Comment 12

[Dorando]

Alternative steps: (clean with javascript.options.showInConsole)
1. Open any link in a tab
2. Start typeahead with / or '
3. Middleclick on the current tab (there should be a focus ring)

Results in:
Error: document.commandDispatcher.focusedElement.parentNode has no properties
Source File: chrome://global/content/findBar.js
Line: 336

Error: document.commandDispatcher.focusedElement.parentNode has no properties
Source File: chrome://global/content/findBar.js
Line: 336

Comment 13

[Daniel C]

I can confirm on Linux-latest-trunk using steps in comment 12.

Middle-click a link, go to new tab, press "/", middle-click tab, open console,
view errors:

Error: document.commandDispatcher.focusedElement.parentNode has no properties
Source File: chrome://global/content/findBar.js
Line: 336

Error: document.commandDispatcher.focusedElement.parentNode has no properties
Source File: chrome://global/content/findBar.js
Line: 336

Comment 14

[Dorando]

Attachment: Checking if the focusedElement has a parentNode

Comment 15

[Mike Schroepfer]

Comment on attachment 192978 [details] [diff] [review]
Checking if the focusedElement has a parentNode

Per bug meeting - approved for 1.8b5 branch.

Comment 16

[:Gavin Sharp [email: [e-mail]]]

Seems like this was somehow forgotten, checked this in trunk&branch.

Trunk:
mozilla/toolkit/components/typeaheadfind/content/findBar.js; new revision: 1.28;

1.8 Branch:
mozilla/toolkit/components/typeaheadfind/content/findBar.js; new revision: 1.21.2.7;