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 iOS Tab::reportPageLoad/document?.body?.innerText #38500

Open
Tracked by #33888
tmancey opened this issue May 22, 2024 · 0 comments
Open
Tracked by #33888

[ads] Refactor iOS Tab::reportPageLoad/document?.body?.innerText #38500

tmancey opened this issue May 22, 2024 · 0 comments
Labels
dev-concern feature/ads OS/iOS Fixes related to iOS browser functionality perf

Comments

@tmancey
Copy link
Contributor

tmancey commented May 22, 2024 •
edited by aseren
Loading

Platforms

iOS

Description

Refactor Tab::reportPageLoad /document?.body?.innerText to improve performance.
@tmancey tmancey added priority/P3 The next thing for us to work on. It'll ride the trains. perf dev-concern QA/Yes release-notes/exclude feature/ads labels May 22, 2024
@tmancey tmancey added OS/Android Fixes related to Android browser functionality OS/Desktop OS/iOS Fixes related to iOS browser functionality labels May 22, 2024
@aseren aseren removed the OS/iOS Fixes related to iOS browser functionality label Jun 4, 2024
@aseren aseren changed the title [ads] Refactor AdsTabHelper::MaybeNotifyTabTextContentDidChange/document?.body?.innerText [ads] Refactor Tab::reportPageLoad/document?.body?.innerText on iOS Jun 4, 2024
@aseren aseren added OS/iOS Fixes related to iOS browser functionality and removed OS/Android Fixes related to Android browser functionality OS/Desktop labels Jun 4, 2024
@aseren aseren added needs-investigation A bug not 100% confirmed/fixed and removed QA/Yes release-notes/exclude needs-investigation A bug not 100% confirmed/fixed priority/P3 The next thing for us to work on. It'll ride the trains. labels Jun 4, 2024
@aseren aseren removed their assignment Jun 4, 2024
@tmancey tmancey mentioned this issue Jun 10, 2024
Open
@tmancey tmancey added priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/exclude labels Jun 10, 2024
@tmancey tmancey mentioned this issue Jun 10, 2024
Open
10 tasks
@tmancey tmancey changed the title [ads] Refactor Tab::reportPageLoad/document?.body?.innerText on iOS [ads] Refactor iOS Tab::reportPageLoad/document?.body?.innerText Jun 10, 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
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/iOS Fixes related to iOS browser functionality perf
Projects
Ads
Status: Dev Concern ❤️
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tmancey @aseren