1864164 - Perma toolkit/components/cookiebanners/test/browser/browser_bannerClicking.js | Cleanup function threw an exception - [Exception... "Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) when Gecko 121 merges to Beta on 2023-11-20

Status: VERIFIED FIXED

(Core :: Privacy: Anti-Tracking, defect, P1)

Description

[Cristian Tuns]

[Tracking Requested - why for this release]:

Central-as-beta simulation:
How to run these simulations:
Failure log

[task 2023-11-10T12:15:15.076Z] 12:15:15     INFO - TEST-PASS | toolkit/components/cookiebanners/test/browser/browser_bannerClicking.js | Skip click telemetry tests on linux. - 
[task 2023-11-10T12:15:15.076Z] 12:15:15     INFO - Leaving test bound test_embedded_iframe_pbm
[task 2023-11-10T12:15:15.076Z] 12:15:15     INFO - Buffered messages finished
[task 2023-11-10T12:15:15.077Z] 12:15:15     INFO - TEST-UNEXPECTED-FAIL | toolkit/components/cookiebanners/test/browser/browser_bannerClicking.js | Cleanup function threw an exception - [Exception... "Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsICookieBannerService.removeAllExecutedRecords]"  nsresult: "0x80040111 (NS_ERROR_NOT_AVAILABLE)"  location: "JS frame :: chrome://mochitests/content/browser/toolkit/components/cookiebanners/test/browser/head.js :: testSetup/< :: line 55"  data: no]
[task 2023-11-10T12:15:15.078Z] 12:15:15     INFO - GECKO(12099) | console.debug: CookieBannerListService: "shutdown"
[task 2023-11-10T12:15:15.078Z] 12:15:15     INFO - GECKO(12099) | console.debug: CookieBannerListService: "shutdown"
[task 2023-11-10T12:15:15.079Z] 12:15:15     INFO - GECKO(12099) | MEMORY STAT vsizeMaxContiguous not supported in this build configuration.
[task 2023-11-10T12:15:15.079Z] 12:15:15     INFO - GECKO(12099) | MEMORY STAT heapAllocated not supported in this build configuration.
[task 2023-11-10T12:15:15.080Z] 12:15:15     INFO - GECKO(12099) | MEMORY STAT | vsize 20983503MB | residentFast 1243MB
[task 2023-11-10T12:15:15.082Z] 12:15:15     INFO - TEST-OK | toolkit/components/cookiebanners/test/browser/browser_bannerClicking.js | took 44656ms

Hi, Tim! Can you please take a look at this? I think this might be caused by Bug 1854940.

Comment 1

[Tim Huang[:timhuang]]

Attachment: Bug 1864164 - Only clear execute records of cookie banner handling if the service is enabled during testing cleanup. r?pbz!

Comment 2

[Norisz Fay [:noriszfay]]

The first failure line seen in the log mentioned in the reporter comment above seems fixed however issues are still present as can be seen here. Failure log here. Could you please take a look? Thank you!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9d523f46c7d8
Only clear execute records of cookie banner handling if the service is enabled during testing cleanup. r=pbz

Comment 4

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/9d523f46c7d8

Comment 5

[Cristian Tuns]

The patch seems to be working in today's beta-simulation: https://treeherder.mozilla.org/jobs?repo=try&group_state=expanded&revision=1322dc18cd47862f2a21044d1ba455d1d9086ac3&searchStr=Linux%2C18.04%2Cx64%2CWebRender%2Ctsan%2Copt%2CMochitests%2Cwith%2Csoftware%2Cwebrender%2Cenabled%2Ctest-linux1804-64-tsan-qr%2Fopt-mochitest-browser-chrome-swr%2Cbc