Make gab@ OWNER of browser_main_loop* and browser_thread*
Motivation: In the light of a few problematic CLs that went by without me being brought in, I'd like to make it explicit that I should be involved in such reviews per having played a lot in the tricky initialization/teardown logic at stake here. e.g. https://chromium-review.googlesource.com/c/chromium/src/+/905424#message-694a8d54856d52017b154097a1475dc21c90bb1a - Proposed fixed : https://chromium-review.googlesource.com/c/chromium/src/+/964606 - Ultimate fix : https://chromium-review.googlesource.com/c/chromium/src/+/969104 https://chromium-review.googlesource.com/c/chromium/src/+/941264 - Cleaned up : https://chromium-review.googlesource.com/c/chromium/src/+/961451/4 - Further cleaned up : https://chromium-review.googlesource.com/c/chromium/src/+/963361 R=jam@chromium.org Bug: None Change-Id: I2db8fae7763e5ad12da4b0a5f6556738e1cdb6f6 Reviewed-on: https://chromium-review.googlesource.com/964617 Commit-Queue: Gabriel Charette <gab@chromium.org> Reviewed-by: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#544448}
This commit is contained in:

committed by
Commit Bot

parent
ed0039350e
commit
e0b331d68b
@ -17,6 +17,10 @@ piman@chromium.org
|
||||
# structural changes, please get a review from a reviewer in this file.
|
||||
per-file BUILD.gn=*
|
||||
|
||||
# For threading (BrowserThread, BrowserMainLoop, TaskScheduler, etc.)
|
||||
per-file *browser_main_loop*=gab@chromium.org
|
||||
per-file *browser_thread*=gab@chromium.org
|
||||
|
||||
# Mac Sandbox profiles.
|
||||
per-file *.sb=set noparent
|
||||
per-file *.sb=rsesek@chromium.org
|
||||
|
Reference in New Issue
Block a user