1806740 - Expose hasRuleForBrowsingContext in GeckoView

Pushed by istorozhko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d31a2cd2fc3c
Add API to expose hasRuleForBrowsingContext r=geckoview-reviewers,amejiamarmol,calu

Cristian Tuns

Comment 3

•

1 year ago

Backed out for causing geckoview failures

Backout link
Push with failures
Failure Log
Failure line: TEST-UNEXPECTED-FAIL | org.mozilla.geckoview.test.GeckoSessionTestRuleTest#checkCookieBannerRuleForSession | java.lang.AssertionError: There should be a rule

Flags: needinfo?(bugzeeeeee)

Pulsebot

Comment 4

•

1 year ago

Pushed by istorozhko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/26c3fcaa9203
Add API to expose hasRuleForBrowsingContext r=geckoview-reviewers,amejiamarmol,calu

Norisz Fay [:noriszfay]

Comment 5

•

1 year ago

Backed out for causing Rules related gv-junit failures

Backout link

Push with failures

Failure log

Failure line: TEST-UNEXPECTED-FAIL | org.mozilla.geckoview.test.GeckoSessionTestRuleTest#checkCookieBannerRuleForSession | java.lang.AssertionError: There should be a rule

Pulsebot

Comment 6

•

1 year ago

Pushed by istorozhko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ec733bc94063
Add API to expose hasRuleForBrowsingContext r=geckoview-reviewers,amejiamarmol,calu

Sandor Molnar[:smolnar]

Comment 7

•

1 year ago

Backed out for causing for causing gv-junit failures.

Backout link: https://hg.mozilla.org/integration/autoland/rev/3387e4f266f095d429421e49529dd54c68262f62

Push with failures

Failure log: TEST-UNEXPECTED-FAIL | org.mozilla.geckoview.test.GeckoSessionTestRuleTest#checkCookieBannerRuleForSession | java.lang.AssertionError: There should be a rule

[:owlish] 🦉 PST

Assignee

Updated

•

1 year ago

Flags: needinfo?(bugzeeeeee)

Whiteboard: [cookie-banner] [geckoview:m111] → [cookie-banner] [geckoview:m111][geckoview:m112]

BugBot [:suhaib / :marco/ :calixte]

Comment 8

•

1 year ago

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:owlish, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.

Flags: needinfo?(calu)

Flags: needinfo?(bugzeeeeee)

:bclark

Updated

•

1 year ago

Whiteboard: [cookie-banner] [geckoview:m111][geckoview:m112] → [cookie-banner] [geckoview:m111][geckoview:m112] [fxdroid] [foundation]

Jira Integration Bot

Updated

•

1 year ago

See Also: → https://mozilla-hub.atlassian.net/browse/FXDROID-204

Pulsebot

Comment 9

•

1 year ago

Pushed by istorozhko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e3c8167a7df6
Add API to expose hasRuleForBrowsingContext r=geckoview-reviewers,amejiamarmol,calu

Cristian Tuns

Comment 10

•

1 year ago

Backed out for causing geckoview-junit-nofis failures in GeckoSessionTestRuleTest#checkCookieBannerRuleForSession

Backout link
Push with failures
Failure Log
Failure line: TEST-UNEXPECTED-FAIL | org.mozilla.geckoview.test.GeckoSessionTestRuleTest#checkCookieBannerRuleForSession | java.lang.AssertionError: There should be a rule

Cathy Lu [:calu]

Updated

•

1 year ago

Flags: needinfo?(calu)

Pulsebot

Comment 11

•

1 year ago

Pushed by istorozhko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7a0e9a32199b
Add API to expose hasRuleForBrowsingContext r=geckoview-reviewers,amejiamarmol,calu

Iulian Moraru

Comment 12

•

1 year ago

Backed out for causing gv-junit failures.

[task 2023-02-27T18:04:39.504Z] 18:04:39     INFO -  TEST-START | org.mozilla.geckoview.test.StorageControllerTest#clearDataFlags
[task 2023-02-27T18:04:43.465Z] 18:04:43     INFO -  org.mozilla.geckoview.test | INSTRUMENTATION_STATUS: numtests=1013
[task 2023-02-27T18:04:43.465Z] 18:04:43     INFO -  org.mozilla.geckoview.test | INSTRUMENTATION_STATUS: stream=
[task 2023-02-27T18:04:43.465Z] 18:04:43     INFO -  org.mozilla.geckoview.test | Error in clearDataFlags(org.mozilla.geckoview.test.StorageControllerTest):
[task 2023-02-27T18:04:43.465Z] 18:04:43     INFO -  org.mozilla.geckoview.test | java.lang.AssertionError: Cookie value should match
[task 2023-02-27T18:04:43.465Z] 18:04:43     INFO -  org.mozilla.geckoview.test | Expected: "ctx=test"
[task 2023-02-27T18:04:43.466Z] 18:04:43     INFO -  org.mozilla.geckoview.test |      but: was "foo=bar; ctx=test"
[task 2023-02-27T18:04:43.466Z] 18:04:43     INFO -  org.mozilla.geckoview.test | 	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
[task 2023-02-27T18:04:43.466Z] 18:04:43     INFO -  org.mozilla.geckoview.test | 	at org.junit.Assert.assertThat(Assert.java:956)
[task 2023-02-27T18:04:43.466Z] 18:04:43     INFO -  org.mozilla.geckoview.test | 	at org.junit.rules.ErrorCollector$1.call(ErrorCollector.java:65)
[task 2023-02-27T18:04:43.466Z] 18:04:43     INFO -  org.mozilla.geckoview.test | 	at org.junit.rules.ErrorCollector.checkSucceeds(ErrorCollector.java:78)
[task 2023-02-27T18:04:43.466Z] 18:04:43     INFO -  org.mozilla.geckoview.test | 	at org.junit.rules.ErrorCollector.checkThat(ErrorCollector.java:63)
[task 2023-02-27T18:04:43.467Z] 18:04:43     INFO -  org.mozilla.geckoview.test | 	at org.mozilla.geckoview.test.rule.GeckoSessionTestRule.checkThat(GeckoSessionTestRule.java:909)
[task 2023-02-27T18:04:43.467Z] 18:04:43     INFO -  org.mozilla.geckoview.test | 	at org.mozilla.geckoview.test.BaseSessionTest.assertThat(BaseSessionTest.kt:144)
[task 2023-02-27T18:04:43.467Z] 18:04:43     INFO -  org.mozilla.geckoview.test | 	at org.mozilla.geckoview.test.StorageControllerTest.clearDataFlags(StorageControllerTest.kt:117)
[task 2023-02-27T18:04:43.467Z] 18:04:43     INFO -  org.mozilla.geckoview.test |
<...>
[task 2023-02-27T18:04:44.096Z] 18:04:44     INFO -  02-27 18:04:42.743 D/GeckoViewConsole( 3471): onEvent GeckoView:WebExtension:PortMessageFromApp {"message":{"id":"bb0c5353-97d1-4fe7-b364-4e0b1f88e2d9","eval":"\n            document.cookie || 'null'\n        "}}
[task 2023-02-27T18:04:44.096Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): Error
[task 2023-02-27T18:04:44.096Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): java.lang.AssertionError: Cookie value should match
[task 2023-02-27T18:04:44.096Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): Expected: "ctx=test"
[task 2023-02-27T18:04:44.096Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471):      but: was "foo=bar; ctx=test"
[task 2023-02-27T18:04:44.097Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
[task 2023-02-27T18:04:44.097Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.junit.Assert.assertThat(Assert.java:956)
[task 2023-02-27T18:04:44.097Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.junit.rules.ErrorCollector$1.call(ErrorCollector.java:65)
[task 2023-02-27T18:04:44.097Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.junit.rules.ErrorCollector.checkSucceeds(ErrorCollector.java:78)
[task 2023-02-27T18:04:44.097Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.junit.rules.ErrorCollector.checkThat(ErrorCollector.java:63)
[task 2023-02-27T18:04:44.097Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.mozilla.geckoview.test.rule.GeckoSessionTestRule.checkThat(GeckoSessionTestRule.java:909)
[task 2023-02-27T18:04:44.098Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.mozilla.geckoview.test.BaseSessionTest.assertThat(BaseSessionTest.kt:144)
[task 2023-02-27T18:04:44.098Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.mozilla.geckoview.test.StorageControllerTest.clearDataFlags(StorageControllerTest.kt:117)
[task 2023-02-27T18:04:44.098Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at java.lang.reflect.Method.invoke(Native Method)
[task 2023-02-27T18:04:44.098Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
[task 2023-02-27T18:04:44.098Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
[task 2023-02-27T18:04:44.098Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
[task 2023-02-27T18:04:44.099Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
[task 2023-02-27T18:04:44.099Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.junit.rules.Verifier$1.evaluate(Verifier.java:35)
[task 2023-02-27T18:04:44.099Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.mozilla.geckoview.test.rule.GeckoSessionTestRule$2.lambda$evaluate$0$org-mozilla-geckoview-test-rule-GeckoSessionTestRule$2(GeckoSessionTestRule.java:1473)
[task 2023-02-27T18:04:44.099Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at org.mozilla.geckoview.test.rule.GeckoSessionTestRule$2$$ExternalSyntheticLambda0.run(D8$$SyntheticClass)
[task 2023-02-27T18:04:44.099Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:428)
[task 2023-02-27T18:04:44.099Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at java.util.concurrent.FutureTask.run(FutureTask.java:237)
[task 2023-02-27T18:04:44.100Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at android.app.Instrumentation$SyncRunnable.run(Instrumentation.java:1950)
[task 2023-02-27T18:04:44.100Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at android.os.Handler.handleCallback(Handler.java:751)
[task 2023-02-27T18:04:44.100Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at android.os.Handler.dispatchMessage(Handler.java:95)
[task 2023-02-27T18:04:44.100Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at android.os.Looper.loop(Looper.java:154)
[task 2023-02-27T18:04:44.100Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at android.app.ActivityThread.main(ActivityThread.java:6077)
[task 2023-02-27T18:04:44.100Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at java.lang.reflect.Method.invoke(Native Method)
[task 2023-02-27T18:04:44.101Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:866)
[task 2023-02-27T18:04:44.101Z] 18:04:44     INFO -  02-27 18:04:42.748 E/GeckoSessionTestRule( 3471): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:756)
[task 2023-02-27T18:04:44.101Z] 18:04:44     INFO -  02-27 18:04:42.750 D/GeckoViewContent( 3471): handleEvent: DOMWindowClose
[task 2023-02-27T18:04:44.101Z] 18:04:44     INFO -  02-27 18:04:42.752 D/GeckoViewConsole( 3471): onEvent GeckoView:WebExtension:List null
[task 2023-02-27T18:04:44.101Z] 18:04:44     INFO -  02-27 18:04:42.755 E/GeckoSessionTestRule( 3471): test_end c5ee677f-bc83-49bd-9e28-2d35f3d0f059 clearDataFlags(org.mozilla.geckoview.test.StorageControllerTest)
[task 2023-02-27T18:04:44.101Z] 18:04:44  WARNING -  TEST-UNEXPECTED-FAIL | org.mozilla.geckoview.test.StorageControllerTest#clearDataFlags | java.lang.AssertionError: Cookie value should match
[task 2023-02-27T18:04:44.101Z] 18:04:44     INFO -  TEST-INFO took 4570ms
[task 2023-02-27T18:04:44.102Z] 18:04:44     INFO -  org.mozilla.geckoview.test | INSTRUMENTATION_STATUS: numtests=1013
[task 2023-02-27T18:04:44.102Z] 18:04:44     INFO -  org.mozilla.geckoview.test | INSTRUMENTATION_STATUS: stream=
[task 2023-02-27T18:04:44.102Z] 18:04:44     INFO -  org.mozilla.geckoview.test | INSTRUMENTATION_STATUS: id=AndroidJUnitRunner
[task 2023-02-27T18:04:44.102Z] 18:04:44     INFO -  org.mozilla.geckoview.test | INSTRUMENTATION_STATUS: test=sessionContext
[task 2023-02-27T18:04:44.102Z] 18:04:44     INFO -  org.mozilla.geckoview.test | INSTRUMENTATION_STATUS: class=org.mozilla.geckoview.test.StorageControllerTest
[task 2023-02-27T18:04:44.102Z] 18:04:44     INFO -  org.mozilla.geckoview.test | INSTRUMENTATION_STATUS: current=766
[task 2023-02-27T18:04:44.102Z] 18:04:44     INFO -  org.mozilla.geckoview.test | INSTRUMENTATION_STATUS_CODE: 1
[task 2023-02-27T18:04:44.103Z] 18:04:44     INFO -  TEST-START | org.mozilla.geckoview.test.StorageControllerTest#sessionContext

Flags: needinfo?(bugzeeeeee)

Iulian Moraru

Updated

•

1 year ago

Flags: needinfo?(bugzeeeeee)

[:owlish] 🦉 PST

Assignee

Updated

•

1 year ago

Flags: needinfo?(bugzeeeeee)

Pulsebot

Comment 13

•

1 year ago

Pushed by istorozhko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bc3bdd8c19f8
Add API to expose hasRuleForBrowsingContext r=geckoview-reviewers,amejiamarmol,calu

Serban Stanca [:SerbanS]

Comment 14

•

1 year ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/bc3bdd8c19f8

Status: NEW → RESOLVED

Closed: 1 year ago

status-firefox112: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → 112 Branch

Chris Peterson [:cpeterson]

Comment 15

•

1 year ago

Irene, does this fix need to be uplifted to Beta 111 for the CBH experiment planned for 111?

status-firefox110: --- → wontfix

status-firefox111: --- → ?

Flags: needinfo?(bugzeeeeee)

[:owlish] 🦉 PST

Assignee

Comment 16

•

1 year ago

Chris, it looks like this API won't be needed for the experiment, so the answer is no.

Flags: needinfo?(bugzeeeeee)

Chris Peterson [:cpeterson]

Comment 17

•

1 year ago

Thanks. In that case, setting firefox111=wontfix.

status-firefox111: ? → wontfix