[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file. Please help us to 
verify the added TEAM/COMPONENT or suggest the correct TEAM/COMPONENT in your 
OWNERS files. Thanks. 

Proposal to add TEAM/COMPONENT information into OWNERS files
http://bit.ly/add-team-component-proposal
Proposal about how to get suggested component for directory. 
http://bit.ly/directory-mapping-proposal
TEAM-COMPONENT mapping
http://bit.ly/component-team-mapping

Additional Information:
Component lists
https://bugs.chromium.org/p/chromium/adminComponents

BUG=679905
Review-Url: https://codereview.chromium.org/2680863003
Cr-Commit-Position: refs/heads/master@{#459591}
Committed: https://chromium.googlesource.com/chromium/src/+/43dd6c1192766bf70c69236bf3cac35aaa3ec79a

[ymzhang1, 2017-02-07 22:47:25]

Hello,

We are adding COMPONENT/TEAM information into OWNERS file. Would you mind
helping us to verify the added TEAM/COMPONENT or suggest the correct
TEAM/COMPONENT in your OWNERS files? Thank you very much!

[ymzhang1, 2017-02-13 22:16:56]

Description was changed from

==========
Add TEAM/COMPONENT

BUG=679905
R=michaeln
==========

to

==========
BUG=679905
R=michaeln
We are adding COMPONENT/TEAM information into OWNERS file. Please help us to
verify the added TEAM/COMPONENT or suggest the correct
TEAM/COMPONENT in your OWNERS files. Thanks. 

Proposal to add TEAM/COMPONENT information into OWNERS files
https://docs.google.com/document/d/1jty6UsFMW9-SYgpQC-ztEc3lltziOQBBArMkROCST...

Proposal about how to get suggested component for directory. 
https://docs.google.com/document/d/1G0UD01aNXdzPsJO-gTLEQiKkDV8XU2XBbi_Gm4g9J...

TEAM-COMPONENT mapping
https://docs.google.com/spreadsheets/d/19JEFMvsxD3eThyGiJRqAjcpx362LHUDdVzICA...
==========

[ymzhang1, 2017-02-13 23:17:05]

Description was changed from

==========
BUG=679905
R=michaeln
We are adding COMPONENT/TEAM information into OWNERS file. Please help us to
verify the added TEAM/COMPONENT or suggest the correct
TEAM/COMPONENT in your OWNERS files. Thanks. 

Proposal to add TEAM/COMPONENT information into OWNERS files
https://docs.google.com/document/d/1jty6UsFMW9-SYgpQC-ztEc3lltziOQBBArMkROCST...

Proposal about how to get suggested component for directory. 
https://docs.google.com/document/d/1G0UD01aNXdzPsJO-gTLEQiKkDV8XU2XBbi_Gm4g9J...

TEAM-COMPONENT mapping
https://docs.google.com/spreadsheets/d/19JEFMvsxD3eThyGiJRqAjcpx362LHUDdVzICA...
==========

to

==========
[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file. Please help us to 
verify the added TEAM/COMPONENT or suggest the correct TEAM/COMPONENT in your 
OWNERS files. Thanks. 

Proposal to add TEAM/COMPONENT information into OWNERS files
http://bit.ly/add-team-component-proposal
Proposal about how to get suggested component for directory. 
http://bit.ly/directory-mapping-proposal
TEAM-COMPONENT mapping
http://bit.ly/component-team-mapping

Additional Information:
Component lists
https://bugs.chromium.org/p/chromium/adminComponents

BUG=679905
==========

[ymzhang1, 2017-03-13 20:52:43]

@michaeln
Remainder to review changes in your OWNERS files. 

+shess@chromium.org to review change in 
third_party/sqlite/OWNERS. 

Thanks!

[ymzhang1, 2017-03-13 21:46:55]

ymzhang@chromium.org changed reviewers:
+ shess@chromium.org

[ymzhang1, 2017-03-13 21:47:08]

+shess@chromium.org to review change in 
third_party/sqlite/OWNERS

[Scott Hess - ex-Googler, 2017-03-13 22:05:19]

https://codereview.chromium.org/2680863003/diff/20001/third_party/sqlite/OWNERS
File third_party/sqlite/OWNERS (right):

https://codereview.chromium.org/2680863003/diff/20001/third_party/sqlite/OWNE...
third_party/sqlite/OWNERS:7: # COMPONENT: UI
This seems like a really odd choice for component, but looking at the list of
components available I have no idea what to even suggest.  I guess this is
really more of an OS feature, almost, or some sort of infrastructure.  leveldb
is under Blink>Storage, even though leveldb is really more broad than that -
perhaps it makes sense for this to be under that component?

[ymzhang1, 2017-03-13 22:15:48]

https://codereview.chromium.org/2680863003/diff/20001/third_party/sqlite/OWNERS
File third_party/sqlite/OWNERS (right):

https://codereview.chromium.org/2680863003/diff/20001/third_party/sqlite/OWNE...
third_party/sqlite/OWNERS:7: # COMPONENT: UI
On 2017/03/13 22:05:19, Scott Hess wrote:
> This seems like a really odd choice for component, but looking at the list of
> components available I have no idea what to even suggest.  I guess this is
> really more of an OS feature, almost, or some sort of infrastructure.  leveldb
> is under Blink>Storage, even though leveldb is really more broad than that -
> perhaps it makes sense for this to be under that component?

Thanks for the suggestion. I updated with Blink>Storage.

[Scott Hess - ex-Googler, 2017-03-13 22:19:04]

lgtm, we can change it later if someone comes up with a better idea.

[Scott Hess - ex-Googler, 2017-03-24 18:17:13]

On 2017/03/13 22:19:04, Scott Hess wrote:
> lgtm, we can change it later if someone comes up with a better idea.

Just cleaning up my review-inbox, should I push the c-q button?

[ymzhang1, 2017-03-24 18:33:01]

On 2017/03/24 18:17:13, Scott Hess wrote:
> On 2017/03/13 22:19:04, Scott Hess wrote:
> > lgtm, we can change it later if someone comes up with a better idea.
> 
> Just cleaning up my review-inbox, should I push the c-q button?

Thanks! I will try to commit. Since I haven't got a lgtm from michaeln@, not
sure the cl could commit successfully. Let's see how it goes.

[ymzhang1, 2017-03-24 18:33:04]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-03-24 18:34:11]

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-03-24 18:50:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-24 18:50:51]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Scott Hess - ex-Googler, 2017-03-24 19:03:24]

