Skip to content

Conversation

CrisBarreiro
Copy link
Contributor

@CrisBarreiro CrisBarreiro commented Aug 20, 2025

Task/Issue URL: https://app.asana.com/1/137249556945/project/1205008441501016/task/1211092095433618?focus=true

Description

C-S-S sends us a ping message so we have a reference to a reply proxy that we store and use to send messages after a user action is initiated on the native side

Steps to test this PR

Pre-requisites

Desktop mode

  • Load wikipedia.org
  • Open menu
  • Switch to desktop mode
  • Check desktop mode is loaded

Breakage reporting

  • Load a page
  • Open menu
  • Check subscription with name getBreakageReportValues is sent, and message with method breakageReportResult is received

Disable protections

  • Load http://privacy-test-pages.site/privacy-protections/gpc/
  • Click start test
  • Check:
    • top frame header - "1"
    • top frame JS API - true
    • frame JS API - true
  • Open menu
  • Click Disable Privacy Protection
  • Click start test
  • Check:
    • top frame header - ...
    • top frame JS API - ...
    • frame JS API - ...

UI changes

n/a

Copy link
Contributor Author

CrisBarreiro commented Aug 20, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@CrisBarreiro CrisBarreiro mentioned this pull request Aug 20, 2025
8 tasks
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from 49789e6 to b21ac79 Compare August 20, 2025 16:25
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch 2 times, most recently from a876d17 to cf0b7c9 Compare August 21, 2025 08:56
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch 2 times, most recently from 8b6def6 to 04a6002 Compare August 21, 2025 09:20
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch 2 times, most recently from 4419a8a to f688897 Compare August 21, 2025 10:03
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from 04a6002 to 168009b Compare August 21, 2025 10:03
@CrisBarreiro CrisBarreiro mentioned this pull request Aug 22, 2025
3 tasks
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from f688897 to 551fa17 Compare August 25, 2025 16:06
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch 2 times, most recently from 4441402 to 02ad05a Compare August 26, 2025 08:23
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 551fa17 to fd8bc1f Compare August 26, 2025 08:23
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from 02ad05a to a5c2c09 Compare August 26, 2025 09:07
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from fd8bc1f to 59f0b6a Compare August 26, 2025 09:07
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch 2 times, most recently from a92db0f to 0aa6479 Compare August 26, 2025 10:54
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 9a234fe to 2a16166 Compare August 27, 2025 08:07
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from 37a86a6 to 8d30eb3 Compare August 27, 2025 08:07
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 2a16166 to c10bb68 Compare August 28, 2025 15:52
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch 2 times, most recently from 512911d to 6b8f7c6 Compare August 29, 2025 07:56
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch 2 times, most recently from 8a684d1 to 377ea75 Compare August 29, 2025 08:15
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from 6b8f7c6 to 8104fea Compare August 29, 2025 08:15
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 377ea75 to 66684c8 Compare August 29, 2025 08:26
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from 40387e5 to 6bed82c Compare September 2, 2025 14:34
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 2ab4490 to 26a4ca4 Compare September 2, 2025 15:05
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch 2 times, most recently from b2a391f to daf105e Compare September 2, 2025 15:33
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 26a4ca4 to 97c6987 Compare September 2, 2025 15:33
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from daf105e to 8414617 Compare September 2, 2025 16:02
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 97c6987 to 2c7ad20 Compare September 2, 2025 16:02
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from 8414617 to 2aab049 Compare September 2, 2025 16:04
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch 2 times, most recently from 8c862b7 to 1fb4b2a Compare September 2, 2025 16:05
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch 2 times, most recently from 7060075 to 95ab626 Compare September 2, 2025 16:55
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 1fb4b2a to 983a564 Compare September 2, 2025 16:55
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from 467e1ca to e887080 Compare September 2, 2025 18:30
@CrisBarreiro CrisBarreiro marked this pull request as ready for review September 2, 2025 18:33
@marcosholgado marcosholgado self-assigned this Sep 3, 2025
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 983a564 to 7c51bf2 Compare September 3, 2025 10:57
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from 8ba6090 to 5e404c4 Compare September 3, 2025 10:57
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 7c51bf2 to 9ced48c Compare September 3, 2025 14:18
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch 2 times, most recently from bd84ff0 to 015c15d Compare September 3, 2025 15:56
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 9ced48c to 2f89eea Compare September 3, 2025 15:56
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/add-subscriptions branch from 015c15d to 59077ce Compare September 3, 2025 15:59
@CrisBarreiro CrisBarreiro force-pushed the cris/adsjs/support-messaging-responses branch from 2f89eea to 3a8cf2d Compare September 3, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants