mozilla hangs when doing Shift+Tab over an input field

VERIFIED FIXED

Status

()

--
critical
VERIFIED FIXED
15 years ago
15 years ago

People

(Reporter: kreimer, Unassigned)

Tracking

(4 keywords)

Trunk
x86
Windows 2000
crash, regression, testcase, verified1.7
Points:
---
Bug Flags:
blocking1.7 +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: patch ready to land, fixed-aviary1.0, URL)

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

15 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a) Gecko/20040511
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a) Gecko/20040511

1) visit http://de.yahoo.com (I've found the same in http://es.yahoo.com)
2) focus the input field labeled as "Suche im Web"
3) Press Shift+Tab
, then mozilla hangs

Reproducible: Always
Steps to Reproduce:
1. visit http://de.yahoo.com (I've found the same in http://es.yahoo.com)
2. focus the input field labeled as "Suche im Web"
3. press Shift+Tab

Actual Results:  
mozilla hangs

Expected Results:  
to give focus to the previous focusable component at de.yahoo.com

Comment 1

15 years ago
Confirming, same problem here, using :
Gecko/20040509 Firefox/0.8.0+

Comment 2

15 years ago
Created attachment 148292 [details]
reduced testcase

body reduced to about 10 lines html, style deleted, original JS retained.

No onload event, so no initial focus.

Comment 3

15 years ago
confirming Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8a) Gecko/20040511

Bug is only seen if JS is enabled, without JS reverse tabbing is working.

to get a TalkbackID I had to use Mozilla 1.7b, TalkBackID: TB45040K
After SHiftTab the cursor started flickering, after about 10 seconds Mozilla
crashed.
Assignee: general → general
Status: UNCONFIRMED → NEW
Component: Browser-General → JavaScript Engine
Ever confirmed: true
Keywords: crash, testcase
QA Contact: general → pschwartau

Updated

15 years ago
Whiteboard: DUPEME

Comment 4

15 years ago
I have thought of Bug 229908 here, but it seems this has been fixed. Anyway can
someone check the stacktrace against of that Bug against the TB stacktrace from
this bug then?
Whiteboard: DUPEME

Comment 5

15 years ago
Created attachment 148347 [details]
html mimimized testcase

minimal html, reduced js, 
doesn´t hang in opera, but cycles only 3 positions.

<div id=srchbx><form name=sf1 ><input name=sub type=submit
value="Suche"></form></div>
Attachment #148292 - Attachment is obsolete: true

Comment 6

15 years ago
100% crash/hang bug with testcase reproductible on 1.7 branch 20040513 build
Flags: blocking1.7?

Comment 7

15 years ago
(In reply to comment #6)
> 100% crash/hang bug with testcase reproductible on 1.7 branch 20040513 build

Do you see this on Windows, or another OS?
Is Talkback installed, and does it come up after a crash?
I can´t reproduce the crash with current nightlies on Win98/96MB RAM/Celeron333
and Win98SE/512MB/AthlonXP1600+. I get a hang, spinning cursor, which lasts for
some minutes, then Mozilla responds again. While hanging, I can switch windows
applications, Mozilla doesn´t respond, but windows works. If I start the
testcase on low RAM on the slow computer, I can get a freeze after some minutes,
windows doesn´t respond anymore.

If you can reproduce this bug, please enable Talkback, and post the Talkback ID
here. On windows, you get the ID, if you start talkback.exe from the
Mozilla/components directory.

1.7b is old, and there have been a lot of changes, it would be fine if there
could be a talkback from a current version.

Talkback of crash with 1.7b release:
http://talkback-public.mozilla.org/talkback/fastfind.jsp?search=2&type=iid&id=TB45040K

Excerpt:

Trigger Reason	Stack overflow

c:/builds/tinderbox/Mozilla1.7b/WINNT_5.0_Clobber/mozilla/content/xml/document/src/nsXMLDocument.cpp
Trigger Line No.	208
Stack Trace 	
nsXMLDocument::QueryInterface
[c:/builds/tinderbox/Mozilla1.7b/WINNT_5.0_Clobber/mozilla/content/xml/document/src/nsXMLDocument.cpp,
line 208]
nsXULDocument::QueryInterface
...

And link to the source:
http://webtools.mozilla.org/bonsai/cvsblame.cgi?file=mozilla/content/xml/document/src/nsXMLDocument.cpp&rev=&root=/cvsroot

Comment 8

15 years ago
(In reply to comment #7)
> Do you see this on Windows, or another OS?
Windows 2003 1.7 branch 20040513

> Is Talkback installed, and does it come up after a crash?
Sorry I didn't wait enough. I've just tried again and I have no crash only hang
for several minutes with 100% CPU and mozilla unusable (can't open any more
windows). After that mozilla is ok again and I can post to this bug for instance.

Comment 9

15 years ago
regressed between Mozilla 1.3a (BuildID 2002121304, working) 
and Mozilla 1.3b (BuildID 2003021005, failing)
Didn´t find files inbetween in http://archive.mozilla.org
Phoenix 0.5 BuildID 20021228 based on Mozilla 1.3a is working.

Is this bug windows only?

TalkbackID TB45040K Mozilla 1.7b 

http://talkback-public.mozilla.org/talkback/fastfind.jsp?search=2&type=iid&id=TB45040K
Keywords: regression
Not a JS bug, over to event handling.
Assignee: general → events
Component: JavaScript Engine → Event Handling
QA Contact: pschwartau → ian
Created attachment 149381 [details] [diff] [review]
Don't mess with focus changes done by the page when tabbing around.

Updated

15 years ago
Attachment #149381 - Flags: superreview?(bryner)
Attachment #149381 - Flags: review?(bryner)
Attachment #149381 - Flags: superreview?(bryner)
Attachment #149381 - Flags: superreview+
Attachment #149381 - Flags: review?(bryner)
Attachment #149381 - Flags: review+

Updated

15 years ago
Attachment #149381 - Flags: approval1.7?

Comment 12

15 years ago
Comment on attachment 149381 [details] [diff] [review]
Don't mess with focus changes done by the page when tabbing around.

a=chofmann for 1.7
Attachment #149381 - Flags: approval1.7? → approval1.7+

Updated

15 years ago
Flags: blocking1.7? → blocking1.7+

Updated

15 years ago
Whiteboard: patch ready to land
Fixed on both the trunk and the 1.7 branch.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Keywords: fixed1.7
Resolution: --- → FIXED
fixed on aviary branch as well
Whiteboard: patch ready to land → patch ready to land, fixed-aviary1.0
verified
Status: RESOLVED → VERIFIED
Keywords: fixed1.7 → verified1.7
You need to log in before you can comment on or make changes to this bug.