Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ads] Refactor AdsTabHelper::MaybeNotifyTabTextContentDidChange/document?.body?.innerText #38796

Open
aseren opened this issue Jun 4, 2024 · 0 comments
Open

[ads] Refactor AdsTabHelper::MaybeNotifyTabTextContentDidChange/document?.body?.innerText #38796

aseren opened this issue Jun 4, 2024 · 0 comments
Labels
dev-concern feature/ads OS/Android Fixes related to Android browser functionality OS/Desktop perf

Comments

@aseren
Copy link

aseren commented Jun 4, 2024 •
edited
Loading

Platforms

Desktop/Android

Description

Refactor AdsTabHelper::MaybeNotifyTabTextContentDidChange/document?.body?.innerText to improve performance. See body_sniffer::BodyHandler or ai_chat_component.
@aseren aseren added perf dev-concern feature/ads OS/Android Fixes related to Android browser functionality OS/Desktop labels Jun 4, 2024
@aseren aseren self-assigned this Jun 4, 2024
@aseren aseren added priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/exclude labels Jun 4, 2024
This was referenced Jun 10, 2024
Open
Open
@tmancey tmancey added this to Ads Jun 10, 2024
@tmancey tmancey moved this to In progress in Ads Jun 10, 2024
@tmancey tmancey moved this from In progress to To Do in Ads Jun 11, 2024
@tmancey tmancey moved this from To Do to Backlog in Ads Sep 4, 2024
@tmancey tmancey removed priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/exclude labels Sep 4, 2024
@tmancey tmancey moved this from Dev Concern ❤️ to Backlog 🔜 in Ads Jan 17, 2025
@tmancey tmancey moved this from Backlog 🔜 to Performance 🚀 in Ads Jan 17, 2025
@tmancey tmancey moved this from Performance 🚀 to Dev Concern ❤️ in Ads Jan 17, 2025
@tmancey tmancey moved this from Dev Concern ❤️ to Backlog 🔜 in Ads Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dev-concern feature/ads OS/Android Fixes related to Android browser functionality OS/Desktop perf
Projects
Ads
Status: Backlog 🔜
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tmancey @aseren