Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(479)

Side by Side Diff: net/http/http_stream_request.cc

Issue 5599004: Initialize HttpStreamRequest::alternate_protocol_ to avoid Valgrind false rep... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « net/http/http_alternate_protocols.h ('k') | tools/valgrind/memcheck/suppressions.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/http/http_stream_request.h" 5 #include "net/http/http_stream_request.h"
6 6
7 #include "base/stl_util-inl.h" 7 #include "base/stl_util-inl.h"
8 #include "base/string_number_conversions.h" 8 #include "base/string_number_conversions.h"
9 #include "base/string_util.h" 9 #include "base/string_util.h"
10 #include "base/stringprintf.h" 10 #include "base/stringprintf.h"
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 connection_(new ClientSocketHandle), 56 connection_(new ClientSocketHandle),
57 factory_(factory), 57 factory_(factory),
58 delegate_(NULL), 58 delegate_(NULL),
59 next_state_(STATE_NONE), 59 next_state_(STATE_NONE),
60 pac_request_(NULL), 60 pac_request_(NULL),
61 using_ssl_(false), 61 using_ssl_(false),
62 using_spdy_(false), 62 using_spdy_(false),
63 force_spdy_always_(HttpStreamFactory::force_spdy_always()), 63 force_spdy_always_(HttpStreamFactory::force_spdy_always()),
64 force_spdy_over_ssl_(HttpStreamFactory::force_spdy_over_ssl()), 64 force_spdy_over_ssl_(HttpStreamFactory::force_spdy_over_ssl()),
65 spdy_certificate_error_(OK), 65 spdy_certificate_error_(OK),
66 alternate_protocol_(HttpAlternateProtocols::UNINITIALIZED),
66 establishing_tunnel_(false), 67 establishing_tunnel_(false),
67 was_alternate_protocol_available_(false), 68 was_alternate_protocol_available_(false),
68 was_npn_negotiated_(false), 69 was_npn_negotiated_(false),
69 preconnect_delegate_(NULL), 70 preconnect_delegate_(NULL),
70 num_streams_(0), 71 num_streams_(0),
71 ALLOW_THIS_IN_INITIALIZER_LIST(method_factory_(this)) { 72 ALLOW_THIS_IN_INITIALIZER_LIST(method_factory_(this)) {
72 if (HttpStreamFactory::use_alternate_protocols()) 73 if (HttpStreamFactory::use_alternate_protocols())
73 alternate_protocol_mode_ = kUnspecified; 74 alternate_protocol_mode_ = kUnspecified;
74 else 75 else
75 alternate_protocol_mode_ = kDoNotUseAlternateProtocol; 76 alternate_protocol_mode_ = kDoNotUseAlternateProtocol;
(...skipping 968 matching lines...) Expand 10 before | Expand all | Expand 10 after
1044 base::TimeDelta::FromMinutes(6), 1045 base::TimeDelta::FromMinutes(6),
1045 100); 1046 100);
1046 break; 1047 break;
1047 default: 1048 default:
1048 NOTREACHED(); 1049 NOTREACHED();
1049 break; 1050 break;
1050 } 1051 }
1051 } 1052 }
1052 1053
1053 } // namespace net 1054 } // namespace net
OLDNEW
« no previous file with comments | « net/http/http_alternate_protocols.h ('k') | tools/valgrind/memcheck/suppressions.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698