Add nasko+codewatch@chromium.org.

WATCHLISTS

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Watchlist Rules
 # Refer: http://dev.chromium.org/developers/contributing-code/watchlists
 
 # IMPORTANT: The regular expression filepath is tested against each path using
 # re.search, so it is not usually necessary to add .*.
 
@@ skipping 387 matching lines @@
       'filepath': 'mouse_lock',
     },
     'nacl': {
       'filepath': 'chrome/nacl/|chrome/test/nacl/|'\
         'chrome/browser/nacl_host/|chrome/common/nacl|'\
         'native_client_sdk/src/doc/',
     },
     'native_client_sdk': {
       'filepath': '^native_client_sdk/',
     },
+    'navigation': {
+      'filepath': 'content/browser/frame_host|'\
+                  'content/browser/renderer_host/render.*host.*|'\

[Charlie Reis, 2014/03/06 22:52:01]

This will include all of the platform-specific render_widget_host_view_mumble
classes.  Is that intentional?  Navigation shouldn't really affect those, so I
might recommend limiting it to render_view_host.*.

[nasko, 2014/03/06 23:01:39]

Wanted to include RenderProcessHost, but you are right that this is more than
needed. Added it separately.

+                  'content/browser/web_contents/web_contents_impl.*|'\
+                  'content/renderer'

[Charlie Reis, 2014/03/06 22:52:01]

content/renderer is pretty huge, and most isn't navigation related.  Maybe we
should narrow it down?

[nasko, 2014/03/06 23:01:39]

Done.

+    },
     'net': {
       'filepath': 'net/',
     },
     'net_error_list': {
       'filepath': 'net/base/net_error_list.h',
     },
     'net_log': {
       'filepath': 'net/base/net_log|' \
         'chrome/browser/net/.*net_log|' \
         'chrome/browser/net/passive_log_collector|' \
@@ skipping 372 matching lines @@
                 'asvitkine+watch@chromium.org'],
     'metrics_xml_files': ['jar@chromium.org', 'asvitkine+watch@chromium.org'],
     'mojo': ['aa@chromium.org',
              'abarth@chromium.org',
              'ben+mojo@chromium.org',
              'darin@chromium.org',
              'viettrungluu+watch@chromium.org'],
     'mouse_lock': ['scheib+watch@chromium.org'],
     'nacl': ['native-client-reviews@googlegroups.com'],
     'native_client_sdk': ['sbc@chromium.org', 'binji@chromium.org'],
+    'navigation': ['nasko+codewatch@chromium.org'],

[Charlie Reis, 2014/03/06 22:52:01]

Can you put creis+watch@chromium.org as well?  :)

[nasko, 2014/03/06 23:01:39]

I wasn't sure if you are interested ;). Added.

     'net': ['cbentzel+watch@chromium.org'],
     'net_error_list': ['mmenke@chromium.org'],
     'net_log': ['mmenke@chromium.org', 'eroman@chromium.org'],
     'npapi_headers': ['stuartmorgan@chromium.org'],
     'ntp': ['estade+watch@chromium.org', 'dbeam+watch-ntp@chromium.org',
             'pedrosimonetti+watch@chromium.org'],
     'omnibox': ['suzhe@chromium.org'],
     'options': ['dbeam+watch-options@chromium.org'],
     'ozone': ['kalyan.kondapally@intel.com', 'ozone-reviews@chromium.org',
               'rjkroege@chromium.org'],
@@ skipping 18 matching lines @@
                'dennisjeffrey@chromium.org'],
     'remoting': ['chromoting-reviews@chromium.org'],
     'sessions': ['marja+watch@chromium.org'],
     'service_worker': ['tzik@chromium.org',
                        'kinuko+watch@chromium.org',
                        'nhiroki@chromium.org',
                        'horo+watch@chromium.org',
                        'alecflett+watch@chromium.org',
                        'jsbell+serviceworker@chromium.org',
                        'serviceworker-reviews@chromium.org'],
-    'site_instance': ['creis+watch@chromium.org', 'ajwong+watch@chromium.org'],
+    'site_instance': ['creis+watch@chromium.org', 'ajwong+watch@chromium.org',
+                      'nasko+codewatch@chromium.org'],
     'spellcheck': ['groby+spellwatch@chromium.org',
                    'rlp@chromium.org',
                    'rouslan+spellwatch@chromium.org'],
     'streams': ['zork+watch@chromium.org'],
     'sync': ['haitaol+watch@chromium.org',
              'tim+watch@chromium.org',
              'maniscalco+watch@chromium.org'],
     'syncfs': ['kinuko+watch@chromium.org',
                'nhiroki@chromium.org',
                'tzik@chromium.org'],
@@ skipping 31 matching lines @@
     'webrtc_media': ['fischman+watch@chromium.org',
                      'wjia+watch@chromium.org',
                      'mcasas+watch@chromium.org'],
     'website_settings': ['markusheintz@chromium.org'],
     'worker': ['jam@chromium.org'],
     'x11': ['derat+watch@chromium.org',
             'sadrul@chromium.org',
             'yusukes+watch@chromium.org'],
   },
 }