Add my_reviews.py script for perf time.

Add my_reviews.py script for perf time.

Add search interface for rietveld.

R=dpranke@chromium.org
BUG=
TEST=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=102191

[Dirk Pranke, 2011-09-21 20:44:18]

I'm not actually convinced that this is useful enough to keep in the depot, but
I don't feel that strongly about it, either.

http://codereview.chromium.org/7977030/diff/1/my_reviews.py
File my_reviews.py (right):

http://codereview.chromium.org/7977030/diff/1/my_reviews.py#newcode26
my_reviews.py:26: """
This comment mentions with_messages and keys_only, but those aren't actually
parameters to the function?

http://codereview.chromium.org/7977030/diff/1/my_reviews.py#newcode53
my_reviews.py:53: # 'subject'
This list of keys isn't actually used anywhere, is it?

http://codereview.chromium.org/7977030/diff/1/my_reviews.py#newcode71
my_reviews.py:71: """
I'm not sure that lines 65-70 actually make things clearer. I might just omit
them.

[M-A Ruel, 2011-09-21 20:54:11]

Please take another look.

I played with the idea to move this into tools/ subdirectory, along weekly but I
don't think the gain is there.

[Dirk Pranke, 2011-09-21 21:05:42]

http://codereview.chromium.org/7977030/diff/3001/my_reviews.py
File my_reviews.py (right):

http://codereview.chromium.org/7977030/diff/3001/my_reviews.py#newcode55
my_reviews.py:55: # 'subject'
I think I wasn't clear. You list the "available bits of data", but we don't
actually print them and can't sort or filter by them, right? So perhaps we
should not list them in the comment? If someone wants to figure out what can be
retrieved, it seems like they can look it up in rietveld.py; here, it's just
likely to get out of date.

[Marc-Antoine Ruel (Google), 2011-09-21 22:47:12]

On 2011/09/21 21:05:42, Dirk Pranke wrote:
> my_reviews.py:55: # 'subject'
> I think I wasn't clear. You list the "available bits of data", but we don't
> actually print them and can't sort or filter by them, right? So perhaps we
> should not list them in the comment? If someone wants to figure out what can
be
> retrieved, it seems like they can look it up in rietveld.py; here, it's just
> likely to get out of date.

Fixed documentation to be more future proof. :)

[commit-bot: I haz the power, 2011-09-21 22:47:24]

No LGTM from valid reviewers yet.

[Dirk Pranke, 2011-09-21 23:10:03]

lgtm

[commit-bot: I haz the power, 2011-09-21 23:16:14]

Change committed as 102191