Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1738)

Unified Diff: Src/GoogleApis/Apis/Http/HttpClientFactory.cs

Issue 13412046: Reimplement OAuth2 library - Step 1 (Closed) Base URL: https://google-api-dotnet-client.googlecode.com/hg/
Patch Set: minor Created 10 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Src/GoogleApis/Apis/Http/HttpClientFactory.cs
===================================================================
--- a/Src/GoogleApis/Apis/Http/HttpClientFactory.cs
+++ b/Src/GoogleApis/Apis/Http/HttpClientFactory.cs
@@ -24,22 +24,22 @@
namespace Google.Apis.Http
{
- /// <summary> The default implementation of the Http client factory. </summary>
+ /// <summary> The default implementation of the HTTP client factory. </summary>
public class HttpClientFactory : IHttpClientFactory
{
- /// <summary> The class logger. </summary>
+ /// <summary>The class logger.</summary>
private static readonly ILogger Logger = ApplicationContext.Logger.ForType<HttpClientFactory>();
public ConfigurableHttpClient CreateHttpClient(CreateHttpClientArgs args)
{
- // create the handler
+ // Create the handler.
var handler = CreateHandler(args);
var configurableHandler = new ConfigurableMessageHandler(handler)
{
ApplicationName = args.ApplicationName
};
- // create the client
+ // Create the client.
var client = new ConfigurableHttpClient(configurableHandler);
foreach (var initializer in args.Initializers)
{
@@ -50,20 +50,20 @@
}
/// <summary>
- /// Creates an Http message handler by the given arguments. Override this method to mock a message handler.
+ /// Creates a HTTP message handler by the given arguments. Override this method to mock a message handler.
/// </summary>
protected virtual HttpMessageHandler CreateHandler(CreateHttpClientArgs args)
{
var handler = new HttpClientHandler();
- // if the framework supports redirect configuration, set it to false, because ConfigurableMessageHnalder
- // handles redirect
+ // If the framework supports redirect configuration, set it to false, because ConfigurableMessageHnalder
+ // handles redirect.
if (handler.SupportsRedirectConfiguration)
{
handler.AllowAutoRedirect = false;
}
- // if the framework supports automatic decompression and GZip is enabled, set automatic decompression
+ // If the framework supports automatic decompression and GZip is enabled, set automatic decompression.
if (handler.SupportsAutomaticDecompression && args.GZipEnabled)
{
handler.AutomaticDecompression = System.Net.DecompressionMethods.GZip |

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b