Adding OWNERS to various ios directories.

Adding OWNERS to various ios directories.

BUG=None

Review-Url: https://codereview.chromium.org/2850453002
Cr-Commit-Position: refs/heads/master@{#467933}
Committed: https://chromium.googlesource.com/chromium/src/+/f32671c0f445b58612ebba5c7a72de7c7981e8e9

[jif-google, 2017-04-27 11:34:55]

Description was changed from

==========
Adding OWNERS.

BUG=None
==========

to

==========
Adding OWNERS.

BUG=None
==========

[jif-google, 2017-04-27 11:34:55]

jif@google.com changed reviewers:
+ justincohen@chromium.org, lpromero@chromium.org, rohitrao@chromium.org,
sczs@chromium.org, sdefresne@chromium.org, stkhapugin@chromium.org

[jif-google, 2017-04-27 11:35:37]

Description was changed from

==========
Adding OWNERS.

BUG=None
==========

to

==========
Adding a OWNERS to various ios directories.

BUG=None
==========

[jif-google, 2017-04-27 11:35:45]

Description was changed from

==========
Adding a OWNERS to various ios directories.

BUG=None
==========

to

==========
Adding OWNERS to various ios directories.

BUG=None
==========

[jif-google, 2017-04-27 11:36:01]

Patchset #1 (id:1) has been deleted

[jif, 2017-04-27 11:37:48]

Reviewers, PTAL at the files where your names are.

[sdefresne, 2017-04-27 12:30:51]

lgtm for ios/chrome/browser/web_state_list

[rohitrao (ping after 24h), 2017-04-27 12:48:52]

lgtm when the new OWNERS sign off on this

Thanks!

https://codereview.chromium.org/2850453002/diff/20001/ios/chrome/browser/find...
File ios/chrome/browser/find_in_page/OWNERS (left):

https://codereview.chromium.org/2850453002/diff/20001/ios/chrome/browser/find...
ios/chrome/browser/find_in_page/OWNERS:2: rohitrao@chromium.org
I may currently be the best owner for the FIP code, because I just rewrote it
all, but I'm happy to hand off reviews if stk and justincohen are comfortable
taking them.

[lpromero, 2017-04-27 14:49:39]

lgtm

[sczs, 2017-04-27 14:54:28]

lgtm

Since I've been refactoring ToolsMenu, I'd feel comfortable being an OWNER on
there as well if sgty

[justincohen, 2017-04-27 14:57:53]

LGTM

[stkhapugin, 2017-04-28 08:24:54]

lgtm

https://codereview.chromium.org/2850453002/diff/20001/ios/chrome/browser/find...
File ios/chrome/browser/find_in_page/OWNERS (left):

https://codereview.chromium.org/2850453002/diff/20001/ios/chrome/browser/find...
ios/chrome/browser/find_in_page/OWNERS:2: rohitrao@chromium.org
On 2017/04/27 12:48:52, rohitrao (ping after 24h) wrote:
> I may currently be the best owner for the FIP code, because I just rewrote it
> all, but I'm happy to hand off reviews if stk and justincohen are comfortable
> taking them.

I'd rather keep Rohit here, but feel free to add me too, as I've read this code
and
I know the feature overall.

[jif, 2017-04-28 08:30:08]

Patchset #2 (id:40001) has been deleted

[jif, 2017-04-28 08:30:50]

The CQ bit was checked by jif@chromium.org

[jif, 2017-04-28 08:30:51]

The patchset sent to the CQ was uploaded after l-g-t-m from sczs@chromium.org,
lpromero@chromium.org, sdefresne@chromium.org, justincohen@chromium.org,
stkhapugin@chromium.org, rohitrao@chromium.org
Link to the patchset: https://codereview.chromium.org/2850453002/#ps60001
(title: "rohit sole owner of ios/c/b/find_in_page.")

[jif, 2017-04-28 08:31:08]

Addressed comments.

thank you all!

[commit-bot: I haz the power, 2017-04-28 08:31:09]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-28 08:40:33]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1493368250927330,
"parent_rev": "fbaad662ea5f8e62e03ab2febe39b7cd7946d047", "commit_rev":
"f32671c0f445b58612ebba5c7a72de7c7981e8e9"}

[commit-bot: I haz the power, 2017-04-28 08:40:40]

Description was changed from

==========
Adding OWNERS to various ios directories.

BUG=None
==========

to

==========
Adding OWNERS to various ios directories.

BUG=None

Review-Url: https://codereview.chromium.org/2850453002
Cr-Commit-Position: refs/heads/master@{#467933}
Committed:
https://chromium.googlesource.com/chromium/src/+/f32671c0f445b58612ebba5c7a72...
==========

[commit-bot: I haz the power, 2017-04-28 08:40:41]

Committed patchset #2 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/f32671c0f445b58612ebba5c7a72...