Folding IWebClientFactory into IWebClient as a single abstraction

`IWebClientFactory` seems excessive and sort of violates Law of Demeter. `IWebClient` has been refactored to sufficiently represent the simple HTTP GET and POST transactions required by `ErrorLog`. `IWebClientFactory` is therefore removed, yielding a simpler design and fewer types.