Closed Bug 1804213 Opened 1 year ago Closed 1 year ago

Network ID table is included in <a> element

Categories

(Core :: XPCOM, defect)

Firefox 108
defect

Tracking

()

VERIFIED FIXED
109 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox107 --- unaffected
firefox108 --- verified
firefox109 --- verified

People

(Reporter: 6k64x4ma, Assigned: padenot)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Attached image screenshot.webp

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:108.0) Gecko/20100101 Firefox/108.0

Steps to reproduce:

  1. Go to about:networking#networkid

Actual results:

The table is unnecessarily included in the <a> element linking to about:logging.

Expected results:

This is regressed by Bug 1347955.

Regressed by: 1347955

:padenot, since you are the author of the regressor, bug 1347955, could you take a look? Also, could you set the severity field?

For more information, please visit auto_nag documentation.

Flags: needinfo?(padenot)
Assignee: nobody → padenot
Flags: needinfo?(padenot)
Attachment #9306836 - Attachment description: WIP: Bug 1804213 - Close <a> in about:networking#networkid. → Bug 1804213 - Close <a> in about:networking#networkid.
Pushed by padenot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fa1cdaa8a32c
Close <a> in about:networking#networkid.
Status: UNCONFIRMED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch

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

The patch landed in nightly and beta is affected.
:padenot, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox108 to wontfix.

For more information, please visit auto_nag documentation.

Flags: needinfo?(padenot)

Comment on attachment 9306836 [details]
Bug 1804213 - Close <a> in about:networking#networkid.

Beta/Release Uplift Approval Request

  • User impact if declined: Bad rendering in about:networking.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce: Open about:networking#networkid, if this reproduces, the table is a link.
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Trivial markup fix.
  • String changes made/needed: none
  • Is Android affected?: Yes
Flags: needinfo?(padenot)
Attachment #9306836 - Flags: approval-mozilla-beta?

Comment on attachment 9306836 [details]
Bug 1804213 - Close <a> in about:networking#networkid.

Switching to Release since there are no more betas for 108 and we are in RC week

Attachment #9306836 - Flags: approval-mozilla-beta? → approval-mozilla-release?

Comment on attachment 9306836 [details]
Bug 1804213 - Close <a> in about:networking#networkid.

Approved for 108.0rc2

Attachment #9306836 - Flags: approval-mozilla-release? → approval-mozilla-release+
Flags: qe-verify+

Reproduced this issue using Firefox 109.0a1 (BuildId:20221206164509) on Windows 10 64bit.

This issue is verified fixed using Firefox 109.0a1 (BuildId:20221209044848) and Firefox 108.0rc2 (BuildId:20221208122842) on Windows 10 64bit, macOS 11 & Ubuntu 22.04

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

Attachment

General

Creator:
Created:
Updated:
Size: