Closed Bug 20148 Opened 25 years ago Closed 25 years ago

[dogfood] cannot set focus in text widget

Categories

(Core :: DOM: Core & HTML, defect, P3)

Product:
Core
Component:
DOM: Core & HTML
Platform:
x86
Other
Type:
defect

Tracking

()

Status:
VERIFIED WORKSFORME

People

(Reporter: asa, Assigned: saari)

References

(
URL
)

Details

(Whiteboard: [PDT+] Can't repro on Win and Linux, working on mac build) 

win32 and linux build 11/26/99
The search form at google.com will not get a caret when you place the mouse
pointer in it and click, nor can you tab to that field.
Similar thing with YahooMail (http://mail.yahoo.com/), can't enter any text in
the login field, but you can in the password field.

<:3)~~
Assignee: karnaze → buster
Reassigning to Steve.
Assignee: buster → kin
Summary: no caret in search form at google.com → cannot set focus in text widget
using build 1999112908 on win95:
1. open browser
2. enter http:www.google.com, wait for page to load
3. attempt to set focus within the search text widget

text widget does not accept focus
I did some basic troubleshooting of the page -- what I found was this. There is
a SCRIPT element used within the head
<SCRIPT LANGUAGE="JavaScript">
<!--
document.f.q.focus();
// -->
</SCRIPT>

If this is placed after the form, then the selection cannot be made. If this is
placed before the form, I can select within the form element. I talked with
rickg and I need to check this in viewer and look at the debug dump content to
see if there are any differences when I place the <SCRIPT> before and after the
form element.
*** Bug 20282 has been marked as a duplicate of this bug. ***
Assignee: kin → rickg
here is the results of the dump:

<html>
<body>
<SCRIPT LANGUAGE="JavaScript">
<!--
document.f.q.focus();
// -->
</SCRIPT>
<form name=f>
<input type=text name=q>
</form>
</body>
</html>


Before:
=======

webshell=015C2580
html@02A9366C refcount=3<
  head@02A935EC refcount=2<
  >
  Text@02A3D270 refcount=3<\n>
  body@02A3D1FC refcount=3<
    Text@02A5A340 refcount=3<\n>
    script@02A5F09C language=JavaScript refcount=2<
      Text@02A5CB60 refcount=2<\r\n<!--\r\ndocument.f.q.focus();\r\n//
-->\r\n>
    >
    Text@02A8EE70 refcount=3<\n>
    form@02A8E9C0 name=f refcount=4<
      Text@02A8E740 refcount=3<\n>
      input@02A8E67C type=Text name=q refcount=7<>
      Text@02A8E320 refcount=3<\n>
    >
    Text@02A8E270 refcount=3<\n>
  >
>


AFTER:
======

webshell=015C2580
html@02A5C1EC refcount=4<
  head@02A5C16C refcount=2<
  >
  Text@0292AB30 refcount=3<\n>
  body@0292EFAC refcount=4<
    Text@029C8AD0 refcount=3<\n>
    form@029C8190 name=f refcount=5<
      Text@029C9F00 refcount=3<\n>
      input@029C9E3C type=Text name=q refcount=7<>
      Text@029C9B20 refcount=3<\n>
    >
    Text@029C9A40 refcount=3<\n>
    script@029C99BC language=JavaScript refcount=2<
      Text@029C9700 refcount=2<\r\n<!--\r\ndocument.f.q.focus();\r\n//
-->\r\n>
    >
    Text@02A908C0 refcount=3<\n>
  >
>

Rick -- what do you think?
Assignee: rickg → pollmann
Eric -- this one doesn't make sense to me. If there were any problem, I'd expect
that the script coming before the form to be a problem. Nonetheless, please take
a look.
Assignee: pollmann → saari
This is a duplaicate of a saari bug - reassigning
*** Bug 20381 has been marked as a duplicate of this bug. ***
*** Bug 20479 has been marked as a duplicate of this bug. ***
*** Bug 20372 has been marked as a duplicate of this bug. ***
Summary: cannot set focus in text widget → [dogfood] cannot set focus in text widget
Phonebook also fails. putting on dogfood radar
Whiteboard: [PDT+]
Putting on PDT+ radar...but we are possitive there is a dup out there already on
PDT+...anyone know it?
saari--

So is this the result of the security problem you mentioned? Should this be
reassigned to me?
QA Contact: cpratt → gerardok
Blame it on the tryptophan. This bug was first introduced with the
1999112508 build, on Turkey Day. I reproduced it on that days Linux build and
it shows up in every Win32 and Linux build thereafter(but not before). I also
note the problem in today's (1999120108) Mac build.

changing QA contact to gerardok so he can reassign to proper team member
Blocks: 19207
*** Bug 20172 has been marked as a duplicate of this bug. ***
norris, if you can fix your bug, I'm pretty sure this will get better.

However, focus should still not get wedged, so I'm working on a fix for that.
Whiteboard: [PDT+] → [PDT+] Can't repro on Win and Linux, working on mac build
This disappeared late tonight on both my linux and windows boxen. Confirmed by
mcafee.

My mac build is being difficult, but it is pretty hard to fix this if I cannot
repro it.
I checked in a fix yesterday that resolved a security problem that was
problably the one you were seeing. I was able to do a search at Google today.
great detective work, this was a tough one!
added a few of the URLs from dup bugs.  be sure to test each of these to make
sure the bug is really gone before declaring victory.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → WORKSFORME
Bug looks to be really gone, even on Mac. Marking worksforme

Thanks norris!
Status: RESOLVED → VERIFIED
http://pb works fine with the 1999121508 build on Linux. Marking verified. (Bug
20488 provided the verification method.)
*** Bug 20488 has been marked as a duplicate of this bug. ***
Verified on 1999-12-15-08-M12 on Mac, and 1999-12-10-09-M12 on Win NT.
Component: HTML: Form Submission → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.