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

Side by Side Diff: WATCHLISTS

Issue 2294603003: Added WATCHLISTS definition filesapp for Files App related changes. (Closed)
Patch Set: Remove needless files. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # Watchlist Rules 5 # Watchlist Rules
6 # Refer: http://dev.chromium.org/developers/contributing-code/watchlists 6 # Refer: http://dev.chromium.org/developers/contributing-code/watchlists
7 7
8 # IMPORTANT: The regular expression filepath is tested against each path using 8 # IMPORTANT: The regular expression filepath is tested against each path using
9 # re.search, so it is not usually necessary to add .*. 9 # re.search, so it is not usually necessary to add .*.
10 10
(...skipping 413 matching lines...) Expand 10 before | Expand all | Expand 10 after
424 'fileapi/' 424 'fileapi/'
425 }, 425 },
426 'filebrowse': { 426 'filebrowse': {
427 'filepath': 'file_browser|' \ 427 'filepath': 'file_browser|' \
428 'file_manager|' \ 428 'file_manager|' \
429 'filebrowse' 429 'filebrowse'
430 }, 430 },
431 'filesapp': { 431 'filesapp': {
432 'filepath': 'ui/file_manager' 432 'filepath': 'ui/file_manager'
433 }, 433 },
434 'filesapp_all': {
fukino 2016/08/30 05:49:16 It looks a bit weird to have 'filesapp' and 'files
oka 2016/08/30 06:08:50 Done.
435 'filepath': 'chrome/browser/chromeos/drive|' \
fukino 2016/08/30 05:49:16 filesapp_all looks missing some directories compar
oka 2016/08/30 06:08:50 Done.
436 'chrome/browser/chromeos/extensions/file_manager|'
fukino 2016/08/30 05:49:16 '\' is missing at the end of line. (testing watchl
oka 2016/08/30 06:08:50 Done. I sorted the lines but forgot to run test af
437 'chrome/browser/chromeos/file_manager|' \
438 'ui/file_manager'
439 },
434 'ftp': { 440 'ftp': {
435 'filepath': 'ftp', 441 'filepath': 'ftp',
436 }, 442 },
437 'gcm': { 443 'gcm': {
438 'filepath': 'chrome/browser/services/gcm/|'\ 444 'filepath': 'chrome/browser/services/gcm/|'\
439 'components/gcm_driver/|'\ 445 'components/gcm_driver/|'\
440 'google_apis/gcm/', 446 'google_apis/gcm/',
441 }, 447 },
442 'geolocation': { 448 'geolocation': {
443 'filepath': 'chrome/browser/geolocation/|'\ 449 'filepath': 'chrome/browser/geolocation/|'\
(...skipping 1283 matching lines...) Expand 10 before | Expand all | Expand 10 after
1727 'eme': ['eme-reviews@chromium.org'], 1733 'eme': ['eme-reviews@chromium.org'],
1728 'events': ['tdresser+watch@chromium.org'], 1734 'events': ['tdresser+watch@chromium.org'],
1729 'extension': ['chromium-apps-reviews@chromium.org', 1735 'extension': ['chromium-apps-reviews@chromium.org',
1730 'extensions-reviews@chromium.org'], 1736 'extensions-reviews@chromium.org'],
1731 'feedback_ui': ['apacible+watch@chromium.org'], 1737 'feedback_ui': ['apacible+watch@chromium.org'],
1732 'fileapi': ['kinuko+fileapi@chromium.org', 1738 'fileapi': ['kinuko+fileapi@chromium.org',
1733 'nhiroki@chromium.org', 1739 'nhiroki@chromium.org',
1734 'tzik@chromium.org'], 1740 'tzik@chromium.org'],
1735 'filebrowse': ['rginda+watch@chromium.org'], 1741 'filebrowse': ['rginda+watch@chromium.org'],
1736 'filesapp': ['mtomasz+watch@chromium.org'], 1742 'filesapp': ['mtomasz+watch@chromium.org'],
1743 'filesapp_all': ['oka+watch@chromium.org'],
1737 'ftp': ['phajdan.jr@chromium.org'], 1744 'ftp': ['phajdan.jr@chromium.org'],
1738 'gcm': ['johnme+watch@chromium.org', 1745 'gcm': ['johnme+watch@chromium.org',
1739 'peter@chromium.org', 1746 'peter@chromium.org',
1740 'zea+watch@chromium.org'], 1747 'zea+watch@chromium.org'],
1741 'geolocation': ['mlamouri+watch-geolocation@chromium.org', 1748 'geolocation': ['mlamouri+watch-geolocation@chromium.org',
1742 'mvanouwerkerk@chromium.org'], 1749 'mvanouwerkerk@chromium.org'],
1743 'gfx_geometry': ['cc-bugs@chromium.org'], 1750 'gfx_geometry': ['cc-bugs@chromium.org'],
1744 'gfx_image': ['rsesek+watch@chromium.org'], 1751 'gfx_image': ['rsesek+watch@chromium.org'],
1745 'gn': ['dpranke@chromium.org', 1752 'gn': ['dpranke@chromium.org',
1746 'tfarina@chromium.org'], 1753 'tfarina@chromium.org'],
(...skipping 189 matching lines...) Expand 10 before | Expand all | Expand 10 after
1936 'raymes+watch@chromium.org'], 1943 'raymes+watch@chromium.org'],
1937 'windows_sandboxing': ['pennymac+watch@chromium.org', 1944 'windows_sandboxing': ['pennymac+watch@chromium.org',
1938 'wfh+watch@chromium.org'], 1945 'wfh+watch@chromium.org'],
1939 'x11': ['derat+watch@chromium.org', 1946 'x11': ['derat+watch@chromium.org',
1940 'sadrul@chromium.org', 1947 'sadrul@chromium.org',
1941 'yusukes+watch@chromium.org'], 1948 'yusukes+watch@chromium.org'],
1942 'xml': ['dominicc+watchlist@chromium.org'], 1949 'xml': ['dominicc+watchlist@chromium.org'],
1943 'zoom': ['wjmaclean@chromium.org'], 1950 'zoom': ['wjmaclean@chromium.org'],
1944 }, 1951 },
1945 } 1952 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698