element.select() does not trigger select event anymore.

RESOLVED FIXED

Status

()

P2
normal
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: arno, Assigned: enndeakin)

Tracking

({regression})

Trunk
x86
Linux
regression
Points:
---
Bug Flags:
blocking1.9.2 +
in-testsuite +

Firefox Tracking Flags

(status1.9.2 beta1-fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

10 years ago
Hi,
when selecting a text input with javascript (input.select()), it's select handler is not executed.
It works fine with firefox 3.0 and 3.5 but is broken in latest nightly.
(Reporter)

Comment 1

10 years ago
Created attachment 387413 [details]
testcase
Regression range would be very useful.
(Reporter)

Comment 3

10 years ago
I don't known how to get a regression range. 
I'm trying to get builds on ftp://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/
but it's a pain to find things: I'm trying to look in *-mozilla-central directories, but in most of them, there is no linux build.  Is there an easy way to get old xulrunner/firefox builds ?
nightly builds are usually used to find regression range.
Linux builds are typically in the same -XX-mozilla-central, but from time to time
the -XX- changes.

It is usually best to use binary search (or similar) to find regression range.
We know that 1.9.1 doesn't have the bug and 1.9.1 branched late November 08. 
So I'd start from a build around mid March 09.
Version: unspecified → Trunk
Sounds like a regression from the focus landing.
Blocks: 178324
Flags: blocking1.9.2?
(Assignee)

Comment 7

10 years ago
Created attachment 387493 [details] [diff] [review]
windows don't GetInterface to nsIDocShellTreeItem, so use GetPrivateRoot instead
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
(Assignee)

Comment 8

10 years ago
The code here is checking if the window is active and doesn't fire a select event if it is not. This was added by bug 92879 to avoid a focus stealing issue. I have a feeling that check may not be even needed anymore though. Perhaps we could simplify the code entirely.
(Assignee)

Comment 9

10 years ago
Created attachment 387538 [details] [diff] [review]
with working test
Attachment #387493 - Attachment is obsolete: true
Attachment #387538 - Flags: review?(bzbarsky)
Attachment #387538 - Flags: review?(bzbarsky) → review+
(Assignee)

Comment 10

10 years ago
http://hg.mozilla.org/mozilla-central/rev/57b1b4511f46
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
status1.9.2: --- → beta1-fixed
Flags: blocking1.9.2? → blocking1.9.2+
Priority: -- → P2
You need to log in before you can comment on or make changes to this bug.