Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(229)

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2655783002: add TEAM/COMPONENT (Closed)

Created:
3 years, 11 months ago by ymzhang1
Modified:
3 years, 10 months ago
CC:
chromium-reviews, feature-media-reviews_chromium.org, jam, abarth-chromium, haraken, darin-cc_chromium.org, blink-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[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: List of components https://bugs.chromium.org/p/chromium/adminComponents BUG=679905 R=guidou, tommi CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Patch Set 1 #

Patch Set 2 : Add TEAM/COMPONENT #

Total comments: 6
Unified diffs Side-by-side diffs Delta from patch set Stats (+42 lines, -0 lines) Patch
M chrome/browser/media/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M chrome/browser/media/webrtc/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M chrome/browser/speech/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M chrome/renderer/media/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M content/browser/resources/media/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M content/browser/speech/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M content/common/media/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M content/public/test/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M content/test/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M courgette/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M media/audio/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/audio_output_devices/OWNERS View 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/mediastream/OWNERS View 1 1 chunk +3 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/peerconnection/OWNERS View 1 chunk +2 lines, -0 lines 1 comment Download
M third_party/WebKit/Source/modules/speech/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/srcobject/OWNERS View 1 chunk +2 lines, -0 lines 1 comment Download
M third_party/libjingle/OWNERS View 1 1 chunk +2 lines, -0 lines 1 comment Download
M third_party/libjingle_xmpp/OWNERS View 1 1 chunk +3 lines, -0 lines 1 comment Download
M third_party/openh264/OWNERS View 1 1 chunk +2 lines, -0 lines 1 comment Download
M third_party/webrtc_overrides/OWNERS View 1 1 chunk +2 lines, -0 lines 1 comment Download

Messages

Total messages: 8 (5 generated)
ymzhang1
Hello, We are adding COMPONENT/TEAM information into OWNERS file. Would you mind helping us to ...
3 years, 11 months ago (2017-01-24 21:37:42 UTC) #1
ymzhang1
Hello, We are adding COMPONENT/TEAM information into OWNERS file. Would you mind helping us to ...
3 years, 10 months ago (2017-02-07 20:45:42 UTC) #6
Guido Urdaneta
3 years, 10 months ago (2017-02-08 12:45:23 UTC) #7 
https://codereview.chromium.org/2655783002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/peerconnection/OWNERS (right):

https://codereview.chromium.org/2655783002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/peerconnection/OWNERS:5: # COMPONENT:
UI>Notifications
This one should be Blink>WebRTC

https://codereview.chromium.org/2655783002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/srcobject/OWNERS (right):

https://codereview.chromium.org/2655783002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/srcobject/OWNERS:3: # COMPONENT:
Blink>HTML>Object
This one should be Blink>Media

https://codereview.chromium.org/2655783002/diff/20001/third_party/libjingle/O...
File third_party/libjingle/OWNERS (right):

https://codereview.chromium.org/2655783002/diff/20001/third_party/libjingle/O...
third_party/libjingle/OWNERS:7: # COMPONENT: Blink>WebRTC
I think this should be Internals>WebRTC, but wait for input from OWNERS.

https://codereview.chromium.org/2655783002/diff/20001/third_party/libjingle_x...
File third_party/libjingle_xmpp/OWNERS (right):

https://codereview.chromium.org/2655783002/diff/20001/third_party/libjingle_x...
third_party/libjingle_xmpp/OWNERS:8: # COMPONENT: Blink>GetUserMedia
I think this should be Internals>WebRTC, but wait for input from OWNERS.

https://codereview.chromium.org/2655783002/diff/20001/third_party/openh264/OW...
File third_party/openh264/OWNERS (right):

https://codereview.chromium.org/2655783002/diff/20001/third_party/openh264/OW...
third_party/openh264/OWNERS:4: # COMPONENT: Blink>WebRTC
I think this should be Internals>WebRTC, but wait for input from OWNERS.

https://codereview.chromium.org/2655783002/diff/20001/third_party/webrtc_over...
File third_party/webrtc_overrides/OWNERS (right):

https://codereview.chromium.org/2655783002/diff/20001/third_party/webrtc_over...
third_party/webrtc_overrides/OWNERS:5: # COMPONENT: Blink>WebRTC
I think this should be Internals>WebRTC, but wait for input from OWNERS.

Powered by Google App Engine
This is Rietveld 408576698