On 2017/03/24 18:33:01, ymzhang1 wrote:
> On 2017/03/24 18:17:13, Scott Hess wrote:
> > On 2017/03/13 22:19:04, Scott Hess wrote:
> > > lgtm, we can change it later if someone comes up with a better idea.
> > 
> > Just cleaning up my review-inbox, should I push the c-q button?
> 
> Thanks! I will try to commit. Since I haven't got a lgtm from michaeln@, not
> sure the cl could commit successfully. Let's see how it goes.

Ah, OK, you're right, this extends beyond me.  I should have stepped in earlier
and had you TBR it, but I think Michael should be more responsive this week so
best to wait for him to circle back.

[michaeln, 2017-03-24 21:54:10]

lgtm 2!

[ymzhang1, 2017-03-24 22:04:33]

On 2017/03/24 21:54:10, michaeln wrote:
> lgtm 2!

Many thanks!

[ymzhang1, 2017-03-24 22:04:37]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-03-24 22:05:22]

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-03-24 23:05:42]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490393077831850,
"parent_rev": "784f9c9cea699d809f28946b5df04bdf0487a063", "commit_rev":
"43dd6c1192766bf70c69236bf3cac35aaa3ec79a"}

[commit-bot: I haz the power, 2017-03-24 23:06:31]

Description was changed from

==========
[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file. Please help us to 
verify the added TEAM/COMPONENT or suggest the correct TEAM/COMPONENT in your 
OWNERS files. Thanks. 

Proposal to add TEAM/COMPONENT information into OWNERS files
http://bit.ly/add-team-component-proposal
Proposal about how to get suggested component for directory. 
http://bit.ly/directory-mapping-proposal
TEAM-COMPONENT mapping
http://bit.ly/component-team-mapping

Additional Information:
Component lists
https://bugs.chromium.org/p/chromium/adminComponents

BUG=679905
==========

to

==========
[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file. Please help us to 
verify the added TEAM/COMPONENT or suggest the correct TEAM/COMPONENT in your 
OWNERS files. Thanks. 

Proposal to add TEAM/COMPONENT information into OWNERS files
http://bit.ly/add-team-component-proposal
Proposal about how to get suggested component for directory. 
http://bit.ly/directory-mapping-proposal
TEAM-COMPONENT mapping
http://bit.ly/component-team-mapping

Additional Information:
Component lists
https://bugs.chromium.org/p/chromium/adminComponents

BUG=679905

Review-Url: https://codereview.chromium.org/2680863003
Cr-Commit-Position: refs/heads/master@{#459591}
Committed:
https://chromium.googlesource.com/chromium/src/+/43dd6c1192766bf70c69236bf3ca...
==========

[commit-bot: I haz the power, 2017-03-24 23:06:33]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/43dd6c1192766bf70c69236bf3ca...