New issue
Advanced search Search tips

Issue 693334 link

Starred by 1 user
Status: Fixed
Owner:
taku...@chromium.org
Closed: Apr 2017
Cc:
mfo...@chromium.org
taku...@chromium.org
powerb@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

Change the OWNERS for Media Router files in c/b/ui/toolbar/

Project Member Reported by taku...@chromium.org, Feb 17 2017 
Right now the files are owned by the OWNERS of chrome/browser/ui/, and I usually ask msw@ for reviews. His reviews have always been very helpful, but it might make more sense if the Cloudview team owned those files. (I'm not a full committer yet.)

Comment 1 by sko...@chromium.org, Mar 2 2017

Owner: taku...@chromium.org
Status: Assigned (was: Untriaged)
Probably makes sense to add imcheng@ and mfoltz@.
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 17 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fcdd75ae58d61cda92e5f10021739972062e5e0c

commit fcdd75ae58d61cda92e5f10021739972062e5e0c
Author: takumif <takumif@chromium.org>
Date: Mon Apr 17 18:36:13 2017

Add OWNERS for c/b/ui/toolbar/*media_router*

This CL adds OWNERS of c/b/media/router/ as per-file OWNERS for *media_router*.

BUG= 693334 

Review-Url: https://codereview.chromium.org/2824593003
Cr-Commit-Position: refs/heads/master@{#464957}

[modify] https://crrev.com/fcdd75ae58d61cda92e5f10021739972062e5e0c/chrome/browser/ui/toolbar/OWNERS

Comment 3 by taku...@chromium.org, Apr 17 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment