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

Issue 6385001: WATCHLIST for prerender code. (Closed)

Created:
9 years, 11 months ago by cbentzel
Modified:
9 years, 7 months ago
Reviewers:
gavinp, mmenke, tburkard
CC:
chromium-reviews, Nirnimesh
Visibility:
Public.

Description

WATCHLIST for prerender code. BUG=None TEST=watchlists.py chrome/browser/prerender/prerender_resource_handler.cc Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72330

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M WATCHLISTS View 2 chunks +4 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
cbentzel
FYI, adding a watchlist so I get cc'ed on any change which touches stuff in ...
9 years, 11 months ago (2011-01-21 20:27:49 UTC) #1
tburkard
9 years, 11 months ago (2011-01-21 20:31:15 UTC) #2
LGTM

On Fri, Jan 21, 2011 at 12:27 PM, <cbentzel@chromium.org> wrote:

> Reviewers: tburkard, Matt Menke, gavinp,
>
> Message:
> FYI, adding a watchlist so I get cc'ed on any change which touches stuff in
> chrome/browser/prerender. Feel free to add yourself as well.
>
> Description:
> WATCHLIST for prerender code.
>
> BUG=None
> TEST=watchlists.py chrome/browser/prerender/prerender_resource_handler.cc
>
> Please review this at http://codereview.chromium.org/6385001/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files:
>  M WATCHLISTS
>
>
> Index: WATCHLISTS
> diff --git a/WATCHLISTS b/WATCHLISTS
> index
>
c967416a46bc91537c76b6861bf8b8f49e603dc3..c721280dfe6486ccf0efd91183adf183d5790fc7
> 100644
> --- a/WATCHLISTS
> +++ b/WATCHLISTS
> @@ -173,6 +173,9 @@
>     'pepper_api': {
>       'filepath': 'ppapi/c/.*',
>     },
> +    'prerender': {
> +      'filepath': 'chrome/browser/prerender/.*'
> +    },
>   },
>
>   'WATCHLISTS': {
> @@ -233,5 +236,6 @@
>     'labs': ['thakis@chromium.org'],
>     'file_system': ['kinuko@chromium.org'],
>     'pepper_api': ['piman+watch@chromium.org <piman%2Bwatch@chromium.org>
> '],
> +    'prerender': ['cbentzel+watch@chromium.org<cbentzel%2Bwatch@chromium.org>
> '],
>   },
>  }
>
>
>

Powered by Google App Engine
This is Rietveld 408576698