Remove HttpAuth::Scheme enum in favor of a string.

Remove HttpAuth::Scheme enum in favor of a string.

The existing HttpAuthHandlerRegistryFactory already allows for new HTTP Authentication schemes to be added to Chrome's networking stack, but an enum value needed to be change for each new one. By using string's instead of the enum, this is no longer a problem.

It does sacrifice some potential compile-time safety checks, and arguably some performance although I am not worried about that here.

[cbentzel, 2012-09-12 21:17:11]

This is a sketch - still needs some work. I am simply soliciting opinions for
whether this is a good idea. See http://codereview.chromium.org/10907203/ for
motivation - I didn't want net/ code changed at all.

[asanka, 2012-09-12 23:31:32]

On 2012/09/12 21:17:11, cbentzel wrote:
> This is a sketch - still needs some work. I am simply soliciting opinions for
> whether this is a good idea. See http://codereview.chromium.org/10907203/ for
> motivation - I didn't want net/ code changed at all.

I'm okay with this change.  Since auth schemes are defined to be
case-insensitive tokens, we can, by convention keep them as lower-case strings.

[eroman, 2013-06-04 20:50:10]

Can this issue be closed? (shows up in my incoming queue)