On 2017/04/18 05:10:57, foolip OOO on Wednesday wrote:
> How about mailto:garykac@chromium.org and mailto:jamiewalch@chromium.org? As
editors of the
> spec, they should be the best reviewers of this code after you.
modules/ have serious code health issues and we've been trying not to add
persons to OWNERS until they get familiar with Blink's implementation. (I should
document this somewhere.)
Hzj_jie
2017/04/19 00:45:55
I am happy to entirely remove this OWNER file, if
On 2017/04/18 07:38:36, haraken wrote:
> On 2017/04/18 05:10:57, foolip OOO on Wednesday wrote:
> > How about mailto:garykac@chromium.org and mailto:jamiewalch@chromium.org? As
> editors of the
> > spec, they should be the best reviewers of this code after you.
>
> modules/ have serious code health issues and we've been trying not to add
> persons to OWNERS until they get familiar with Blink's implementation. (I
should
> document this somewhere.)
I am happy to entirely remove this OWNER file, if it's not a must-have. I see 74
folders under modules, but only 66 of them have OWNERS file.
Otherwise, I am also OK to add GaryKac@ and JamieWalch@ to it, though the
COMPONENT: Services>Chromoting has covered them already.
Issue 2805763004: [System-Keyboard-Lock] Forward navigator functions to RenderFrameHost
(Closed)
Created 3 years, 8 months ago by Hzj_jie
Modified 3 years, 7 months ago
Reviewers: whywhat, Avi (use Gerrit), dcheng, foolip, haraken, esprehn
Base URL:
Comments: 209