Add WebKit bugzilla/git search to my_activity.py

Add WebKit bugzilla/git search to my_activity.py

This is unfortunately imperfect in a number of ways:

* Needs to get info from git commits, so needs a local WebKit git repo, and an explicit option to find it.
* Bugzilla doesn't return nearly as much information as other bug servers, and individual issues would have to be hit (and parsed) to get it.
* Searching for comments includes any issue ever commented on that was changed over the time period requested, so may have false positives.
* Reviews themselves can't be searched from Bugzilla, so can only be detected as "reviews" once landed and "issues" until then.

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

[cjhopman, 2012-11-04 16:32:03]

This is great, multiple people had requested WebKit integration (there's a bug
filed at crbug.com/158933).

Are all bugs on WebKit's buganizer public, or are there some that need
authentication to view? If there are some private, Just adding a TODO/mentioning
it on the bug after landing this would be fine.

https://codereview.chromium.org/11365078/diff/2001/my_activity.py
File my_activity.py (right):

https://codereview.chromium.org/11365078/diff/2001/my_activity.py#newcode23
my_activity.py:23: import csv
Nit: csv after cookielib

https://codereview.chromium.org/11365078/diff/2001/my_activity.py#newcode41
my_activity.py:41: # Imported later, once options are set
Nit: full stop

https://codereview.chromium.org/11365078/diff/2001/my_activity.py#newcode74
my_activity.py:74: # commit-bot is author for contributors who are not
committers
Nit: full stop

https://codereview.chromium.org/11365078/diff/2001/my_activity.py#newcode592
my_activity.py:592: # TODO(enne): should convert full names to usernames via
CommitterList
Nit: full stop

https://codereview.chromium.org/11365078/diff/2001/my_activity.py#newcode676
my_activity.py:676: self.webkit_repo = None
I've missed how this disables the webkit checks. It appears that this will still
hit line 541, get repo=None and then I guess maybe just get an error/no output
from the git cmd?

https://codereview.chromium.org/11365078/diff/2001/my_activity.py#newcode796
my_activity.py:796: # If a webkit bug was a review, don't list it as an issue
Nit: full stop

https://codereview.chromium.org/11365078/diff/2001/my_activity.py#newcode833
my_activity.py:833: help='WebKit repository, default=%default')
Maybe note here that it should be a local path?

[enne (OOO), 2012-11-04 23:35:47]

On 2012/11/04 16:32:03, cjhopman wrote:
> Are all bugs on WebKit's buganizer public, or are there some that need
> authentication to view? If there are some private, Just adding a
TODO/mentioning
> it on the bug after landing this would be fine.

Yeah, there are some security bugs that aren't visible.  I'll make sure the bug
is updated to say that after this lands.

https://codereview.chromium.org/11365078/diff/2001/my_activity.py
File my_activity.py (right):

https://codereview.chromium.org/11365078/diff/2001/my_activity.py#newcode676
my_activity.py:676: self.webkit_repo = None
On 2012/11/04 16:32:03, cjhopman wrote:
> I've missed how this disables the webkit checks. It appears that this will
still
> hit line 541, get repo=None and then I guess maybe just get an error/no output
> from the git cmd?

You're quite right.  This used to work, but then a last minute edit broke it. 
New revision should be robust to disabled webkit checks.

[cjhopman, 2012-11-05 15:38:34]

lgtm.

[cmp, 2012-11-05 22:16:50]

lgtm

[commit-bot: I haz the power, 2012-11-06 01:09:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/enne@chromium.org/11365078/5001

[commit-bot: I haz the power, 2012-11-06 01:11:41]

Change committed as 166100