-
Notifications
You must be signed in to change notification settings - Fork 864
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.
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
Remove check_includes = false for //brave/browser/extensions #25826
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ads LGTM
[puLL-Merge] - brave/brave-core@25826 Here's my review of the pull request: DescriptionThis PR makes significant changes to the Brave browser's codebase, primarily focusing on reorganizing and refactoring various components related to extensions, Tor, WebTorrent, and other Brave-specific features. The changes aim to improve code structure, reduce dependencies, and separate concerns. ChangesChanges
Possible Issues
Security HotspotsNo significant security issues were identified in this change. However, any changes to the Tor and WebTorrent components should be carefully reviewed to ensure they don't introduce any privacy or security vulnerabilities. Overall, this PR appears to be a substantial refactoring effort aimed at improving code organization and reducing unnecessary dependencies. While the changes seem well-intentioned, thorough testing across all affected components is crucial to ensure no regressions or unintended side effects are introduced. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rewards ++
Some new targets with check_includes = false have been created to minimize the amount of code where gn check does not apply.
72a9249
to
7d393fc
Compare
There are no functional changes and this fixes an intermittent build error so I'm going to admin merge |
Released in v1.72.64 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++
Resolves brave/brave-browser#10661
Resolves brave/brave-browser#41398
Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan: