[git-webkit] Prompt user for bugzilla credentials during setup

[JonWBedard]

1957c74

[git-webkit] Prompt user for bugzilla credentials during setup
https://bugs.webkit.org/show_bug.cgi?id=236659
<rdar://problem/88980650 >

Reviewed by NOBODY (OOPS!).

* Tools/Scripts/libraries/webkitcorepy/webkitcorepy/credentials.py:
(credentials): Store credentials under "bugs.webkit.org" instead of "https://bugs.webkit.org";
* Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/setup.py:
(Setup.git): Prompt user for bugzilla credentials.
* Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/setup_unittest.py:

[JonWBedard]

The "how" of this patch is pretty straight forward, but the consequences might be somewhat controversial.

• This will block setup on having a valid bugzilla account
• Moving from https:// to in keychain aligns with what Safari does, but will break folks who have already run git-webkit setup

[aj062]

• This will block setup on having a valid bugzilla account

You can consider making this optional, where user has option to skip this step. If skipped git-webkit wouldn't comment on the bug automatically, but rest everything will keep working.

[aproskuryakov]

• This will block setup on having a valid bugzilla account

This does seem very controversial indeed. Mostly because it's something that we'd ask new contributors to do preemptively, before they had a chance to hack on the code and decide if they have something to submit.

Existing contributors would almost certainly already have the credentials in Mac keychain, so we don't need to be asked again.

[JonWBedard]

• This will block setup on having a valid bugzilla account

This does seem very controversial indeed. Mostly because it's something that we'd ask new contributors to do preemptively, before they had a chance to hack on the code and decide if they have something to submit.

Existing contributors would almost certainly already have the credentials in Mac keychain, so we don't need to be asked again.

webkitcorepy.credentials is decently smart, the reason I'm making the shift from https://<hostname> to <hostname> in this PR is because we are checking the existing keychain values, if we have valid bugzilla credentials, folks won't be prompted

[JonWBedard]

Updated PR lets folks cntrl-C out of providing bugzilla credentials.