Closed Bug 1751674 Opened 2 years ago Closed 2 years ago

HTML Select not working properly after 1750395

Categories

(GeckoView :: General, defect)

Unspecified
All
defect

Tracking

(firefox-esr91 unaffected, firefox96 unaffected, firefox97 unaffected, firefox98blocking fixed)

RESOLVED FIXED
98 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox96 --- unaffected
firefox97 --- unaffected
firefox98 blocking fixed

People

(Reporter: anastasiostam, Assigned: emilio)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

_handleselect function in GeckoViewPrompt.jsm expects the target, however the parent event is passed from handleEvent.

This is a major regression as HTML Select entities in Fenix are not working properly

See https://github.com/mozilla-mobile/fenix/issues/23370

Regressions: 1750395
Regressed by: 1750395
No longer regressions: 1750395

I will try to make a patch for this

Set release status flags based on info from the regressing bug 1750395

Has Regression Range: --- → yes

Emilio it looks like this is a regression from one of your patches. Could you take a look please?

Flags: needinfo?(emilio)

Ugh, so we have zero automated test coverage for this? That's unfortunate, that's why I get for only testing that patch via try (I was traveling and couldn't do a full android build). Fixing :/

Assignee: nobody → emilio
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(emilio)

We had zero coverage for this, apparently. Upon writing tests, the only
test we had for this was @Ignored :(

Write better tests for this. Add tests for <select multiple> which I
broke because the mozshowdropdown events only fire for dropdown selects.

I'm sorry anyways, I should've tested manually, but I was traveling and
found issues when trying to do an artifact build so I relied on try, oh
well.

Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a5833843c7c8
Fix a couple issues with my patch for bug 1750395. r=agi
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch
Flags: in-testsuite+

Hello Emilio,
I tested this on the latest nightly Fenix and noticed the following:

  1. The first click-select works fine.
  2. The second time you click the "html select" it does nothing (it should have opened again the select popup)
    Can you check if that is related to this bug?
    It worked before on my installation of Fenix stable 96.0.2

(I checked with https://www.w3schools.com/tags/tryit.asp?filename=tryhtml_select )

Flags: needinfo?(emilio)
Blocks: 1752133

Thanks, bug 1752133 should fix that and another pre-existing issue.

Flags: needinfo?(emilio)
Blocks: 1752523
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: