Closed Bug 1815819 Opened 2 years ago Closed 2 months ago

Omnibox keywords can be activated in search mode

Categories

(Firefox :: Address Bar, defect, P2)

Firefox 111
defect

Tracking

()

VERIFIED FIXED
130 Branch
Tracking Status
firefox111 --- wontfix
firefox130 --- verified

People

(Reporter: gregp, Assigned: yazan)

References

Details

(Keywords: papercut, Whiteboard: [snt-scrubbed][search-papercut])

Attachments

(2 files)

Attached image omnibox.png

Steps to reproduce:

  1. Navigate to https://github.com/gregorypappas/onDeleteSuggestion
  2. Download and unzip the extension
  3. Navigate to about:debugging -> This Firefox -> Temporary Extensions
  4. Select 'Load Extension' and install the unzipped extension from step 4
  5. Type @google omni 123 in the address bar

Actual results:
The extension's keyword is activated and the extension receives events

Expected results:
The extension's keyword does not activate.

This isn't intended behavior, right? It feels very broken.

The severity field is not set for this bug.
:adw, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(adw)

It's not expected indeed, in search mode the omnibox provider should not even run :(

Severity: -- → S3
Flags: needinfo?(adw)
Keywords: papercut
Priority: -- → P2
Whiteboard: [snt-scrubbed][search-papercut]
OS: Linux → All
Hardware: x86_64 → All
Duplicate of this bug: 1796788
Assignee: nobody → yalmacki
Pushed by jteow@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e0b29efee642
Set omnibox provider isActive to false when on search mode. r=jteow,urlbar-reviewers
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 130 Branch

Reproducible on a 2024-07-09 Nightly build on Windows 10 using the steps from Comment 0.
Verified as fixed on Firefox Nightly 130.0a1 on Windows 10, Ubuntu 22, macOS 14.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: