Closed Bug 1776167 Opened 2 years ago Closed 2 years ago

Frequent intermitent devtools/client/webconsole/test/browser/browser_console_enable_network_monitoring.js when Gecko 103 merges to Beta on 2022-06-27

Categories

(DevTools :: Console, defect)

defect

Tracking

(firefox-esr91 unaffected, firefox-esr102 unaffected, firefox101 unaffected, firefox102 unaffected, firefox103+ fixed, firefox104 verified)

VERIFIED FIXED
104 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox-esr102 --- unaffected
firefox101 --- unaffected
firefox102 --- unaffected
firefox103 + fixed
firefox104 --- verified

People

(Reporter: CrissCozmuta, Assigned: bomsy)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Central-as-beta simulation
How to run these simulations

Failure log

[task 2022-06-22T13:22:26.103Z] 13:22:26     INFO - TEST-START | devtools/client/webconsole/test/browser/browser_console_enable_network_monitoring.js
[task 2022-06-22T13:22:30.889Z] 13:22:30     INFO - GECKO(2772) | 2022-06-22 13:22:30.888 firefox[2772:48953] Persistent UI failed to open file file:///Users/cltbld/Library/Saved%20Application%20State/org.mozilla.firefox.savedState/window_1.data: No such file or directory (2)
[task 2022-06-22T13:29:57.163Z] 13:29:57     INFO - TEST-INFO | started process screencapture
[task 2022-06-22T13:29:57.285Z] 13:29:57     INFO - TEST-INFO | screencapture: exit 0
[task 2022-06-22T13:29:57.286Z] 13:29:57     INFO - Buffered messages logged at 13:22:26
[task 2022-06-22T13:29:57.286Z] 13:29:57     INFO - Entering test bound testEnableNetworkMonitoringInBrowserConsole
[task 2022-06-22T13:29:57.287Z] 13:29:57     INFO - Set the focus on the Browser Console
[task 2022-06-22T13:29:57.287Z] 13:29:57     INFO - Setting the netxhr category to checked
[task 2022-06-22T13:29:57.287Z] 13:29:57     INFO - Setting the net category to checked
[task 2022-06-22T13:29:57.288Z] 13:29:57     INFO - Console message: [JavaScript Warning: "The Web Console logging API (console.log, console.info, console.warn, console.error) has been disabled by a script on this page."]
[task 2022-06-22T13:29:57.288Z] 13:29:57     INFO - Check that the 'Enable Network Monitoring' setting is off by default
[task 2022-06-22T13:29:57.289Z] 13:29:57     INFO - TEST-PASS | devtools/client/webconsole/test/browser/browser_console_enable_network_monitoring.js | setting is disabled - 
[task 2022-06-22T13:29:57.289Z] 13:29:57     INFO - Checking that "test-image.png" was not logged
[task 2022-06-22T13:29:57.290Z] 13:29:57     INFO - Buffered messages logged at 13:22:54
[task 2022-06-22T13:29:57.290Z] 13:29:57     INFO - TEST-PASS | devtools/client/webconsole/test/browser/browser_console_enable_network_monitoring.js | Message "test-image.png" wasn't logged
[task 2022-06-22T13:29:57.290Z] 13:29:57     INFO -  - 
[task 2022-06-22T13:29:57.291Z] 13:29:57     INFO - Turn on network monitoring
[task 2022-06-22T13:29:57.291Z] 13:29:57     INFO - Buffered messages logged at 13:23:11
[task 2022-06-22T13:29:57.292Z] 13:29:57     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 9
[task 2022-06-22T13:29:57.292Z] 13:29:57     INFO - Buffered messages logged at 13:23:56
[task 2022-06-22T13:29:57.293Z] 13:29:57     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 8
[task 2022-06-22T13:29:57.293Z] 13:29:57     INFO - Buffered messages logged at 13:24:41
[task 2022-06-22T13:29:57.293Z] 13:29:57     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 7
[task 2022-06-22T13:29:57.294Z] 13:29:57     INFO - Buffered messages logged at 13:25:26
[task 2022-06-22T13:29:57.294Z] 13:29:57     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 6
[task 2022-06-22T13:29:57.294Z] 13:29:57     INFO - Buffered messages logged at 13:26:11
[task 2022-06-22T13:29:57.295Z] 13:29:57     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 5
[task 2022-06-22T13:29:57.295Z] 13:29:57     INFO - Buffered messages logged at 13:26:56
[task 2022-06-22T13:29:57.296Z] 13:29:57     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 4
[task 2022-06-22T13:29:57.296Z] 13:29:57     INFO - Buffered messages logged at 13:27:41
[task 2022-06-22T13:29:57.296Z] 13:29:57     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 3
[task 2022-06-22T13:29:57.297Z] 13:29:57     INFO - Buffered messages logged at 13:28:27
[task 2022-06-22T13:29:57.297Z] 13:29:57     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 2
[task 2022-06-22T13:29:57.297Z] 13:29:57     INFO - Buffered messages logged at 13:28:31
[task 2022-06-22T13:29:57.298Z] 13:29:57     INFO - Console message: [JavaScript Error: "Invalid ETag value "undefined"" {file: "resource://services-settings/SyncHistory.jsm" line: 52}]
[task 2022-06-22T13:29:57.298Z] 13:29:57     INFO - store@resource://services-settings/SyncHistory.jsm:52:13
[task 2022-06-22T13:29:57.298Z] 13:29:57     INFO - async*remoteSettingsFunction/remoteSettings.pollChanges@resource://services-settings/remote-settings.js:472:8
[task 2022-06-22T13:29:57.298Z] 13:29:57     INFO - async*notify@resource://services-settings/RemoteSettingsComponents.jsm:24:25
[task 2022-06-22T13:29:57.298Z] 13:29:57     INFO - TM_notify/<@resource://gre/modules/UpdateTimerManager.jsm:225:50
[task 2022-06-22T13:29:57.298Z] 13:29:57     INFO - TM_notify@resource://gre/modules/UpdateTimerManager.jsm:296:7
[task 2022-06-22T13:29:57.298Z] 13:29:57     INFO - 
[task 2022-06-22T13:29:57.299Z] 13:29:57     INFO - Buffered messages logged at 13:29:12
[task 2022-06-22T13:29:57.299Z] 13:29:57     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 1
[task 2022-06-22T13:29:57.299Z] 13:29:57     INFO - Buffered messages finished
[task 2022-06-22T13:29:57.300Z] 13:29:57     INFO - TEST-UNEXPECTED-FAIL | devtools/client/webconsole/test/browser/browser_console_enable_network_monitoring.js | Test timed out - 
[task 2022-06-22T13:29:57.300Z] 13:29:57     INFO - Clear the browser console output
[task 2022-06-22T13:29:57.300Z] 13:29:57     INFO - Browser console cleared
[task 2022-06-22T13:29:57.301Z] 13:29:57     INFO - Wait for all Browser Console targets to be attached
[task 2022-06-22T13:29:57.301Z] 13:29:57     INFO - Waiting 1 seconds.
[task 2022-06-22T13:29:57.302Z] 13:29:57     INFO - Close the Browser Console
[task 2022-06-22T13:29:57.302Z] 13:29:57     INFO - Browser Console closed
[task 2022-06-22T13:29:57.302Z] 13:29:57     INFO - GECKO(2772) | console.warn: "IGNORED REDUX ACTION:" "AUTOCOMPLETE_CLEAR"
[task 2022-06-22T13:29:57.303Z] 13:29:57     INFO - TEST-PASS | devtools/client/webconsole/test/browser/browser_console_enable_network_monitoring.js | The main process DevToolsServer has no pending connection when the test ends - 
[task 2022-06-22T13:29:57.303Z] 13:29:57     INFO - GECKO(2772) | MEMORY STAT | vsize 8105MB | residentFast 506MB | heapAllocated 147MB
[task 2022-06-22T13:29:57.304Z] 13:29:57     INFO - TEST-OK | devtools/client/webconsole/test/browser/browser_console_enable_network_monitoring.js | took 451098ms

Hi Hubert,

This seems to be related to the landing of https://bugzilla.mozilla.org/show_bug.cgi?id=1771249
It first appeared on the beta-sim from 21-June here
Can you please take a look?

Thank you!

Flags: needinfo?(hmanilla)
Flags: needinfo?(hmanilla)
Summary: Frequent intermitent devtools/client/webconsole/test/browser/browser_console_enable_network_monitoring.js Gecko 103 merges to Beta on 2022-06-27 → Frequent intermitent devtools/client/webconsole/test/browser/browser_console_enable_network_monitoring.js when Gecko 103 merges to Beta on 2022-06-27

Hi Cosmin,
Thanks i'm taking a look.

Flags: needinfo?(hmanilla)
Assignee: nobody → hmanilla

browser_console_enable_network_monitoring.js is failing on beta because MBT is not enabled by default
so the the leagcy-listener copepath is used. This adds chromeContext to the network resource and fixes
the issue.

Patch is working in today's beta-sim.

Pushed by hmanilla@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3661ef687ece [devtools] Fix chromeContext for the legacy-listeners codepath r=ochameau
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 104 Branch

The patch was verified in today's beta sim

Status: RESOLVED → VERIFIED

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

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

For more information, please visit auto_nag documentation.

Flags: needinfo?(hmanilla)

Comment on attachment 9282936 [details]
Bug 1776167 - [devtools] Fix chromeContext for the legacy-listeners codepath r=ochameau

Beta/Release Uplift Approval Request

  • User impact if declined: Developers using the non-multiprocess browser toolbox, will see all network logs as content log messages.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Just adds the chromeContext property, not risky
  • String changes made/needed:
  • Is Android affected?: No
Flags: needinfo?(hmanilla)
Attachment #9282936 - Flags: approval-mozilla-beta?

Comment on attachment 9282936 [details]
Bug 1776167 - [devtools] Fix chromeContext for the legacy-listeners codepath r=ochameau

Approved for 103.0b3, thanks.

Attachment #9282936 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: