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

Side by Side Diff: WATCHLISTS

Issue 125146: Add some more rules to WATCHLISTS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 11 years, 6 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 | Annotate | Revision Log
« 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) 2006-2009 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2006-2009 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 8
9 { 9 {
10 'WATCHLIST_DEFINITIONS': { 10 'WATCHLIST_DEFINITIONS': {
11 'this_file': { 11 'this_file': {
12 'filepath': '^WATCHLISTS$', 12 'filepath': '^WATCHLISTS$',
13 }, 13 },
14 'valgrind': { 14 'valgrind': {
15 'filepath': 'tools/valgrind/.*', 15 'filepath': 'tools/valgrind/.*',
16 }, 16 },
17 'mac': { 17 'mac': {
18 'filepath': 'cocoa|\.mm$|(_mac|_posix)\.(cc|h)$', 18 'filepath': 'cocoa|\.mm$|(_mac|_posix)\.(cc|h)$',
19 }, 19 },
20 'views': { 20 'views': {
21 # Applies to all files and subdirs within this directory. 21 # Applies to all files and subdirs within this directory.
22 'filepath': 'views/.*', 22 'filepath': 'views/.*',
23 }, 23 },
24 'browser': { 24 'browser': {
25 'filepath': 'chrome/browser/.*', 25 'filepath': 'chrome/browser/.*',
26 }, 26 },
27 'renderer': {
28 'filepath': 'chrome/browser/renderer_host/.*|chrome/renderer/.*|chrome/com mon/render.*',
29 },
30 'webkit': {
31 'filepath': 'webkit/.*',
32 },
33 'plugin': {
34 'filepath': 'chrome/browser/plugin.*|chrome/plugin/.*|chrome/common/plugin .*',
35 },
27 }, 36 },
28 37
29 'WATCHLISTS': { 38 'WATCHLISTS': {
30 'this_file': ['nirnimesh@chromium.org'], 39 'this_file': ['nirnimesh@chromium.org'],
31 'valgrind': ['nirnimesh@chromium.org', 'dank@chromium.org'], 40 'valgrind': ['nirnimesh@chromium.org', 'dank@chromium.org'],
32 'mac': ['jrg@chromium.org'], 41 'mac': ['jrg@chromium.org'],
33 'views': ['ben@chromium.org'], 42 'views': ['ben@chromium.org'],
34 'browser': ['ben@chromium.org'], 43 'browser': ['ben@chromium.org'],
44 'renderer': ['darin@chromium.org'],
45 'webkit': ['darin@chromium.org'],
46 'plugin': ['darin@chromium.org'],
35 }, 47 },
36 } 48 }
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