Skip to content

Commit

Permalink
Create dom_login watch list and added myself to it
Browse files Browse the repository at this point in the history
This is a watch list for all of the files that are core to the DOM based login. There has been a couple of occassions where people have refactored this code not realizing that it is under very active development and thus causing issues with CLs that are currently in the air.

One of the rules, specifically 'chrome/browser/chromeos/login/.*dom.*' is meant for files that have not been created yet in the repo. The CL (6729018) for creating these files should be landing in the next day or two, so I didn't want to go through the bother of having multiple CLs for a change I know is coming.

Patch from Ryan Harrison <rharrison@chromium.org>

BUG=none
TEST=manual


Review URL: http://codereview.chromium.org/6765029

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80284 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
rharrison@chromium.org committed Apr 3, 2011
1 parent 7630872 commit 946b7c1
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions WATCHLISTS
Original file line number Diff line number Diff line change
Expand Up @@ -180,6 +180,14 @@
'chromedriver': {
'filepath': 'chrome/test/webdriver|third_party/webdriver'
},
'dom_login' : {
'filepath': 'chrome/browser/chromeos/frame/.*dom.*' \
'|chrome/browser/chromeos/login/.*dom.*' \
'|chrome/browser/chromeos/login/.*login_display.*' \
'|chrome/browser/chromeos/login/existing_user_controller.*' \
'|chrome/browser/resources/login.html' \
'|chrome/browser/ui/webui/chromeos/login/.*'
},
},

'WATCHLISTS': {
Expand Down Expand Up @@ -246,5 +254,6 @@
'pepper_api': ['piman+watch@chromium.org'],
'prerender': ['cbentzel+watch@chromium.org', 'tburkard+watch@chromium.org'],
'chromedriver': ['kkania@chromium.org'],
'dom_login': ['rharrison@chromium.org'],
},
}

0 comments on commit 946b7c1

Please sign in to comment.